Modify ↓

Opened 4 years ago

Closed 4 years ago

#13942 closed defect (fixed)

[PATCH] Second and subsequent of rules are ignored

Reported by: Andrew G. Khodotov Owned by: Cauly
Priority: normal Component: TracConfigurablePermissionPlugin
Severity: critical Keywords:
Cc: Trac Release: 1.2

Description

Only first rule will be appyed to a ticket. In this way Example 1 in the wiki is incorrect: second variation with two rules is unfunctional, unlike the first.

Simple test: 

rule1 = ticket, TICKET_VIEW, *, *, deny
rule2 = ticket, TICKET_VIEW, *, *, allow

Always denied. 

rule1 = ticket, TICKET_VIEW, *, *, allow
rule2 = ticket, TICKET_VIEW, *, *, deny

Always allowed.

Attached patch has solve the problem with 'ticket' realm. I think with 'wiki' will be the same bug, but I do not use this realm.

Attachments (1)

configurable_permission_SeveralRules.patch​ (691 bytes) - added by Andrew G. Khodotov 4 years ago.

Download all attachments as: .zip

Change History (2)

Changed 4 years ago by Andrew G. Khodotov

Attachment: configurable_permission_SeveralRules.patch​ added

comment:1 Changed 4 years ago by Cauly

Resolution: → fixed
Status: new → closed

Hi, I merged the patch and applied the change to wiki also. Thanks a lot for your feedback!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Cauly.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment

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

Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.