id,summary,reporter,owner,description,type,status,priority,milestone,component,version,severity,resolution,keywords,cc,branch,changelog,apichanges,internalchanges 11697,ReportModule uses magic number -1 where None should be used,Ryan J Ollos,Ryan J Ollos,"I found this refactoring from a while back on one of my local branches. The use of `-1` in `ReportModule.process_request` should be replaced with `None` in order to be more Pythonic. At least, that is the first thought I had.",defect,closed,normal,1.1.2,report system,,normal,fixed,refactoring,,," * Fine-grained permissions can be used to restrict access to the //Report List// page using the resource id `-1`. * Fine-grained permission checks are implemented on the //Report List// resource (for !ReportModule) and the ticket realm (for !QueryModule) before displaying the mainnav and contextual navigation items. * Fine-grained permissions can be used to grant the `REPORT_CREATE` permission for the report realm (using `[report:*]` in the authz file).","Added class attribute `REPORT_LIST_ID` to the `ReportModule` class, containing the resource id of the //Report List// page.",