Edgewall Software
Modify

Opened 8 years ago

#12595 new enhancement

TracAdmin should label invalid permissions

Reported by: Ryan J Ollos Owned by:
Priority: normal Milestone: next-major-releases
Component: admin/console Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

In #10752 we added styling for invalid permissions on the Permissions Admin page.

TracAdmin gives no indication if a permission is invalid:

$ trac-admin ../tracenvs/proj-milestonetest permission list user2

User   Action
-------------------------
user2  INVALID_PERMISSION
user2  TICKET_APPEND
user2  TICKET_CHGPROP
user2  TICKET_CREATE
user2  TICKET_MODIFY
user2  WIKI_CREATE
user2  WIKI_MODIFY


Available actions:
 BROWSER_VIEW, CHANGESET_VIEW, CONFIG_VIEW, EMAIL_VIEW, FILE_VIEW,
 LOG_VIEW, MILESTONE_ADMIN, MILESTONE_CREATE, MILESTONE_DELETE,
 MILESTONE_MODIFY, MILESTONE_VIEW, PERMISSION_ADMIN, PERMISSION_GRANT,
 PERMISSION_REVOKE, REPORT_ADMIN, REPORT_CREATE, REPORT_DELETE,
 REPORT_MODIFY, REPORT_SQL_VIEW, REPORT_VIEW, ROADMAP_ADMIN, ROADMAP_VIEW,
 SEARCH_VIEW, TICKET_ADMIN, TICKET_APPEND, TICKET_BATCH_MODIFY,
 TICKET_CHGPROP, TICKET_CREATE, TICKET_EDIT_CC, TICKET_EDIT_COMMENT,
 TICKET_EDIT_DESCRIPTION, TICKET_MODIFY, TICKET_VIEW, TIMELINE_VIEW,
 TRAC_ADMIN, VERSIONCONTROL_ADMIN, WIKI_ADMIN, WIKI_CREATE, WIKI_DELETE,
 WIKI_MODIFY, WIKI_RENAME, WIKI_VIEW

I think that trac-admin properly lists INVALID_PERMISSION as being assigned to user2, and does not include INVALID_PERMISSION in the list of Available actions. However, there should be a label on INVALID_PERMISSION, such as the following:

$ trac-admin ../tracenvs/proj-milestonetest permission list user2

User   Action
-------------------------
user2  INVALID_PERMISSION *
user2  TICKET_APPEND
user2  TICKET_CHGPROP
user2  TICKET_CREATE
user2  TICKET_MODIFY
user2  WIKI_CREATE
user2  WIKI_MODIFY

(*) Permission is no longer defined

Available actions:
 BROWSER_VIEW, CHANGESET_VIEW, CONFIG_VIEW, EMAIL_VIEW, FILE_VIEW,
 LOG_VIEW, MILESTONE_ADMIN, MILESTONE_CREATE, MILESTONE_DELETE,
 MILESTONE_MODIFY, MILESTONE_VIEW, PERMISSION_ADMIN, PERMISSION_GRANT,
 PERMISSION_REVOKE, REPORT_ADMIN, REPORT_CREATE, REPORT_DELETE,
 REPORT_MODIFY, REPORT_SQL_VIEW, REPORT_VIEW, ROADMAP_ADMIN, ROADMAP_VIEW,
 SEARCH_VIEW, TICKET_ADMIN, TICKET_APPEND, TICKET_BATCH_MODIFY,
 TICKET_CHGPROP, TICKET_CREATE, TICKET_EDIT_CC, TICKET_EDIT_COMMENT,
 TICKET_EDIT_DESCRIPTION, TICKET_MODIFY, TICKET_VIEW, TIMELINE_VIEW,
 TRAC_ADMIN, VERSIONCONTROL_ADMIN, WIKI_ADMIN, WIKI_CREATE, WIKI_DELETE,
 WIKI_MODIFY, WIKI_RENAME, WIKI_VIEW

Attachments (1)

Screen Shot 2016-09-20 at 13.38.27.png (54.1 KB ) - added by Ryan J Ollos 8 years ago.

Download all attachments as: .zip

Change History (1)

by Ryan J Ollos, 8 years ago

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The ticket will remain with no owner.
The ticket will be disowned.
as The resolution will be set. Next status will be 'closed'.
The owner will be changed from (none) to anonymous. Next status will be 'assigned'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.