Add ability for admin to claim a question

Author: stefano-maggiolo

cms/server/admin/handlers/__init__.py

@@ -2,7 +2,7 @@
 # -*- coding: utf-8 -*-
 
 # Contest Management System - http://cms-dev.github.io/
-# Copyright © 2015-2016 Stefano Maggiolo <s.maggiolo@gmail.com>
+# Copyright © 2015-2018 Stefano Maggiolo <s.maggiolo@gmail.com>
 # Copyright © 2016 Myungwoo Chun <mc.tamaki@gmail.com>
 # Copyright © 2016 Peyman Jabbarzade Ganje <peyman.jabarzade@gmail.com>
 # Copyright © 2016 Amir Keivan Mohtashami <akmohtashami97@gmail.com>
@@ -61,7 +61,8 @@
 from .contestquestion import \
     QuestionsHandler, \
     QuestionReplyHandler, \
-    QuestionIgnoreHandler
+    QuestionIgnoreHandler, \
+    QuestionClaimHandler
 from .contestranking import \
     RankingHandler
 from .task import \
@@ -160,6 +161,7 @@
     (r"/contest/([0-9]+)/questions", QuestionsHandler),
     (r"/contest/([0-9]+)/question/([0-9]+)/reply", QuestionReplyHandler),
     (r"/contest/([0-9]+)/question/([0-9]+)/ignore", QuestionIgnoreHandler),
+    (r"/contest/([0-9]+)/question/([0-9]+)/claim", QuestionClaimHandler),
 
     # Contest's ranking
 

cms/server/admin/handlers/contestquestion.py

@@ -137,6 +137,42 @@ def post(self, contest_id, question_id):
                         question.subject,
                         question.participation.user.username,
                         question.participation.contest.name,
-                        ["unignored", "ignored"][should_ignore])
+                        "ignored" if should_ignore else "unignored")
+
+        self.redirect(ref)
+
+
+class QuestionClaimHandler(BaseHandler):
+    """Called when the manager chooses to claim or unclaim a question."""
+
+    @require_permission(BaseHandler.PERMISSION_MESSAGING)
+    def post(self, contest_id, question_id):
+        ref = self.url("contest", contest_id, "questions")
+        question = self.safe_get_item(Question, question_id)
+        self.contest = self.safe_get_item(Contest, contest_id)
+
+        # Protect against URLs providing incompatible parameters.
+        if self.contest is not question.participation.contest:
+            raise tornado.web.HTTPError(404)
+
+        # Can claim/unclaim only a question not ignored or answered.
+        if question.ignored or question.reply_timestamp is not None:
+            raise tornado.web.HTTPError(405)
+
+        should_claim = self.get_argument("claim", "no") == "yes"
+
+        # Commit the change.
+        if should_claim:
+            question.admin = self.current_user
+        else:
+            question.admin = None
+        if self.try_commit():
+            logger.info("Question '%s' by user %s in contest %s has "
+                        "been %s by %s",
+                        question.subject,
+                        question.participation.user.username,
+                        question.participation.contest.name,
+                        "claimed" if should_claim else "unclaimed",
+                        self.current_user.name)
 
         self.redirect(ref)

cms/server/admin/static/aws_style.css

@@ -620,6 +620,11 @@ body.admin .notifications .answered {
     border: 0.143em solid #555555;
 }
 
+body.admin .notifications .claim_reply {
+    float: right;
+    padding-right: 0.5em;
+}
+
 body.admin .notifications .ignore_reply {
     float: right;
     padding-right: 0.5em;

cms/server/admin/templates/questions.html

@@ -61,13 +61,28 @@ <h1>Questions</h1>
       <hr>
       <div class="notification_subject">Not yet replied.</div>
       <hr>
-        {% if not msg.admin is none and msg.ignored %}
+        {% if msg.admin is not none %}
       <div class="notification_admin_owner">
-        Ignored by: {{ msg.admin is none and "<deleted>" or msg.admin.name }}</div>
+        {{ "Ignored" if msg.ignored else "Claimed" }} by: {{ msg.admin.name }}
+      </div>
         {% endif %}
       {% endif %}
 {% if admin.permission_all or admin.permission_messaging %}
       {% if msg.reply_timestamp is none %}
+        {% if not msg.ignored %}
+      <div class="claim_reply">
+        <form class="claim_question_form" action="{{ url("contest", contest.id, "question", msg.id, "claim") }}" name="claim{{ msg.id }}" method="POST">
+          {{ xsrf_form_html|safe }}
+          {% if msg.admin is none %}
+          <input type="hidden" name="claim" value="yes"/>
+          <a href="javascript:void(0);" onclick="document.claim{{ msg.id }}.submit();">Claim</a>
+          {% else %}
+          <input type="hidden" name="claim" value="no"/>
+          <a href="javascript:void(0);" onclick="document.claim{{ msg.id }}.submit();">Unclaim</a>
+          {% endif %}
+        </form>
+      </div>
+        {% endif %}
       <div class="ignore_reply">
         <form class="ignore_question_form" action="{{ url("contest", contest.id, "question", msg.id, "ignore") }}" name="ignore{{ msg.id }}" method="POST">
           {{ xsrf_form_html|safe }}