Launchpad itself

Navigation.redirectSubTree(None) should OOPS

Bug #645751 reported by Michael Hudson-Doyle
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

Bug 645544 caused particularly weird symptoms because redirectSubTree(None) somewhere along the line stringified it's argument and ended up redirecting the user to +branch/None and then into a redirect loop. If redirectSubTree(None) had raised, say, AssertionError, the mistake would have been much more obvious.

Revision history for this message
Gary Poster (gary) wrote :

c.l.w.publisher.Navigation.redirectSubTree. +1 on ValueError. Very easy change.

Changed in launchpad-foundations:
status: New → Triaged
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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