Ignore:
Timestamp:
03/02/11 14:03:39 (2 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, b8c30da8ebdbdcbd675a873997cc3e95f567de49, 4287315ec967a3da168d83963c14001db8487d53
Children:
a3574e
Parents:
c3cd24
git-author:
Alexis Castilla Hernandez <alexis.castilla.hernandez@…> (03/02/11 10:10:57)
git-committer:
Jose Benito <jose.benito.gonzalez@…> (03/02/11 14:03:39)
Message:

[IMP] Code impromevents in AR and PR

  • Changed set and get names in methods which are not getters or setters
  • Changed numbers by constant names in type of paper reviewing
  • Removed not necessary try except in get methods
  • Included parameter manager in abstract reviewing services
  • Removed the 'for..' to fossilize in abstractReviewing.py and reviewing.py
  • Removed HTML code from notification templates python file to .tpl files and fixed a bug with 'from' var
  • Added _() to tabs names
  • New classes base for inheritance in PropToAcc? and PropToRej? (both, track manager and conference manager)
  • Moved code to AbstractReviewing?.js and changed to OO
  • Changed the python loop by js loop in PropToAcc/PropToRej? files
File:
1 edited

Legend:

Unmodified
Added
Removed
  • indico/MaKaC/webinterface/tpls/ShowExistingReviewingMaterial.tpl

    r9b6bab r4bbdbe  
    11<% import MaKaC.webinterface.urlHandlers as urlHandlers %> 
    22<% from MaKaC.webinterface.rh.conferenceBase import RHSubmitMaterialBase %> 
     3<% from MaKaC.reviewing import ConferencePaperReview as CPR %> 
     4 
    35<div id="showStep1" class="groupTitleSmallPaper"><span><%= _("Step 1 - Upload paper") %></span></div> 
    46<%= existingMaterialsTitle %> 
     
    3638    var uploadAction = Indico.Urls.UploadAction.contribution; 
    3739    var visibility = ''; 
    38      <% if  self._target.getConference().getConfPaperReview().getChoice() == 3: %> 
     40     <% if  self._target.getConference().getConfPaperReview().getChoice() == CPR.LAYOUT_REVIEWING: %> 
    3941        <% if not self._target.getReviewManager().getLastReview().isAuthorSubmitted() and not self._target.getReviewManager().getLastReview().getEditorJudgement().isSubmitted():%> 
    4042            <% if showSendButton: %> 
     
    4951        <% end %> 
    5052    <% end %> 
    51     <% if self._target.getConference().getConfPaperReview().getChoice() == 2: %> 
     53    <% if self._target.getConference().getConfPaperReview().getChoice() == CPR.CONTENT_REVIEWING: %> 
    5254        <% if not self._target.getReviewManager().getLastReview().isAuthorSubmitted() and not (self._target.getReviewManager().getLastReview().getRefereeJudgement().isSubmitted() or self._target.getReviewManager().getLastReview().anyReviewerHasGivenAdvice()): %> 
    5355            <% if showSendButton: %> 
     
    6264        <% end %> 
    6365    <% end %> 
    64     <% if  self._target.getConference().getConfPaperReview().getChoice() == 4: %> 
     66    <% if  self._target.getConference().getConfPaperReview().getChoice() == CPR.CONTENT_AND_LAYOUT_REVIEWING: %> 
    6567        <% 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()): %> 
    6668            <% if showSendButton: %> 
Note: See TracChangeset for help on using the changeset viewer.