Edgewall Software

Opened 5 years ago

Last modified 5 years ago

#11697 closed defect

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

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:
API Changes:

Description

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 (0)

Note: See TracTickets for help on using tickets.