Opened 6 years ago

Last modified 6 years ago

#9835 new defect

Sensitive checkbox should be hidden for users who aren't allowed to check it

Reported by: Daniel Kahn Gillmor Owned by: Daniel Kahn Gillmor
Priority: normal Component: SensitiveTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12


as of r11287, the SensitiveTicketsPlugin prevents people from setting a ticket to sensitive if that would cause the ticket itself to be hidden from them.

We should hide the checkbox in the "ticket properties" section if we know that the user is going to get an error if they check it.

I believe one way to do this would be to extend ITemplateStreamFilter to conditionally either inject css to set the objects to "display:none" or to simply remove the objects entirely. I don't know how to do this yet, though. I would welcome patches!

Some example CSS if the checkbox needed to be hidden might be:

label[for="field-sensitive"]: { display: none; }
input#field-sensitive: { display: none; }

Attachments (0)

Change History (1)

comment:1 Changed 6 years ago by Daniel Kahn Gillmor

Note that this is probably only relevant when limit_sensitivity is enabled in trac.ini

Modify Ticket

Change Properties
Set your email in Preferences
as new The owner will remain Daniel Kahn Gillmor.

Add Comment

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

Note: See TracTickets for help on using tickets.