Changeset 3ec350 in indico


Ignore:
Timestamp:
11/11/10 16:57:05 (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:
14d8ef
Parents:
d570ef
git-author:
Alexis Castilla Hernandez <alexis.castilla.hernandez@…> (11/02/10 15:07:00)
git-committer:
Jose Benito <jose.benito.gonzalez@…> (11/11/10 16:57:05)
Message:

[FIX] Several things fixed:

  • Upload paper link (protection and visibility).
  • Protection in the uploaded materials of paper reviewing.
  • Send Button visibility.
Location:
indico/MaKaC/webinterface
Files:
6 edited

Legend:

Unmodified
Added
Removed

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

    rce685f r3ec350  
    2525 
    2626class WPContributionReviewing( WPContributionModifBase ): 
    27      
     27 
    2828    def __init__(self, rh, contribution): 
    2929        WPContributionModifBase.__init__(self, rh, contribution) 
    3030        self._aw = rh.getAW() 
    31      
    32     def _setActiveTab( self ): 
    33         self._subtabReviewing.setActive() 
    34          
    35     def _getTabContent( self, params ):   
     31 
     32    def _setActiveTab( self ): 
     33        self._subtabReviewing.setActive() 
     34 
     35    def _getTabContent( self, params ): 
    3636        wc = WContributionReviewing(self._target.getConference(), self._aw) 
    3737        assignRefereeURL = urlHandlers.UHAssignReferee.getURL(self._target) 
    … …  
    4141        assignReviewingURL = urlHandlers.UHAssignReviewing.getURL(self._target) 
    4242        removeAssignReviewingURL = urlHandlers.UHRemoveAssignReviewing.getURL(self._target) 
    43      
     43 
    4444        return wc.getHTML(self._target, assignRefereeURL, removeAssignRefereeURL, assignEditingURL, removeAssignEditingURL, assignReviewingURL, removeAssignReviewingURL) 
    4545 
    46 class WContributionReviewing(wcomponents.WTemplated):  
    47      
     46class WContributionReviewing(wcomponents.WTemplated): 
     47 
    4848    def __init__(self, conference, aw): 
    4949        self._conf = conference 
    5050        self._aw = aw 
    51      
     51 
    5252    def getHTML( self, target, assignRefereeURL, removeAssignRefereeURL, assignEditingURL, removeAssignEditingURL, assignReviewingURL, removeAssignReviewingURL ): 
    5353 
    … …  
    6060                  "removeAssignReviewingURL" : removeAssignReviewingURL} 
    6161        return wcomponents.WTemplated.getHTML(self, params) 
    62          
     62 
    6363    def getVars( self): 
    6464        vars = wcomponents.WTemplated.getVars( self ) 
    … …  
    6868        reviewManager = self.__target.getReviewManager() 
    6969        canAssignReferee = self._conf.getConfReview().isPaperReviewManager(self._aw.getUser()) or self._conf.canModify(self._aw) 
    70          
     70 
    7171        vars["Conference"] = self._conf 
    7272        vars["ConfReview"] = self._conf.getConfReview() 
    … …  
    8787        else: 
    8888            vars["removeRefereeConfirm"] = "" 
    89          
     89 
    9090        return vars 
    9191 
    9292 
    9393class WPContributionReviewingJudgements( WPContributionModifBase ): 
    94      
     94 
    9595    def __init__(self, rh, contribution): 
    9696        WPContributionModifBase.__init__(self, rh, contribution) 
    9797        self._aw = rh.getAW() 
    98      
     98 
    9999    def _setActiveTab( self ): 
    100100        self._subtabReviewing.setActive() 
    101101        self._subTabJudgements.setActive() 
    102          
    103          
    104     def _getTabContent( self, params ):   
    105         wc = WContributionReviewingJudgements(self._target.getConference(), self._aw)         
     102 
     103 
     104    def _getTabContent( self, params ): 
     105        wc = WContributionReviewingJudgements(self._target.getConference(), self._aw) 
    106106        finalJudgeURL = urlHandlers.UHFinalJudge.getURL(self._target) 
    107      
     107 
    108108        return wc.getHTML(self._target, finalJudgeURL) 
    109      
     109 
    110110class WPContributionModifReviewingMaterials( WPContributionModifBase ): 
    111      
     111 
    112112    def __init__(self, rh, contribution): 
    113113        WPContributionModifBase.__init__(self, rh, contribution) 
    114114        self._aw = rh.getAW() 
    115      
     115 
    116116    def _setActiveTab( self ): 
    117117        self._subtabReviewing.setActive() 
    118118        self._subTabRevMaterial.setActive() 
    119          
    120          
     119 
     120 
    121121    def _getTabContent( self, pars ): 
    122         wc=wcomponents.WShowExistingReviewingMaterial(self._target, True) 
     122        wc=wcomponents.WShowExistingReviewingMaterial(self._target, False, False) 
    123123        return wc.getHTML( pars ) 
    124      
    125 class WContributionReviewingJudgements(wcomponents.WTemplated):  
    126      
     124 
     125class WContributionReviewingJudgements(wcomponents.WTemplated): 
     126 
    127127    def __init__(self, conference, aw): 
    128128        self._conf = conference 
    129129        self._aw = aw 
    130      
     130 
    131131    def getHTML( self, target, finalJudgeURL ): 
    132132 
    … …  
    134134        params = {"finalJudgeURL": finalJudgeURL} 
    135135        return wcomponents.WTemplated.getHTML(self, params) 
    136          
     136 
    137137    def getVars( self): 
    138138        vars = wcomponents.WTemplated.getVars( self ) 
    … …  
    140140        conferenceChoice = self._conf.getConfReview().getChoice() 
    141141        conferenceChoiceStr = self._conf.getConfReview().getReviewingMode() 
    142         reviewManager = self.__target.getReviewManager()         
     142        reviewManager = self.__target.getReviewManager() 
    143143        vars["Conference"] = self._conf 
    144144        vars["ConfReview"] = self._conf.getConfReview() 
    … …  
    152152        vars["Review"] = self.__target.getReviewManager().getLastReview() 
    153153        vars["TrackList"] = self._conf.getTrackList() 
    154          
    155         return vars 
    156      
     154 
     155        return vars 
     156 
    157157 
    158158 
    159159class WPJudgeEditing( WPContributionModifBase ): 
    160      
     160 
    161161    def _setActiveTab( self ): 
    162162        self._subtabReviewing.setActive() 
    163163        self._tabJudgeEditing.setActive() 
    164          
    165     def _getTabContent( self, params ):   
     164 
     165    def _getTabContent( self, params ): 
    166166        wc = WJudgeEditing(self._target) 
    167167        return wc.getHTML(self._target) 
    168168 
    169 class WJudgeEditing(wcomponents.WTemplated):  
    170      
     169class WJudgeEditing(wcomponents.WTemplated): 
     170 
    171171    def __init__(self, contrib): 
    172172        self._contrib = contrib 
    173      
     173 
    174174    def getHTML( self, target): 
    175175        return wcomponents.WTemplated.getHTML(self, {}) 
    176          
    177     def getVars( self): 
    178         vars = wcomponents.WTemplated.getVars( self ) 
    179          
     176 
     177    def getVars( self): 
     178        vars = wcomponents.WTemplated.getVars( self ) 
     179 
    180180        vars["Contribution"] = self._contrib 
    181181        vars["ConfReview"] = self._contrib.getConference().getConfReview() 
    … …  
    187187 
    188188class WPGiveAdvice( WPContributionModifBase ): 
    189          
     189 
    190190    def _setActiveTab( self ): 
    191191        self._subtabReviewing.setActive() 
    192192        self._tabGiveAdvice.setActive() 
    193193 
    194          
    195     def _getTabContent( self, params ):   
     194 
     195    def _getTabContent( self, params ): 
    196196        wc = WGiveAdvice(self._target, self._getAW()) 
    197197        return wc.getHTML() 
    198198 
    199199class WGiveAdvice(wcomponents.WTemplated): 
    200      
     200 
    201201    def __init__(self, contrib, aw): 
    202202        self._contrib = contrib 
    … …  
    206206        self.__reviewer = self._aw.getUser() 
    207207        return wcomponents.WTemplated.getHTML(self, {}) 
    208          
     208 
    209209    def getVars(self): 
    210210        vars = wcomponents.WTemplated.getVars( self ) 
    211          
     211 
    212212        vars["Contribution"] = self._contrib 
    213213        vars["ConfReview"] = self._contrib.getConference().getConfReview() 
    … …  
    215215        vars["Advice"] = self._contrib.getReviewManager().getLastReview().getAdviceFrom(self.__reviewer) 
    216216        vars["Review"] = self._contrib.getReviewManager().getLastReview() 
    217          
    218         return vars 
    219      
     217 
     218        return vars 
     219 
    220220class WContributionReviewingDisplay(wcomponents.WTemplated): 
    221     """  
    222221    """ 
    223      
     222    """ 
     223 
    224224    def __init__(self, contribution): 
    225225        self._contribution = contribution 
    226          
     226 
    227227    def getHTML(self, params): 
    228228        return wcomponents.WTemplated.getHTML(self, params) 
    229      
     229 
    230230    def getVars(self): 
    231231        vars = wcomponents.WTemplated.getVars( self ) 
    … …  
    235235        vars["ConferenceChoice"] = self._contribution.getConference().getConfReview().getChoice() 
    236236        return vars 
    237      
     237 
    238238class WPContributionReviewingHistory(WPContributionModifBase): 
    239      
     239 
    240240    def _setActiveTab( self ): 
    241241        self._subtabReviewing.setActive() 
    242242        self._subTabReviewingHistory.setActive() 
    243          
     243 
    244244    def _getTabContent( self, params ): 
    245245        wc = WContributionReviewingHistory(self._target) 
    246246        return wc.getHTML({"ShowReviewingTeam" : True}) 
    247247 
    248 class WContributionReviewingHistory(wcomponents.WTemplated):  
    249      
     248class WContributionReviewingHistory(wcomponents.WTemplated): 
     249 
    250250    def __init__(self, contribution): 
    251251        self._contribution = contribution 
    252252        self._conf = contribution.getConference() 
    253      
     253 
    254254    def getHTML( self, params ): 
    255255 
    256256        return wcomponents.WTemplated.getHTML(self, params) 
    257          
    258     def getVars( self): 
    259         vars = wcomponents.WTemplated.getVars( self ) 
    260          
     257 
     258    def getVars( self): 
     259        vars = wcomponents.WTemplated.getVars( self ) 
     260 
    261261        vars["ConferenceChoice"] = self._conf.getConfReview().getChoice() 
    262262        vars["Versioning"] = self._contribution.getReviewManager().getSortedVerioning() 

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

    rc3dea8 r3ec350  
    256256 
    257257        vars["MaterialList"] = wcomponents.WShowExistingMaterial(self._contrib, showTitle=False).getHTML() 
    258         vars["ReviewingMatList"] = wcomponents.WShowExistingReviewingMaterial(self._contrib, False).getHTML() 
     258        vars["ReviewingMatList"] = wcomponents.WShowExistingReviewingMaterial(self._contrib, False, True).getHTML() 
    259259 
    260260        vars["duration"]="" 

  • indico/MaKaC/webinterface/rh/contribMod.py

    ra89f94 r3ec350  
    13731373            and (not material or material.getReviewingState() < 3): 
    13741374            return 
    1375         RHContribModifBaseSpecialSesCoordRights._checkProtection(self) 
     1375        if not (RCContributionPaperReviewingStaff.hasRights(self) and not self._target.getReviewManager().getLastReview().isAuthorSubmitted()): 
     1376            RHContribModifBaseSpecialSesCoordRights._checkProtection(self) 
    13761377 
    13771378    def _checkParams(self, params): 

  • indico/MaKaC/webinterface/rh/fileAccess.py

    rb71d0a r3ec350  
    2626from MaKaC.webinterface.pages import files 
    2727from MaKaC.common import Config 
    28 from MaKaC.errors import MaKaCError, NoReportError 
     28from MaKaC.errors import MaKaCError, NoReportError, AccessError 
    2929 
    3030from email.Utils import formatdate 
     31from MaKaC.conference import Reviewing 
     32from MaKaC.webinterface.rh.contribMod import RCContributionPaperReviewingStaff 
     33from copy import copy 
    3134 
    3235class RHFileAccess( RHFileBase, RHDisplayBaseProtected ): 
    3336    _uh  = urlHandlers.UHFileAccess 
    34      
     37 
    3538    def _checkParams( self, params ): 
    3639        try: 
    … …  
    4043 
    4144    def _checkProtection( self ): 
     45        if isinstance(self._file.getOwner(), Reviewing): 
     46            selfcopy = copy(self) 
     47            selfcopy._target = self._file.getOwner().getContribution() 
     48            if not (RCContributionPaperReviewingStaff.hasRights(selfcopy) or \ 
     49                selfcopy._target.canUserSubmit(self.getAW().getUser()) or \ 
     50                self._target.canModify( self.getAW() )): 
     51                raise AccessError() 
    4252        RHDisplayBaseProtected._checkProtection( self ) 
    43      
     53 
    4454    def _process( self ): 
    4555 
    … …  
    6171class RHFileAccessStoreAccessKey( RHFileBase ): 
    6272    _uh = urlHandlers.UHFileEnterAccessKey 
    63      
     73 
    6474    def _checkParams( self, params ): 
    6575        RHFileBase._checkParams(self, params ) 
    6676        self._accesskey = params.get( "accessKey", "" ).strip() 
    67      
     77 
    6878    def _checkProtection( self ): 
    6979        pass 
    70          
     80 
    7181    def _process( self ): 
    7282        access_keys = self._getSession().getVar("accessKeys") 
    … …  
    7888        self._redirect( url ) 
    7989 
    80          
     90 
    8191class RHVideoWmvAccess( RHLinkBase, RHDisplayBaseProtected ): 
    8292    _uh  = urlHandlers.UHVideoWmvAccess 
    83      
     93 
    8494    def _checkParams( self, params ): 
    8595        try: 
    … …  
    91101        """targets for this RH are exclusively URLs so no protection apply""" 
    92102        return 
    93      
     103 
    94104    def _process( self ): 
    95105        p = files.WPVideoWmv(self, self._link ) 
    96106        return p.display() 
    97      
     107 
    98108class RHVideoFlashAccess( RHLinkBase, RHDisplayBaseProtected ): 
    99109    _uh  = urlHandlers.UHVideoFlashAccess 
    100      
     110 
    101111    def _checkParams( self, params ): 
    102112        try: 
    … …  
    108118        """targets for this RH are exclusively URLs so no protection apply""" 
    109119        return 
    110      
     120 
    111121    def _process( self ): 
    112122        p = files.WPVideoFlash(self, self._link ) 

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

    rc84bed r3ec350  
    3232    var visibility = ''; 
    3333     <% if  self._target.getConference().getConfReview().getChoice() == 3: %> 
    34         <% if not self._target.getReviewManager().getLastReview().getEditorJudgement().isSubmitted():%> 
     34        <% if not self._target.getReviewManager().getLastReview().isAuthorSubmitted() and not self._target.getReviewManager().getLastReview().getEditorJudgement().isSubmitted():%> 
    3535            visibility = 'visible'; 
    3636        <% end %> 
    … …  
    4040    <% end %> 
    4141    <% if self._target.getConference().getConfReview().getChoice() == 2: %> 
    42         <% if not (self._target.getReviewManager().getLastReview().getRefereeJudgement().isSubmitted() or self._target.getReviewManager().getLastReview().anyReviewerHasGivenAdvice()): %> 
     42        <% if not self._target.getReviewManager().getLastReview().isAuthorSubmitted() and not (self._target.getReviewManager().getLastReview().getRefereeJudgement().isSubmitted() or self._target.getReviewManager().getLastReview().anyReviewerHasGivenAdvice()): %> 
    4343            visibility = 'visible'; 
    4444        <% end %> 
    … …  
    4848    <% end %> 
    4949    <% if  self._target.getConference().getConfReview().getChoice() == 4: %> 
    50         <% if not (self._target.getReviewManager().getLastReview().getRefereeJudgement().isSubmitted() or self._target.getReviewManager().getLastReview().anyReviewerHasGivenAdvice() or self._target.getReviewManager().getLastReview().getEditorJudgement().isSubmitted()): %> 
     50        <% if not self._target.getReviewManager().getLastReview().isAuthorSubmitted() and not (self._target.getReviewManager().getLastReview().getRefereeJudgement().isSubmitted() or self._target.getReviewManager().getLastReview().anyReviewerHasGivenAdvice() or self._target.getReviewManager().getLastReview().getEditorJudgement().isSubmitted()): %> 
    5151            visibility = 'visible'; 
    5252        <% end %> 
    … …  
    6464<% end %> 
    6565 
    66 <% if existingMaterialsTitle == " ": %> 
     66<% if showSendButton: %> 
    6767    $E('SendBtnForm').dom.style.display = ''; 
    6868<% end %> 

  • indico/MaKaC/webinterface/wcomponents.py

    r08c9ac r3ec350  
    56145614class WShowExistingReviewingMaterial(WTemplated): 
    56155615 
    5616     def __init__(self,target,showTitle=True): 
     5616    def __init__(self,target,showTitle=True, showSendButton=False): 
    56175617        self._target=target 
    56185618        self._showTitle=showTitle 
     5619        self._showSendButton = showSendButton 
    56195620 
    56205621 
    … …  
    56325633        vars["resourcesLinkModifHandler"] = vars.get("resourcesLinkModifHandler", None) 
    56335634        vars["resourcesLinkProtectHandler"] = vars.get("resourcesLinkProtectHandler", None) 
     5635        # This var shows you if you are requesting the template from the contribution display (True) 
     5636        # or from the paper reviewing tab in the contribution (False), as consequence you will have (or not) 
     5637        # the "send" button 
     5638        vars["showSendButton"] = self._showSendButton 
    56345639 
    56355640        return vars 
Note: See TracChangeset for help on using the changeset viewer.