Milimetric has uploaded a new change for review.
https://gerrit.wikimedia.org/r/73518
Change subject: fixed the current_user usage in Job.__init__
......................................................................
fixed the current_user usage in Job.__init__
Change-Id: I8a7ffa6332b8b0d58033d9640dfd8715876960e1
---
M wikimetrics/models/job.py
1 file changed, 9 insertions(+), 4 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/analytics/wikimetrics
refs/changes/18/73518/1
diff --git a/wikimetrics/models/job.py b/wikimetrics/models/job.py
index 10674d9..ed93a3b 100644
--- a/wikimetrics/models/job.py
+++ b/wikimetrics/models/job.py
@@ -69,10 +69,15 @@
name=None,
result_key=None,
children=[]):
- if current_user.is_authenticated():
- self.user_id = current_user.id
- else:
- self.user_id = None
+
+ self.user_id = None
+ try:
+ if current_user.is_authenticated():
+ self.user_id = current_user.id
+ except RuntimeError:
+ # nothing to worry about, just using current_user outside
+ # of a web context. This should only happen during testing
+ pass
self.status = status
self.name = name
--
To view, visit https://gerrit.wikimedia.org/r/73518
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I8a7ffa6332b8b0d58033d9640dfd8715876960e1
Gerrit-PatchSet: 1
Gerrit-Project: analytics/wikimetrics
Gerrit-Branch: master
Gerrit-Owner: Milimetric <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
