Edgewall Software

Ticket #923 (closed defect: fixed)

Opened 4 years ago

Last modified 4 years ago

"trac_admin <database> permission list" should list REPORT_SQL_VIEW

Reported by: Juanma Barranquero Owned by: cmlenz
Priority: low Milestone: 0.8
Component: admin/console Version: devel
Severity: trivial Keywords: documentation
Cc:

Description (last modified by cmlenz) (diff)

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

Change History

Changed 4 years ago by cmlenz

  • description modified (diff)

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

Changed 4 years ago by Juanma Barranquero

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 

Changed 4 years ago by cmlenz

  • owner changed from daniel to cmlenz
  • status changed from new to assigned

Changed 4 years ago by cmlenz

  • status changed from assigned to closed
  • resolution set to fixed

Patch applied in [1051]. Many thanks!

Add/Change #923 ("trac_admin <database> permission list" should list REPORT_SQL_VIEW)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
to The owner will change from cmlenz. Next status will be 'closed'
 
Note: See TracTickets for help on using tickets.