Edgewall Software

Changes between Version 2 and Version 3 of Ticket #11145, comment 16


Ignore:
Timestamp:
Jul 23, 2014, 1:50:31 AM (10 years ago)
Author:
Ryan J Ollos

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11145, comment 16

    v2 v3  
    77 On a related note, there is some odd-looking code in the preparation of `show_email_addresses`. Does anyone have an idea of what type of Exceptions might be encountered here?: [browser:/trunk/trac/web/chrome.py@12930:878#L875]. The code was added in [8785#file4].
    88 
    9 There is a possible problem that became more clear after these changes. For whatever reason, the usernames are replaced with email addresses in the RSS feed when possible. However, `EMAIL_VIEW` permission is not checked when rendering the RSS feed. Only the `show_email_addresses` option is considered: [browser:/trunk/trac/web/chrome.py@12930:1118#L1115]. The email addresses won't be added to the map if `show_email_addresses` is `false` Is this a defect?
     9There is a possible problem that became more clear after these changes. For whatever reason, the usernames are replaced with email addresses in the RSS feed when possible. However, `EMAIL_VIEW` permission is not checked when rendering the RSS feed. Only the `show_email_addresses` option is considered: [browser:/trunk/trac/web/chrome.py@12930:1118#L1115]. The email addresses won't be added to the map if `show_email_addresses` is `false`. Is this a defect?