Modify

Opened 8 years ago

Last modified 9 months ago

#6259 new enhancement

Email address should be a mailto link

Reported by: matt.caron@… 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)

um_email_links.patch.zip (674 bytes) - added by matt.caron@… 8 years ago.
Patch to make email addresses links (zip to fool anti-spam filter)
um_email_links.patch (1.3 KB) - added by lkraav 7 years ago.
re-uploading without zip

Download all attachments as: .zip

Change History (9)

Changed 8 years ago by matt.caron@…

Attachment: um_email_links.patch.zip added

Patch to make email addresses links (zip to fool anti-spam filter)

Changed 7 years ago by lkraav

Attachment: um_email_links.patch added

re-uploading without zip

comment:1 Changed 7 years ago by lkraav

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 in reply to:  1 Changed 6 years ago by Steffen Hoffmann

Keywords: permission email obfuscation added
Owner: changed from Catalin BALAN to Steffen Hoffmann
Status: newassigned

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 6 years ago by Steffen Hoffmann

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 6 years ago by Steffen Hoffmann

(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 6 years ago by Steffen Hoffmann

(In [10927]) UserManagerPlugin: Post-chance template cleanup, refs #6259.

Converting from tab to space indentation, removing mixed whitespace too.

comment:6 Changed 9 months ago by Ryan J Ollos

Status: assignednew

comment:7 Changed 9 months ago by Ryan J Ollos

Owner: Steffen Hoffmann deleted

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.