Modify

Opened 12 years ago

Last modified 4 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:
Priority: normal Component: SensitiveTicketsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

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 (2)

comment:1 Changed 12 years ago by Daniel Kahn Gillmor

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

comment:2 Changed 4 years ago by Ryan J Ollos

Owner: Daniel Kahn Gillmor 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.