Modify

Opened 7 years ago

Closed 5 years ago

Last modified 2 years ago

#8114 closed enhancement (fixed)

Require REPORT_VIEW permission to access ticket stats page, and eliminate TSTATS_VIEW permission

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Component: TracTicketStatsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.11

Description

I'm still thinking about this, but as noted in comment:7:ticket:3594, I'm not sure it makes sense to have another permission for presenting in another way information that is already available given other permissions.

The number of opened and closed tickets in a specified time period can normally be viewed from the timeline. That is, this is really just a graphical presentation of the timeline filter Tickets opened and closed, as well as showing the number of open tickets on a certain date, which can be viewed if one has REPORT_VIEW permission. Well, the latter is not exactly equivalent because historical information is shown on the graph, but it is close.

There are a couple of options here:

  • Make the required permission configurable, with the default being TSTAT_VIEW.
  • Make the required permission configurable, with the default being REPORT_VIEW and/or TSTAT_VIEW.
  • Drop the TSTAT_VIEW permission, and require TIMELINE_VIEW and/or REPORT_VIEW permissions.

I also wonder if we might remove the mainnav link and add metanav links to the timeline and/or report pages.

Attachments (0)

Change History (4)

comment:1 Changed 5 years ago by Ryan J Ollos

Status: newassigned
Summary: Require REPORT_VIEW permission to access ticket stats page, and elminate TSTATS_VIEW permissionRequire REPORT_VIEW permission to access ticket stats page, and eliminate TSTATS_VIEW permission

comment:2 Changed 5 years ago by Ryan J Ollos

Resolution: fixed
Status: assignedclosed

(In [13100]) Fixes #8114: Removed TSTATS_VIEW permission. REPORT_VIEW is now required for the "Ticket Stats" tab to be present and to view the graph.

comment:3 in reply to:  description Changed 5 years ago by Ryan J Ollos

Replying to rjollos:

I also wonder if we might remove the mainnav link and add metanav links to the timeline and/or report pages.

I'm still considering this, but it would be handled in another ticket.

comment:4 Changed 2 years ago by Ryan J Ollos

TICKET_VIEW, rather than REPORT_VIEW, is required after [14703].

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Ryan J Ollos.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.