Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#10941 closed defect (fixed)

Change permission action definition for intuitive inheritance

Reported by: hasienda Owned by: hasienda
Priority: normal Component: VotePlugin
Severity: normal Keywords: permission
Cc: Trac Release:


From description:

If a user has a valid session (created through the Preferences panel) and the VOTE_MODIFY permission, they will be able to vote.

This is not true, because he/she won't currently see the down-/up-vote links at all. Therefore more intuitive permission inheritance is suggested:

  • a/tracvote/

    diff -u a/tracvote/ b/tracvote/ 
    118118    # IPermissionRequestor method
    119119    def get_permission_actions(self):
    120         return ['VOTE_VIEW', 'VOTE_MODIFY']
     120        action = 'VOTE_VIEW'
     121        return [('VOTE_MODIFY', [action]), action]
    122123    ### ITemplateProvider methods

For what its worth, if you assign someone VOTE_MODIFY, it does actually rely on seeing the vote links, doesn't it?

Attachments (0)

Change History (6)

comment:1 Changed 4 years ago by rjollos

Yes, make sense. Thanks for catching that. Please feel free to commit.

Wiring up the unit tests has been on my TODO list for this plugin. For this change though, I don't imagine that a unit test is needed. A functional test would be nice, but better to get the fix in there and we'll add the functional test infrastructure when time allows.

comment:2 Changed 4 years ago by hasienda

  • Owner changed from rjollos to hasienda

comment:3 Changed 4 years ago by hasienda

  • Resolution set to fixed
  • Status changed from new to closed

(In [12771]) TracVote: Allow for more intuitive permission setup by action inheritance, closes #10941.

comment:4 Changed 4 years ago by hasienda

(In [12775]) VotePlugin: Add an unit test harness, refs #10941.

Including more author comments and explicit license file for clarity.

comment:5 Changed 4 years ago by rjollos

(In [12814]) Refs #10941: Added license header to

comment:6 Changed 4 years ago by rjollos

(In [12815]) Refs #10941: Replaced a single line containing tab-indentation with space-indentation.

Add Comment

Modify Ticket

as closed The owner will remain hasienda.
The resolution will be deleted. Next status will be 'reopened'.

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

Note: See TracTickets for help on using tickets.