Edgewall Software
Modify ↓

Opened 20 years ago

Closed 20 years ago

#923 closed defect (fixed)

"trac_admin <database> permission list" should list REPORT_SQL_VIEW

Reported by: Juanma Barranquero Owned by: Christopher Lenz
Priority: low Milestone: 0.8
Component: admin/console Version: devel
Severity: trivial Keywords: documentation
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description (last modified by Christopher Lenz)

REPORT_SQL_VIEW privilege was implemented back in [407], but is not listed among the output of "permission list".

  • scripts/trac-admin

     
    407407        print ' ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW, '
    408         print ' CONFIG_VIEW and TRAC_ADMIN.'
     408        print ' REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.'
    409409        print

Attachments (0)

Change History (4)

comment:1 by Christopher Lenz, 20 years ago

Description: modified (diff)

I'm pretty sure this would also require a change to the trac-admin unit tests.

comment:2 by Juanma Barranquero, 20 years ago

Something like this?

  • trac/tests/tracadmin-tests.txt

     
    8484 ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW,
    85  CONFIG_VIEW and TRAC_ADMIN.
     85 REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.
    8686
    … …  
    116116 ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW,
    117  CONFIG_VIEW and TRAC_ADMIN.
     117 REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.
    118118
    … …  
    149149 ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW,
    150  CONFIG_VIEW and TRAC_ADMIN.
     150 REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.
    151151
    … …  
    179179 ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW,
    180  CONFIG_VIEW and TRAC_ADMIN.
     180 REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.
    181181
    … …  
    208208 ROADMAP_VIEW, ROADMAP_ADMIN, TIMELINE_VIEW, SEARCH_VIEW,
    209  CONFIG_VIEW and TRAC_ADMIN.
     209 REPORT_SQL_VIEW, CONFIG_VIEW and TRAC_ADMIN.
    210210

comment:3 by Christopher Lenz, 20 years ago

Owner: changed from daniel to Christopher Lenz
Status: new → assigned

comment:4 by Christopher Lenz, 20 years ago

Resolution: → fixed
Status: assigned → closed

Patch applied in [1051]. Many thanks!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Christopher Lenz.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Christopher Lenz to the specified user.

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