launchpad-reviewers team mailing list archive

[Colin Watson <cjwatson@xxxxxxxxxxxxx>]

Colin Watson has proposed merging ~cjwatson/launchpad:testfix-object-lookup-redirection into launchpad:master.

Commit message:
Fix DeprecatedAssignedBugsView

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~cjwatson/launchpad/+git/launchpad/+merge/387663

RedirectionView recently gained a context property to support webservice object lookup, but this broke DeprecatedAssignedBugsView, which inherits from RedirectionView and tried to assign to self.context in its constructor.  Explicitly override the property instead.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~cjwatson/launchpad:testfix-object-lookup-redirection into launchpad:master.

diff --git a/lib/lp/bugs/browser/bug.py b/lib/lp/bugs/browser/bug.py
index d5f68ca..48749dc 100644
--- a/lib/lp/bugs/browser/bug.py
+++ b/lib/lp/bugs/browser/bug.py
@@ -989,7 +989,7 @@ class DeprecatedAssignedBugsView(RedirectionView):
     """
 
     def __init__(self, context, request):
-        self.context = context
+        self._context = context
         self.request = request
         self.status = 303
 
@@ -998,6 +998,10 @@ class DeprecatedAssignedBugsView(RedirectionView):
             getUtility(ILaunchBag).user, view_name='+assignedbugs')
         super(DeprecatedAssignedBugsView, self).__call__()
 
+    @property
+    def context(self):
+        return self._context
+
 
 normalize_mime_type = re.compile(r'\s+')