← Back to team overview

launchpad-reviewers team mailing list archive

[Merge] ~ilasc/launchpad:flake8-f841 into launchpad:master

 

Ioana Lasc has proposed merging ~ilasc/launchpad:flake8-f841 into launchpad:master.

Commit message:
flake8: fix F841

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~ilasc/launchpad/+git/launchpad/+merge/406644
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~ilasc/launchpad:flake8-f841 into launchpad:master.
diff --git a/lib/lp/services/webapp/publication.py b/lib/lp/services/webapp/publication.py
index c012a1c..955685a 100644
--- a/lib/lp/services/webapp/publication.py
+++ b/lib/lp/services/webapp/publication.py
@@ -657,7 +657,7 @@ class LaunchpadBrowserPublication(
                 labels={
                     'success': False,
                     'pageid': self._prepPageIDForMetrics(
-                        request._orig_env.get('launchpad.pageid')),
+                        orig_env.get('launchpad.pageid')),
                     })
         elif (hasattr(request, '_traversal_start') and
               'traversal_duration_ms' not in logging_context.flat):
@@ -677,7 +677,7 @@ class LaunchpadBrowserPublication(
                 labels={
                     'success': False,
                     'pageid': self._prepPageIDForMetrics(
-                        request._orig_env.get('launchpad.pageid')),
+                        orig_env.get('launchpad.pageid')),
                     })
         else:
             # The exception wasn't raised in the middle of the traversal nor