Edgewall Software

Changes between Initial Version and Version 4 of Ticket #11166


Ignore:
Timestamp:
Jun 5, 2013, 10:23:14 AM (6 years ago)
Author:
Ryan J Ollos <ryan.j.ollos@…>
Comment:

I'm expanding scope of this ticket to deal with users not having REPORT_VIEW.

Currently the wiki TracLink is the only TracLink to utilize the forbidden class, but it looks like we could use it here as well.

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #11166

    • Property Keywords permissions added
    • Property Summary TracLinks to non-existent reports should have the "missing" class and should not have an href attributeTracLinks to non-existent and forbidden reports should not be followable
  • Ticket #11166 – Description

    initial v4  
    1 For example, {10000}. The HTML is:
    2 {{{
    3 #!text/html
     1For example, the non-existent report {10000} is rendered as a followable link. The HTML is:
     2{{{#!text/html
    43<a class="report" href="/report/10000">{10000}</a>
    54}}}
    65but should be:
    7 {{{
    8 #!text/html
     6{{{#!text/html
    97<a class="missing report">{10000}</a>
    108}}}
    119resulting in:
    1210
    13 {{{
    14 #!html
     11{{{#!html
    1512<a class="missing report">{10000}</a>
    1613}}}
     14
     15Also, when the user doesn't have permission to view a report the !TracLink is rendered as a link that can be followed, but the behavior should be similar to when the report doesn't exist.