Fix and/or test use of `is None` in Bug.isSubscribed and Bug.isSubscribedToDupes

Bug #389486 reported by Deryck Hodge
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

Both Bug.isSubscribed and Bug.isSubscribedToDupes use an `if person is None: return False` pattern to allow use of these methods without a person, presumably to work without a logged in user. This came up in a review of mine for another branch. Either these methods need to remove the check against None and ensure that the calling site never passes in None, or these methods need to be tested for the None case or when dealing with anonymous users. Or perhaps even some combination of the two.

Deryck Hodge (deryck)
tags: added: story-fix-test-suite
Revision history for this message
Diogo Matsubara (matsubara) wrote :

Hi Deryck,

setting importance to low for now. Please increase importance as you see fit.
Thanks

tags: added: cleanup
Changed in malone:
importance: Undecided → Low
status: New → Triaged
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.