Changeset ac1609 in indico


Ignore:
Timestamp:
10/04/10 17:17:54 (3 years 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:
5b4690
Parents:
334896
git-author:
csvetelina.angelova@…> (02/01/10 17:27:22)
git-committer:
Jose Benito <jose.benito.gonzalez@…> (10/04/10 17:17:54)
Message:

[FIX] - different fixes in the PRM

  • there is the option to enable/disable the automatic e-mails when is added/removed a PRM.
  • Competences - depending on the reviewing mode not all of the reviewers are shown here. For example, if we have only the Layout Reviewing, will be shown only the Layout Reviewers and the PRM.
  • Team - there are now warning messages that the added/removed reviewers will receive an e-mails if the option is enabled from the Setup.
  • Assign Contributions - when we want to remove a Referee or Layout Reviewer from a contributions that don't have ones - there is an alert.
  • Judgements/Final? Judgements - depending on the reviewing mode are shown/not shown the Referee/Layout/Conten? Reviewers Judgements
  • layout fix - For Referee/Content/Layout? Reviewers - the field "Judgements"goes last, because this is the last this that they should do.
  • Display Manager - depending on the reviewing mode in the Display/My? Conference there are different link shown.
Location:
indico/MaKaC
Files:
12 edited

Legend:

Unmodified
Added
Removed

  • indico/MaKaC/reviewing.py

    r423698 rac1609  
    7878        self._defaultAbstractReviwerDueDate = None 
    7979         
    80         #auto e-mails         
     80        #auto e-mails  
     81        self._enablePRMEmailNotif = True        
    8182        self._enableRefereeEmailNotif = False 
    8283        self._enableEditorEmailNotif = False 
    … …  
    176177     
    177178    #auto e-mails methods for assign/remove reviewing team to the conference notification 
     179    def getEnablePRMEmailNotif(self): 
     180        try : 
     181            if self._enablePRMEmailNotif  : 
     182                pass 
     183        except AttributeError : 
     184            self._enablePRMEmailNotif = True 
     185        return self._enablePRMEmailNotif 
     186 
     187    def enablePRMEmailNotif(self): 
     188        self._enablePRMEmailNotif = True 
     189 
     190    def disablePRMEmailNotif(self): 
     191        self._enablePRMEmailNotif = False 
     192     
    178193    def getEnableRefereeEmailNotif(self): 
    179194        try : 
    … …  
    815830                self._userCompetences[newPaperReviewManager] = [] 
    816831            self.notifyModification() 
     832        if self._enablePRMEmailNotif == True: 
    817833            notification = ConferenceReviewingNotification(newPaperReviewManager, 'Paper Review Manager', self._conference) 
    818834            GenericMailer.sendAndLog(notification, self._conference, "MaKaC/reviewing.py", newPaperReviewManager) 
    … …  
    835851            paperReviewManager.unlinkTo(self._conference, "paperReviewManager") 
    836852            self.notifyModification() 
    837             notification = ConferenceReviewingRemoveNotification(paperReviewManager, 'Paper Review Manager', self._conference) 
    838             GenericMailer.sendAndLog(notification, self._conference, "MaKaC/reviewing.py", paperReviewManager) 
     853            if self._enablePRMEmailNotif == True: 
     854                notification = ConferenceReviewingRemoveNotification(paperReviewManager, 'Paper Review Manager', self._conference) 
     855                GenericMailer.sendAndLog(notification, self._conference, "MaKaC/reviewing.py", paperReviewManager) 
    839856        else: 
    840857            raise MaKaCError("Cannot remove a paper review manager who is not yet paper review manager") 
    … …  
    11411158        """ 
    11421159        roles=[] 
    1143         if self.isPaperReviewManager(user): 
     1160        if self.isPaperReviewManager(user) and not self._choice == 1: 
    11441161            roles.append('Manager of Paper Reviewing Module') 
    1145         if self.isReferee(user): 
     1162        if self.isReferee(user) and (self._choice == 2 or self._choice == 4): 
    11461163            roles.append('Referee') 
    1147         if self.isEditor(user): 
     1164        if self.isEditor(user) and (self._choice == 3 or self._choice == 4): 
    11481165            roles.append('Layout Reviewer') 
    1149         if self.isReviewer(user): 
     1166        if self.isReviewer(user) and (self._choice == 2 or self._choice == 4): 
    11501167            roles.append('Content Reviewer') 
    11511168        if self.isAbstractManager(user): 

  • indico/MaKaC/services/implementation/reviewing.py

    r0c1608 rac1609  
    242242        if date: 
    243243            return datetime.datetime.strftime(date,'%d/%m/%Y %H:%M') 
    244          
     244 
     245class ConferenceReviewingAutoEmailsModificationPRM(ConferenceReviewingSetupTextModificationBase ): 
     246     
     247    def _handleSet(self): 
     248        if self._value: 
     249            self._confReview.enablePRMEmailNotif() 
     250        else: 
     251            self._confReview.disablePRMEmailNotif() 
     252         
     253    def _handleGet(self): 
     254        return self._confReview.getEnablePRMEmailNotif() 
     255        
    245256class ConferenceReviewingAutoEmailsModificationReferee(ConferenceReviewingSetupTextModificationBase ): 
    246257     
    … …  
    848859    "conference.removeAllReviewers" : ConferenceReviewingRemoveAllReviewers, 
    849860     
     861    "conference.PRMEmailNotif" : ConferenceReviewingAutoEmailsModificationPRM, 
    850862    "conference.RefereeEmailNotif" : ConferenceReviewingAutoEmailsModificationReferee, 
    851863    "conference.EditorEmailNotif" : ConferenceReviewingAutoEmailsModificationEditor, 

  • indico/MaKaC/webinterface/pages/conferences.py

    r334896 rac1609  
    278278 
    279279                
    280                 if showrefereearea: 
     280                if showrefereearea and (self._conf.getConfReview().getChoice()==2 or self._conf.getConfReview().getChoice()==4): 
    281281                    self._assignContribOpt.setVisible(True) 
    282282                    self._judgeListOpt.setVisible(True) 
    283283             
    284                 if showreviewerarea: 
     284                if showreviewerarea and (self._conf.getConfReview().getChoice()==2 or self._conf.getConfReview().getChoice()==4): 
    285285                    self._judgereviewerListOpt.setVisible(True) 
    286286             
    287                 if showeditorarea: 
     287                if showeditorarea and (self._conf.getConfReview().getChoice()==3 or self._conf.getConfReview().getChoice()==4): 
    288288                    self._judgeeditorListOpt.setVisible(True) 
    289289 

  • indico/MaKaC/webinterface/pages/contributionReviewing.py

    r334896 rac1609  
    133133 
    134134        conferenceChoice = self._conf.getConfReview().getChoice() 
     135        conferenceChoiceStr = self._conf.getConfReview().getReviewingMode() 
    135136        reviewManager = self.__target.getReviewManager()         
    136137        vars["Conference"] = self._conf 
    … …  
    138139        vars["Contribution"] = self.__target 
    139140        vars["ConferenceChoice"] = conferenceChoice 
     141        vars["ConferenceChoiceStr"] = conferenceChoiceStr 
    140142        vars["FinalJudge"] = reviewManager.getLastReview().getRefereeJudgement().getJudgement() 
    141143        vars["Editing"] = reviewManager.getLastReview().getEditorJudgement() 

  • indico/MaKaC/webinterface/tpls/ConfModifUserCompetences.tpl

    rc818e6 rac1609  
    22 
    33<% if not ConfReview.hasReviewing(): %> 
    4 <table align="center"><tr><td><%= _("Type of reviewing has not been chosen yet")%></td></tr></table> 
     4<p style="padding-left: 25px;"><font color="gray"><%= _("Type of reviewing has not been chosen yet")%></font></p> 
    55<% end %> 
    66<% elif ConfReview.getAllUserCompetences() == []: %> 
    … …  
    2727     
    2828    <% for user, competences in ConfReview.getAllUserCompetences(): %> 
     29    <% if ConfReview.getUserReviewingRoles(user): %> 
    2930    <% if ConfReview.isPaperReviewManager(user) or ConfReview.isReferee(user) or ConfReview.isEditor(user) or ConfReview.isReviewer(user):%> 
    3031        <tr valign="top"> 
    … …  
    3839            </td> 
    3940        </tr> 
     41    <% end %> 
    4042    <% end %> 
    4143    <% end %> 

  • indico/MaKaC/webinterface/tpls/ConfModificationReviewingFrame.tpl

    r423698 rac1609  
    44<% if ConfReview.hasReviewing(): %> 
    55<table width="85%%" align="center" border="0"> 
    6     <tr> 
    76        <td id="revControlRefereeEditorReviewerHelp"  colspan="3" class="groupTitle"  style="padding-top: 15px; padding-bottom: 15px;"> 
    87            <%= _("Step 2: Assign Reviewers")%> 
    98        </td> 
    10     </tr>    
     9    </tr> 
     10    <tr> 
     11        <td> 
     12            <% if ConfReview.getEnableRefereeEmailNotif() or ConfReview.getEnableEditorEmailNotif() or ConfReview.getEnableReviewerEmailNotif(): %> 
     13                <div style="padding-top: 10px; padding-bottom: 15px;"> 
     14                    <em><%=_("An automatically generated e-mail will be send to the Reviewers you will assign")%></em><br> 
     15                    <em><%= _("You  can  modify this from the Paper Reviewing Setup")%></em> 
     16                </div> 
     17            <% end %> 
     18        </td> 
     19    </tr> 
     20    <tr> 
    1121</table>  
     22 
    1223<table width="83%%" align="right" border="0"> 
    13      
    1424<% if ConfReview.getChoice() == 3 or ConfReview.getChoice() == 1:%> 
    1525<% pass %> 
    1626<% end %> 
    17 <% else: %>  
     27<% else: %> 
    1828    <tr> 
    1929        %(refereeTable)s 

  • indico/MaKaC/webinterface/tpls/ConfModificationReviewingFramePRM.tpl

    ra0cc55 rac1609  
    33 
    44<% if not ConfReview.hasReviewing(): %> 
    5 <table align="center"><tr><td><%= _("Type of reviewing has not been chosen yet")%></td></tr></table> 
     5<p style="padding-left: 25px;"><font color="gray"><%= _("Type of reviewing has not been chosen yet")%></font></p> 
    66<% end %> 
    77<%else:%> 
    88<br> 
    99<table width="85%%" align="center" border="0"> 
     10    <tr>  
     11        <td id="revControlPRMHelp"  colspan="3" class="groupTitle" style="padding-top: 15px;"><%= _("Step 1: Assign Managers of Paper Reviewing Module")%></td> 
     12    </tr> 
    1013    <tr> 
    11         <em><%= _("Please, follow the steps to assign Paper Reviewing Team")%></em>  
    12         <td id="revControlPRMHelp"  colspan="3" class="groupTitle" style="padding-top: 15px;"><%= _("Step 1: Assign Managers of Paper Reviewing Module")%></td> 
     14        <td colspan="3"> 
     15            <% if ConfReview.getEnablePRMEmailNotif(): %> 
     16                <div style="padding-top: 10px; padding-bottom: 15px;"> 
     17                    <em><%=_("An automatically generated e-mail will be send to the Paper Review Managers you will assign")%></em><br> 
     18                    <em><%= _("You  can  modify this from the Paper Reviewing Setup")%></em> 
     19                </div> 
     20            <% end %> 
     21        </td> 
    1322    </tr> 
    1423    <tr> 

  • indico/MaKaC/webinterface/tpls/ConfModificationReviewingSettings.tpl

    r321374 rac1609  
    9191                <%= stateTable %> 
    9292            </td> 
    93              
    9493        </form> 
    9594    </tr> 
    … …  
    261260        <% end %> 
    262261            <tr id="autoEmails" style="display:<%=display%>"> 
    263                 <td id="automaticNotificationHelp" colspan="5" class="reviewingsubtitle"><%= _("Automatic e-mails")%> 
     262                <td id="automaticNotificationHelp" colspan="5" class="reviewingsubtitle"><%= _("Automatic e-mails can be send to")%>: 
    264263                   <% inlineContextHelp(_('Here you can enable/disable automatic e-mails sending.<br/>Notifications can be send to the Reviewing Team in the next several situations<br/><ul><li>when are added/removed Reviewers for the conference</li><li>when are assinged/removed contributions to Reviewers</li><li>when authors of the contributions have been submitted materials</li></ul>Notifications can be send to the authors when their contributions had been judged by the Reviewers.'))%> 
    265264                </td> 
    266265            </tr> 
     266            <tr id="autoEmailsPRMLabel" style="display:<%=display%>"> 
     267           <td style="padding-top: 10px;"> 
     268               <%= _("Paper Review Managers when")%>: 
     269           </td> 
     270        </tr> 
     271       <tr id="PRMNotif" style="white-space:nowrap; display: <%=display%>"> 
     272        <td> 
     273            <div> 
     274                <span id="PRMNotifButton">                     
     275                </span> 
     276            </div> 
     277        </td> 
     278       </tr> 
    267279        <% if ConfReview.hasPaperReviewing(): %> 
    268280            <% display = 'table-row' %> 
    … …  
    271283            <% display = 'none' %> 
    272284        <% end %> 
     285        <tr id="autoEmailsRefereeLabel" style="display:<%=display%>"> 
     286           <td style="padding-top: 10px;"> 
     287               <%= _("Referees when")%>: 
     288           </td> 
     289        </tr> 
    273290           <tr id="refereeNotif" style="white-space:nowrap; display: <%=display%>"> 
    274291            <td> 
    … …  
    279296        </td> 
    280297       </tr> 
     298       <tr id="refereeNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
     299        <td> 
     300            <div> 
     301                <span id="refereeNotifForContributionButton">                     
     302                </span> 
     303            </div> 
     304        </td> 
     305       </tr> 
     306       <tr id="authorSubmittedMatRefereeNotif" style="white-space:nowrap; display: <%=display%>"> 
     307        <td> 
     308            <div> 
     309                <span id="authorSubmittedMatRefereeNotifButton">                     
     310                </span> 
     311            </div> 
     312        </td> 
     313       </tr> 
     314        <tr id="autoEmailsContentLabel" style="display:<%=display%>"> 
     315           <td style="padding-top: 10px;"> 
     316               <%= _("Content Reviewers when")%>: 
     317           </td> 
     318        </tr> 
     319       <tr id="reviewerNotif" style="white-space:nowrap; display: <%=display%>"> 
     320        <td> 
     321            <div> 
     322                <span id="reviewerNotifButton">                    
     323                </span> 
     324            </div> 
     325        </td> 
     326       </tr> 
     327       <tr id="reviewerNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
     328        <td> 
     329            <div> 
     330                <span id="reviewerNotifForContributionButton">                    
     331                </span> 
     332            </div> 
     333        </td> 
     334       </tr> 
     335       <tr id="authorSubmittedMatReviewerNotif" style="white-space:nowrap; display: <%=display%>"> 
     336        <td> 
     337            <div> 
     338                <span id="authorSubmittedMatReviewerNotifButton">                    
     339                </span> 
     340            </div> 
     341        </td> 
     342       </tr> 
    281343        <% if ConfReview.hasPaperEditing(): %> 
    282344            <% display = 'table-row' %> 
    … …  
    285347            <% display = 'none' %> 
    286348        <% end %> 
     349        <tr id="autoEmailsEditorLabel" style="display:<%=display%>"> 
     350           <td style="padding-top: 10px;"> 
     351               <%= _("Layout Reviewers when")%>: 
     352           </td> 
     353        </tr> 
    287354       <tr id="editorNotif" style="white-space:nowrap; display: <%=display%>"> 
    288355          <td> 
    … …  
    293360         </td> 
    294361       </tr> 
    295         <% if ConfReview.hasPaperReviewing(): %> 
     362       <tr id="editorNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
     363          <td> 
     364            <div> 
     365                <span id="editorNotifForContributionButton">                     
     366                </span> 
     367            </div> 
     368         </td> 
     369       </tr> 
     370       <tr id="authorSubmittedMatEditorNotif" style="white-space:nowrap; display: <%=display%>"> 
     371          <td> 
     372            <div> 
     373                <span id="authorSubmittedMatEditorNotifButton">                     
     374                </span> 
     375            </div> 
     376         </td> 
     377       </tr> 
     378       <% if ConfReview.hasReviewing(): %> 
    296379            <% display = 'table-row' %> 
    297380        <% end %> 
    … …  
    299382            <% display = 'none' %> 
    300383        <% end %> 
    301        <tr id="reviewerNotif" style="white-space:nowrap; display: <%=display%>"> 
    302         <td> 
    303             <div> 
    304                 <span id="reviewerNotifButton">                    
    305                 </span> 
    306             </div> 
    307         </td> 
    308        </tr> 
    309        <% if ConfReview.hasPaperReviewing(): %> 
     384        <tr id="autoEmailsAuthor" style="display:<%=display%>"> 
     385           <td style="padding-top: 10px;"> 
     386               <%= _("Author of the paper when a judgement is submitted by")%>: 
     387           </td> 
     388        </tr> 
     389         <% if ConfReview.hasPaperReviewing(): %> 
    310390            <% display = 'table-row' %> 
    311391        <% end %> 
    … …  
    313393            <% display = 'none' %> 
    314394        <% end %> 
    315        <tr id="refereeNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
    316         <td> 
    317             <div> 
    318                 <span id="refereeNotifForContributionButton">                     
     395       <tr id="refereeJudgementNotif" style="white-space:nowrap; display: <%=display%>"> 
     396        <td> 
     397            <div> 
     398                <span id="refereeJudgementNotifButton">                     
     399                </span> 
     400            </div> 
     401        </td> 
     402       </tr> 
     403       <tr id="reviewerJudgementNotif" style="white-space:nowrap; display: <%=display%>"> 
     404        <td> 
     405            <div> 
     406                <span id="reviewerJudgementNotifButton">                    
    319407                </span> 
    320408            </div> 
    … …  
    327415            <% display = 'none' %> 
    328416        <% end %> 
    329        <tr id="editorNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
    330           <td> 
    331             <div> 
    332                 <span id="editorNotifForContributionButton">                     
    333                 </span> 
    334             </div> 
    335          </td> 
    336        </tr> 
    337         <% if ConfReview.hasPaperReviewing(): %> 
    338             <% display = 'table-row' %> 
    339         <% end %> 
    340         <% else: %> 
    341             <% display = 'none' %> 
    342         <% end %> 
    343        <tr id="reviewerNotifForContribution" style="white-space:nowrap; display: <%=display%>"> 
    344         <td> 
    345             <div> 
    346                 <span id="reviewerNotifForContributionButton">                    
    347                 </span> 
    348             </div> 
    349         </td> 
    350        </tr> 
    351        <% if ConfReview.hasPaperReviewing(): %> 
    352             <% display = 'table-row' %> 
    353         <% end %> 
    354         <% else: %> 
    355             <% display = 'none' %> 
    356         <% end %> 
    357        <tr id="authorSubmittedMatRefereeNotif" style="white-space:nowrap; display: <%=display%>"> 
    358         <td> 
    359             <div> 
    360                 <span id="authorSubmittedMatRefereeNotifButton">                     
    361                 </span> 
    362             </div> 
    363         </td> 
    364        </tr> 
    365         <% if ConfReview.hasPaperEditing(): %> 
    366             <% display = 'table-row' %> 
    367         <% end %> 
    368         <% else: %> 
    369             <% display = 'none' %> 
    370         <% end %> 
    371        <tr id="authorSubmittedMatEditorNotif" style="white-space:nowrap; display: <%=display%>"> 
    372           <td> 
    373             <div> 
    374                 <span id="authorSubmittedMatEditorNotifButton">                     
    375                 </span> 
    376             </div> 
    377          </td> 
    378        </tr> 
    379         <% if ConfReview.hasPaperReviewing(): %> 
    380             <% display = 'table-row' %> 
    381         <% end %> 
    382         <% else: %> 
    383             <% display = 'none' %> 
    384         <% end %> 
    385        <tr id="authorSubmittedMatReviewerNotif" style="white-space:nowrap; display: <%=display%>"> 
    386         <td> 
    387             <div> 
    388                 <span id="authorSubmittedMatReviewerNotifButton">                    
    389                 </span> 
    390             </div> 
    391         </td> 
    392        </tr> 
    393        <% if ConfReview.hasPaperReviewing(): %> 
    394             <% display = 'table-row' %> 
    395         <% end %> 
    396         <% else: %> 
    397             <% display = 'none' %> 
    398         <% end %> 
    399        <tr id="refereeJudgementNotif" style="white-space:nowrap; display: <%=display%>"> 
    400         <td> 
    401             <div> 
    402                 <span id="refereeJudgementNotifButton">                     
    403                 </span> 
    404             </div> 
    405         </td> 
    406        </tr> 
    407         <% if ConfReview.hasPaperEditing(): %> 
    408             <% display = 'table-row' %> 
    409         <% end %> 
    410         <% else: %> 
    411             <% display = 'none' %> 
    412         <% end %> 
    413417       <tr id="editorJudgementNotif" style="white-space:nowrap; display: <%=display%>"> 
    414418          <td> 
    … …  
    419423         </td> 
    420424       </tr> 
    421         <% if ConfReview.hasPaperReviewing(): %> 
    422             <% display = 'table-row' %> 
    423         <% end %> 
    424         <% else: %> 
    425             <% display = 'none' %> 
    426         <% end %> 
    427        <tr id="reviewerJudgementNotif" style="white-space:nowrap; display: <%=display%>"> 
    428         <td> 
    429             <div> 
    430                 <span id="reviewerJudgementNotifButton">                    
    431                 </span> 
    432             </div> 
    433         </td> 
    434        </tr> 
    435425</table> 
    436426 
    … …  
    442432<% end %> 
    443433<form action="<%= setTemplateURL %>" method="post" ENCTYPE="multipart/form-data"> 
    444 <table id="templateTable" width="90%%" align="center" border="0" style="display:<%=display%>"> 
     434<table id="templateTable" width="90%%" align="center" border="0" style="padding-top: 10px; display:<%=display%>"> 
    445435    <tr> 
    446436        <td id="uploadTemplateHelp" colspan="5" class="reviewingsubtitle"><%= _("Upload a template")%></td> 
    … …  
    578568        $E('reviewerDefaultDateRow').dom.style.display = 'none'; 
    579569        $E('autoEmails').dom.style.display = 'none'; 
     570        $E('autoEmailsPRMLabel').dom.style.display = 'none'; 
     571        $E('autoEmailsRefereeLabel').dom.style.display = 'none'; 
     572        $E('autoEmailsContentLabel').dom.style.display = 'none'; 
     573        $E('autoEmailsEditorLabel').dom.style.display = 'none'; 
     574        $E('autoEmailsAuthor').dom.style.display = 'none'; 
     575        $E('PRMNotif').dom.style.display = 'none'; 
    580576        $E('refereeNotif').dom.style.display = 'none'; 
    581577        $E('editorNotif').dom.style.display = 'none'; 
    … …  
    604600        $E('editorDefaultDateRow').dom.style.display = 'none'; 
    605601        $E('reviewerDefaultDateRow').dom.style.display = ''; 
    606         $E('autoEmails').dom.style.display = '' 
     602        $E('autoEmails').dom.style.display = ''; 
     603        $E('autoEmailsPRMLabel').dom.style.display = ''; 
     604        $E('PRMNotif').dom.style.display = ''; 
     605        $E('autoEmailsRefereeLabel').dom.style.display = ''; 
     606        $E('autoEmailsContentLabel').dom.style.display = ''; 
     607        $E('autoEmailsEditorLabel').dom.style.display = 'none'; 
     608        $E('autoEmailsAuthor').dom.style.display = ''; 
    607609        $E('refereeNotif').dom.style.display = ''; 
    608610        $E('editorNotif').dom.style.display = 'none'; 
    … …  
    639641        $E('reviewerDefaultDateRow').dom.style.display = 'none'; 
    640642        $E('autoEmails').dom.style.display = ''; 
     643        $E('autoEmailsPRMLabel').dom.style.display = ''; 
     644        $E('PRMNotif').dom.style.display = ''; 
     645        $E('autoEmailsRefereeLabel').dom.style.display = 'none'; 
     646        $E('autoEmailsContentLabel').dom.style.display = 'none'; 
     647        $E('autoEmailsEditorLabel').dom.style.display = ''; 
     648        $E('autoEmailsAuthor').dom.style.display = ''; 
    641649        $E('refereeNotif').dom.style.display = 'none'; 
    642650        $E('editorNotif').dom.style.display = ''; 
    … …  
    670678        $E('editorDefaultDateRow').dom.style.display = ''; 
    671679        $E('reviewerDefaultDateRow').dom.style.display = ''; 
     680        $E('autoEmailsRefereeLabel').dom.style.display = ''; 
     681        $E('autoEmailsContentLabel').dom.style.display = ''; 
     682        $E('autoEmailsEditorLabel').dom.style.display = ''; 
     683        $E('autoEmailsAuthor').dom.style.display = ''; 
    672684        $E('autoEmails').dom.style.display = ''; 
     685        $E('autoEmailsPRMLabel').dom.style.display = ''; 
     686        $E('PRMNotif').dom.style.display = ''; 
    673687        $E('refereeNotif').dom.style.display = ''; 
    674688        $E('editorNotif').dom.style.display = ''; 
    … …  
    837851   
    838852 
     853$E('PRMNotifButton').set(IndicoUI.Widgets.Generic.switchOptionButton('reviewing.conference.PRMEmailNotif',  
     854                                            {conference: '<%= ConfReview.getConference().getId() %>', 
     855                                            AutoEmailsToChange: 'PRM'},  
     856                                            'are added/removed from the conference', 
     857                                            true 
     858)); 
     859 
    839860$E('refereeNotifButton').set(IndicoUI.Widgets.Generic.switchOptionButton('reviewing.conference.RefereeEmailNotif',  
    840861                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    841862                                            AutoEmailsToChange: 'Referee'},  
    842                                             'When a Referee is added or removed from the team', 
     863                                            'are added/removed from the conference', 
    843864                                            true 
    844865));  
    … …  
    846867                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    847868                                            AutoEmailsToChange: 'Reviewer'},  
    848                                             'When a Content Reviewer is added or removed from the team', 
     869                                            'are added/removed from the conference', 
    849870                                            true 
    850871)); 
    … …  
    852873                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    853874                                            AutoEmailsToChange: 'Editor'},  
    854                                             'When a Layout Reviewer is added or removed from the team', 
     875                                            'are added/removed from the conference', 
    855876                                            true 
    856877)); 
    … …  
    858879                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    859880                                            AutoEmailsToChange: 'Referee'},  
    860                                             'When a contribution is assigned or unassigned to a Referee', 
     881                                            'have assigned/unassigned contributions', 
    861882                                            true 
    862883)); 
    … …  
    864885                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    865886                                            AutoEmailsToChange: 'Reviewer'},  
    866                                             'When a contribution is assigned or unassigned to a Content Reviewer', 
     887                                            'have assigned/unassigned contributions', 
    867888                                            true 
    868889)); 
    … …  
    870891                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    871892                                            AutoEmailsToChange: 'Editor'},  
    872                                             'When a contribution is assigned or unassigned to a Layout Reviewer', 
     893                                            'have assigned/unassigned contributions', 
    873894                                            true 
    874895)); 
    … …  
    876897                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    877898                                            AutoEmailsToChange: 'Referee'},  
    878                                             'To the author when the Referee submits judgement', 
     899                                            'Referee', 
    879900                                            true 
    880901)); 
    … …  
    882903                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    883904                                            AutoEmailsToChange: 'Reviewer'},  
    884                                             'To the author when a Content Reviewer submits judgement', 
     905                                            'Content Reviewer', 
    885906                                            true 
    886907)); 
    … …  
    888909                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    889910                                            AutoEmailsToChange: 'Editor'},  
    890                                             'To the author when the Layout Reviewer submits judgement', 
     911                                            'Layout Reviewer', 
    891912                                            true 
    892913)); 
    … …  
    894915                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    895916                                            AutoEmailsToChange: 'Referee'},  
    896                                             'To the Referee when an author submits paper', 
     917                                            'the author submits a paper', 
    897918                                            true 
    898919)); 
    … …  
    900921                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    901922                                            AutoEmailsToChange: 'Reviewer'},  
    902                                             'To the Content Reviewer when an author submits paper', 
     923                                            'the author submits a paper', 
    903924                                            true 
    904925)); 
    … …  
    906927                                            {conference: '<%= ConfReview.getConference().getId() %>', 
    907928                                            AutoEmailsToChange: 'Editor'},  
    908                                             'To the Layout Reviewer when an author submits paper', 
     929                                            'the author submits a paper', 
    909930                                            true 
    910931)); 

  • indico/MaKaC/webinterface/tpls/ConfReviewingAssignContributions.tpl

    r0c1608 rac1609  
    732732    } 
    733733     
     734    contributionsWithoutEditor = [] 
     735    for (i in contributions) { 
     736        contributionId = contributions[i] 
     737        contribution = getContribution(contributionId) 
     738        if (contribution.reviewManager.editor = null) { 
     739            contributionsWithoutEditor.push(contributionId) 
     740        } 
     741    } 
     742     
     743    if (contributionsWithoutEditor.length == contributions.length) { 
     744        alert($T("There is no assigned Layout Reviewer to remove.") 
     745        ); 
     746        return false; 
     747    }  
     748     
    734749    if (contributionsWithoutReviewers.length > 0) { 
    735750        title =$T('Contributions without reviewer');     
    … …  
    757772     
    758773    return true;     
     774} 
     775 
     776var removeEditorAlerts = function(contributions, role) { 
     777     
     778    contributionsWithoutEditor = [] 
     779    for (i in contributions) { 
     780        contributionId = contributions[i] 
     781        contribution = getContribution(contributionId) 
     782        if (contribution.reviewManager.editor == null) { 
     783            contributionsWithoutEditor.push(contributionId) 
     784        } 
     785    } 
     786     
     787    if (contributionsWithoutEditor.length == contributions.length) { 
     788        alert($T("There is no assigned Layout Reviewer to remove.") 
     789        ); 
     790        return false; 
     791    }  
     792     
     793    return true; 
     794     
     795} 
     796 
     797var removeNoRefereeAlerts = function(contributions, role) { 
     798     
     799    contributionsWithoutEditor = [] 
     800    for (i in contributions) { 
     801        contributionId = contributions[i] 
     802        contribution = getContribution(contributionId) 
     803        if (contribution.reviewManager.referee == null) { 
     804            contributionsWithoutEditor.push(contributionId) 
     805        } 
     806    } 
     807     
     808    if (contributionsWithoutEditor.length == contributions.length) { 
     809        alert($T("There is no assigned Referee to remove.") 
     810        ); 
     811        return false; 
     812    }  
     813     
     814    return true; 
     815     
    759816} 
    760817 
    … …  
    10071064        }  
    10081065   } 
    1009      
     1066    
     1067    
    10101068    indicoRequest( 
    10111069        'reviewing.conference.userCompetencesList', 
    … …  
    11121170            function(result,error) { 
    11131171                if (!error) { 
     1172                    if (!removeNoRefereeAlerts(checkedContributions, role)) { 
     1173                                return; 
     1174                        } 
    11141175                    for (i in checkedContributions) { 
    11151176                        contributionId = checkedContributions[i]; 
    … …  
    11361197            function(result,error) { 
    11371198                if (!error) { 
     1199                    if (!removeEditorAlerts(checkedContributions, role)) { 
     1200                                return; 
     1201                        } 
    11381202                    for (i in checkedContributions) { 
    11391203                        contributionId = checkedContributions[i]; 

  • indico/MaKaC/webinterface/tpls/ContributionDisplayFull.tpl

    r334896 rac1609  
    106106                                    <td></td> 
    107107                                    <td> 
    108                                         <%=_("Note: Press this button to submit your materials for reviewing")%>.<br> 
    109                                         <%=_("By doing this you will lock your materials until they are reviewed")%>. 
     108                                        <%=_("By clicking on this button you will lock your materials until they are reviewed")%>. 
    110109                                    </td> 
    111110                            <% end %> 

  • indico/MaKaC/webinterface/tpls/ContributionReviewingJudgements.tpl

    r0c1608 rac1609  
    77 
    88 
    9  
     9<div style="padding-left: 10px; padding-top: 10px"> 
     10<em><%= _("The reviewing mode choosen for this conference is")%>: <%= ConferenceChoiceStr%></em> 
     11</div> 
    1012<!-- Judgement of the editor --> 
     13<% if ConferenceChoice == 3 or ConferenceChoice == 4:%> 
    1114<table width="90%%" align="center" border="0" style="margin-bottom: 1em; margin-top: 1em"> 
    1215    <tr> 
    … …  
    3134    </tr> 
    3235</table> 
     36<% end %> 
    3337 
    3438<!-- List of advices from the reviewers --> 
     39<% if ConferenceChoice == 2 or ConferenceChoice == 4:%> 
    3540<table width="90%%" align="center" border="0" style="margin-bottom: 1em"> 
    3641    <tr> 
    … …  
    5964    </tr> 
    6065</table> 
    61  
    62  
     66<% end %> 
    6367 
    6468<!-- Final reviewing of the referee --> 
     69<% if ConferenceChoice == 2 or ConferenceChoice == 4:%> 
    6570<table width="90%%" align="center" border="0" style="margin-bottom: 1em"> 
    6671    <tr> 
    … …  
    108113            <% if not Review.getRefereeJudgement().isSubmitted(): %> 
    109114                <tr> 
    110                     <td colspan="2" align="center"> 
     115                    <td colspan="2" align="left"> 
    111116                        <span> 
    112117                        <%= _("This contribution has not been judged yet.")%><br> 
    … …  
    117122            <% end %> 
    118123        <% end %> 
    119          
    120         <tr> 
    121             <td nowrap class="titleCellTD" style="padding-top: 10px;"><span class="titleCellFormat"><strong><%= _("Judgement")%>:</strong></span></td> 
    122             <td style="padding-top: 10px;"> 
    123                 <div id="inPlaceEditJudgement"><strong><%= Review.getRefereeJudgement().getJudgement() %></strong></div> 
    124             </td> 
    125         </tr> 
    126124        <tr> 
    127125            <td nowrap class="titleCellTD"> 
    … …  
    135133            <td> 
    136134                <div id="inPlaceEditComments"></div> 
     135            </td> 
     136        </tr> 
     137        <tr> 
     138            <td nowrap class="titleCellTD"><span class="titleCellFormat"><strong><%= _("Judgement")%>:</strong></span></td> 
     139            <td> 
     140                <div id="inPlaceEditJudgement"><strong><%= Review.getRefereeJudgement().getJudgement() %></strong></div> 
    137141            </td> 
    138142        </tr> 
    … …  
    155159                         
    156160</table> 
     161<% end %> 
    157162 
    158163<script type="text/javascript"> 

  • indico/MaKaC/webinterface/tpls/JudgeEditing.tpl

    rc818e6 rac1609  
    88    </tr> 
    99    <tr> 
    10         <td nowrap class="titleCellTD"><span class="titleCellFormat"><%= _("Reviewing questions")%></span></td> 
     10        <td nowrap class="titleCellTD"><span class="titleCellFormat"><%= _("Reviewing questions")%>:</span></td> 
    1111        <td width="60%%" id="criteriaListDisplay"> 
     12        </td> 
     13    </tr>     
     14    <tr> 
     15        <td nowrap class="titleCellTD"><span class="titleCellFormat"><%= _("Comments")%>:</span></td> 
     16        <td> 
     17            <div id="inPlaceEditComments"></div> 
    1218        </td> 
    1319    </tr> 
    1420    <tr> 
    15         <td nowrap class="titleCellTD"><span class="titleCellFormat">Judgement</span></td> 
     21        <td nowrap class="titleCellTD"><span class="titleCellFormat"><strong><%= _("Judgement")%>:</strong></span></td> 
    1622        <td> 
    1723            <div id="inPlaceEditJudgement"><%= Editing.getJudgement() %></div> 
    1824        </td> 
    1925    </tr> 
     26    <% if not Editing.getJudgement(): %> 
     27        <% display = 'span' %> 
     28    <% end %> 
     29    <% else: %> 
     30            <% display = 'none' %> 
     31    <% end %> 
    2032    <tr> 
    21         <td nowrap class="titleCellTD"><span class="titleCellFormat">Comments</span></td> 
    22         <td> 
    23             <div id="inPlaceEditComments"></div> 
    24             These comments, along with your judgement, will be sent by e-mail to the author(s) 
    25         </td> 
    26     </tr> 
    27     <tr> 
    28         <td colspan="10"> 
     33        <td colspan="10" style="padding-top: 20px;"> 
    2934            <span id="submitbutton"></span> 
    3035            <span id="submittedmessage"></span> 
    … …  
    3237    </tr> 
    3338</table> 
    34  
    3539 
    3640<script type="text/javascript"> 
Note: See TracChangeset for help on using the changeset viewer.