Merlijn van Deen has uploaded a new change for review. (
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(-)
git pull ssh://gerrit.wikimedia.org:29418/labs/striker
refs/changes/74/404174/1
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: newchange
Gerrit-Change-Id: I61dcc40ec1fe83ebe77ad2eb593cc3d9cd322ce4
Gerrit-PatchSet: 1
Gerrit-Project: labs/striker
Gerrit-Branch: master
Gerrit-Owner: Merlijn van Deen <valhall...@arctus.nl>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
