jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/404174 )
Change subject: order login_required and require_tools_member correctly
......................................................................
order login_required and require_tools_member correctly
As require_tools_member requires request.user to be available,
login_required needs to be processed first, and therefore also
needs to be the first decorator.
Bug: T177407
Change-Id: I61dcc40ec1fe83ebe77ad2eb593cc3d9cd322ce4
---
M striker/tools/views/tool.py
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
BryanDavis: Looks good to me, approved
jenkins-bot: Verified
Zoranzoki21: Looks good to me, but someone else must approve
diff --git a/striker/tools/views/tool.py b/striker/tools/views/tool.py
index bf2e309..c4e31af 100644
--- a/striker/tools/views/tool.py
+++ b/striker/tools/views/tool.py
@@ -62,8 +62,8 @@
})
-@require_tools_member
@login_required
+@require_tools_member
def create(req):
form = ToolCreateForm(req.POST or None, req.FILES or None)
if req.method == 'POST':
--
To view, visit https://gerrit.wikimedia.org/r/404174
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I61dcc40ec1fe83ebe77ad2eb593cc3d9cd322ce4
Gerrit-PatchSet: 1
Gerrit-Project: labs/striker
Gerrit-Branch: master
Gerrit-Owner: Merlijn van Deen <[email protected]>
Gerrit-Reviewer: BryanDavis <[email protected]>
Gerrit-Reviewer: Zoranzoki21 <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
