Given we aren't replacing parts of the url any more, we can simplify the following:
uri = URI(canonical_url(bug.default_bugtask))
# Empty the traversal stack, since we're redirecting.
self.request.setTraversalStack([])
# And perform a temporary redirect.
return RedirectionView(str(uri), self.request, status=303)
to just be
# Empty the traversal stack, since we're redirecting.
self.request.setTraversalStack([])
# And perform a temporary redirect.
return RedirectionView(canonical_url(bug.default_bugtask), self.request, status=303)
Given we aren't replacing parts of the url any more, we can simplify the following:
uri = URI(canonical_ url(bug. default_ bugtask) ) request. setTraversalSta ck([]) (str(uri) , self.request, status=303)
# Empty the traversal stack, since we're redirecting.
self.
# And perform a temporary redirect.
return RedirectionView
to just be
# Empty the traversal stack, since we're redirecting. request. setTraversalSta ck([]) (canonical_ url(bug. default_ bugtask) , self.request, status=303)
self.
# And perform a temporary redirect.
return RedirectionView
But on the whole, it looks fine.