Opened 14 years ago
Last modified 7 years ago
#6259 new enhancement
Email address should be a mailto link
| Reported by: | Owned by: | ||
|---|---|---|---|
| Priority: | normal | Component: | UserManagerPlugin |
| Severity: | minor | Keywords: | permission email obfuscation |
| Cc: | Trac Release: | 0.11 |
Description
The email addresses displayed in the macros should be mailto links, so I can just click on them and my system does what it should.
Attachments (2)
Change History (9)
Changed 14 years ago by
| Attachment: | um_email_links.patch.zip added |
|---|
comment:1 follow-up: 2 Changed 14 years ago by
this patch works well, any particular reason why it isn't in trunk?
perhaps one could tie this with EMAIL_VIEW, too, somehow?
comment:2 Changed 12 years ago by
| Keywords: | permission email obfuscation added |
|---|---|
| Owner: | changed from Catalin BALAN to Steffen Hoffmann |
| Status: | new → assigned |
Replying to lkraav:
this patch works well, any particular reason why it isn't in trunk?
Yes, the author is still interested but busy with other stuff (told me so recently).
perhaps one could tie this with EMAIL_VIEW, too, somehow?
You mean permission-dependent address formatting (truncating)? This would be Chrome.format_author in trac.web.chrome, right? In general, yes, this should be there for privacy and consistency with Trac core.
In it's current state most of UM is for TRAC_ADMIN users only, so there would be no visible effect, since the top permission inherits EMAIL_VIEW anyway. However this is about WikiMacros, so would be really sensible in the context of #9506, but at publicly available pages only.
comment:3 Changed 12 years ago by
Note: IMHO private method _get_cells() in tracusermanager.profile.macros is the best place the let email obfuscation happen. This method however has an exact duplicate method in tracusermanager.admin - most probably a place to start code re-factoring.
comment:4 Changed 12 years ago by
(In [10926]) UserManagerPlugin: Add mailto: email links in UserProfilesList output, refs #6259.
Thanks for Matt Caron's patch, extended by code to hide emails as suggested by Leho Kraav.
comment:5 Changed 12 years ago by
(In [10927]) UserManagerPlugin: Post-chance template cleanup, refs #6259.
Converting from tab to space indentation, removing mixed whitespace too.
comment:6 Changed 7 years ago by
| Status: | assigned → new |
|---|
comment:7 Changed 7 years ago by
| Owner: | Steffen Hoffmann deleted |
|---|
Patch to make email addresses links (zip to fool anti-spam filter)