Changeset a23363 in indico

Timestamp:

07/28/11 15:43:07 (22 months ago)

Author:

Jose Benito <jose.benito.gonzalez@…>

Branches:

master, burotel, hello-world-walkthrough, ipv6, v0.98-series, v0.98.2, v0.98.3, v0.98b1, v0.98b2, v0.99, 051b2622c51afb171a1dedb46a0df4fbb0cbd02e, d9941f8582b36b24821a11ea5ba16fda6a457fb1

Children:

8e555d

Parents:

0b8fed

Message:

[FIX] Add several reviewers at the same time

• Fixed bug trying to add several reviewers at the same time
• closes #801

Conflicts:

indico/MaKaC/webinterface/tpls/AbstractReviewingTeam.tpl

Location:

indico/MaKaC

Files:

• services/implementation/abstractReviewing.py (modified) (1 diff)
• webinterface/tpls/AbstractReviewingTeam.tpl (modified) (2 diffs)

indico/MaKaC/services/implementation/abstractReviewing.py

@@ -160 +160 @@
         pm = ParameterManager(self._params)
         self._trackId = pm.extract("track", pType=str, allowEmpty=False)
-        self._reviewerId = pm.extract("user", pType=str, allowEmpty=False)
-
-    def _getAnswer(self):
-        ah = user.AvatarHolder()
-        av = ah.getById(self._reviewerId)
-        self._conf.getTrackById(self._trackId).addCoordinator(av)
+        self._reviewerList = pm.extract("userList", pType=list, allowEmpty=False)
+
+    def _getAnswer(self):
+        for reviewer in self._reviewerList:
+            ah = user.AvatarHolder()
+            av = ah.getById(reviewer["id"])
+            self._conf.getTrackById(self._trackId).addCoordinator(av)
         return True
 

indico/MaKaC/webinterface/tpls/AbstractReviewingTeam.tpl

@@ -46 +46 @@
 
     // Create the handlers
-    var addReviewerHandler = function(user, setResult) {
+    var addReviewerHandler = function(userList, setResult) {
         indicoRequest(
                 'abstractReviewing.team.addReviewer',
@@ -52 +52 @@
                     conference: '<%= conf.getId() %>',
                     track: '<%= i %>',
-                    user: user[0]['id']
+                    userList: useList
                 },
                 function(result,error) {