Edgewall Software

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#11697 closed defect (fixed)

ReportModule uses magic number -1 where None should be used — at Version 3

Reported by: Ryan J Ollos Owned by: Ryan J Ollos
Priority: normal Milestone: 1.1.2
Component: report system Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Refactored ReportModule to use None instead of -1 to indicate "no report".

API Changes:
Internal Changes:


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.

Change History (3)

comment:1 by Ryan J Ollos, 8 years ago

Proposed changes in log:rjollos.git:t11697.

comment:2 by Ryan J Ollos, 8 years ago


On second thought, the change should go on the trunk since it could affect plugins implementing IRequestFilter, ITemplateStreamFilter and possibly others.

comment:3 by Ryan J Ollos, 8 years ago

Release Notes: modified (diff)
Resolution: fixed
Status: newclosed

Fixed in [13048].

Note: See TracTickets for help on using tickets.