Changeset 228d27 in indico


Ignore:
Timestamp:
02/18/11 18:05:40 (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, 051b2622c51afb171a1dedb46a0df4fbb0cbd02e, 0da0c1403bae8e51d8229f460181c71b9e6dda72
Children:
de7e17
Parents:
64824a
git-author:
Leszek Syroka <leszek.marek.syroka@…> (01/12/11 17:02:51)
git-committer:
Jose Benito <jose.benito.gonzalez@…> (02/18/11 18:05:40)
Message:

[FIX] Volatile variables

  • volatile variables revised
  • _v_send_email removed
  • _v_allowReSchedule removed
  • fix#605
Location:
indico/MaKaC
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • indico/MaKaC/accessControl.py

    r9f7b3f6 r228d27  
    220220                    self.revokeAccessEmail(email) 
    221221                    av.linkTo(self.getOwner(), "manager") 
    222         #self._v_canuseraccess[av] = False 
    223         #return self._v_canuseraccess[av] 
    224222        return False 
    225223 
    … …  
    237235    def grantModificationEmail(self, email): 
    238236        """looks if the email is in the managersEmail list (list with the users with access to modification) 
    239         and if it's not it adds the email to the list""" 
    240         if not email in self.getModificationEmail(): 
     237        and if it's not it adds the email to the list 
     238            Returns True is email was added to the list, False if it was already there. 
     239        """ 
     240        if not email.lower() in map(lambda x: x.lower(), self.getModificationEmail()): 
    241241            self.getModificationEmail().append(email) 
    242242            self._p_changed = 1 
    243         self._notify('modificationGranted', email) 
     243            self._notify('modificationGranted', email) 
     244            return True 
     245        return False 
    244246 
    245247    def revokeModificationEmail(self, email): 
    … …  
    268270    def canModify( self, user ): 
    269271        """tells whether the specified user has modification privileges""" 
    270         #try: 
    271         #    return self._v_canmodify[user] 
    272         #except AttributeError: 
    273         #    self._v_canmodify = {} 
    274         #except KeyError: 
    275         #    pass 
    276272        if AdminList.getInstance().isAdmin( user ): 
    277             #self._v_canmodify[user] = 1 
    278             #return self._v_canmodify[user] 
    279273            return True 
    280274        for principal in self.managers: 
    281275            if (isinstance(principal, MaKaC.user.Avatar) or isinstance(principal, MaKaC.user.CERNGroup) or isinstance(principal, MaKaC.user.Group)) and principal.containsUser( user ): 
    282                 #self._v_canmodify[user] = 1 
    283                 #return self._v_canmodify[user] 
    284276                return True 
    285277        ret = False 
    … …  
    293285        return ret 
    294286 
    295         #self._v_canmodify[user] = 0 
    296         #return self._v_canmodify[user] 
    297         #return False 
    298  
    299287    def getModifierList( self ): 
    300288        """returns a list of those principals which have modification 

  • indico/MaKaC/conference.py

    r9f7b3f6 r228d27  
    62526252                r.linkTo(self, "manager") 
    62536253            elif sb.getEmail() != "": 
    6254                 self.__ac.grantModificationEmail(sb.getEmail()) 
    6255                 #send email once 
    6256                 try : 
    6257                     self._v_emailSent 
    6258                 except: 
    6259                     self._v_emailSent = [] 
    6260                 if sendEmail and not sb.getEmail() in self._v_emailSent: 
    6261                     self._v_emailSent.append(sb.getEmail()) 
     6254                modificationEmailGranted = self.__ac.grantModificationEmail(sb.getEmail()) 
     6255                if modificationEmailGranted and sendEmail: 
    62626256                    notif = pendingQueues._PendingManagerNotification( [sb] ) 
    62636257                    mail.GenericMailer.sendAndLog( notif, self.getConference() ) 
    … …  
    92769270 
    92779271    def _grantSubmissionEmail(self, email): 
    9278         if not email in self.getSubmitterEmailList(): 
    9279             self.getSubmitterEmailList().append(email.lower()) 
     9272        """ 
     9273            Returns True if submission email was granted. False if email was already in the list. 
     9274        """ 
     9275        if not email.lower() in map(lambda x: x.lower(), self.getSubmitterEmailList()): 
     9276            self.getSubmitterEmailList().append(email) 
     9277            return True 
     9278        return False 
    92809279 
    92819280    def revokeSubmissionEmail(self, email): 
    92829281        if email in self.getSubmitterEmailList(): 
    9283            self.getSubmitterEmailList().remove(email) 
    9284            self._p_changed=1 
     9282            self.getSubmitterEmailList().remove(email) 
     9283            self._p_changed=1 
    92859284 
    92869285    def grantSubmission(self,sb, sendEmail=True): 
    … …  
    93019300            elif sb.getEmail() != "": 
    93029301                self.getConference().getPendingQueuesMgr().addPendingSubmitter(sb, False) 
    9303                 self._grantSubmissionEmail(sb.getEmail()) 
    9304                 #send email once 
    9305                 try : 
    9306                     self._v_emailSent 
    9307                 except: 
    9308                     self._v_emailSent = [] 
    9309                 if sendEmail and not sb.getEmail() in self._v_emailSent: 
    9310                     self._v_emailSent.append(sb.getEmail()) 
     9302                submissionEmailGranted = self._grantSubmissionEmail(sb.getEmail()) 
     9303                if submissionEmailGranted and sendEmail: 
    93119304                    notif = pendingQueues._PendingSubmitterNotification( [sb] ) 
    93129305                    mail.GenericMailer.sendAndLog( notif, self.getConference() ) 

  • indico/MaKaC/schedule.py

    r64824a r228d27  
    8383        self._owner=owner 
    8484        self._entryGen=Counter() 
    85         self._v_allowReSchedule=True 
    8685        self._allowParallel=True 
    8786 
    … …  
    262261        except AttributeError: 
    263262            self._allowParallel=True 
    264         try: 
    265             if self._v_allowReSchedule: 
    266                 pass 
    267         except AttributeError: 
    268             self._v_allowReSchedule=True 
    269         if self._v_allowReSchedule: 
    270             self._v_allowReSchedule=False 
    271             self._entries.sort(self.cmpEntries) 
    272             lastEntry=None 
    273             for entry in self._entries: 
    274                 if lastEntry is not None: 
    275                     if not self._allowParallel: 
    276                         if lastEntry.collides(entry): 
    277                             entry.setStartDate(lastEntry.getEndDate()) 
    278                 lastEntry=entry 
    279             self._v_allowReSchedule=True 
     263        self._entries.sort(self.cmpEntries) 
     264        lastEntry=None 
     265        for entry in self._entries: 
     266            if lastEntry is not None: 
     267                if not self._allowParallel: 
     268                    if lastEntry.collides(entry): 
     269                        entry.setStartDate(lastEntry.getEndDate()) 
     270            lastEntry=entry 
    280271        self._p_changed = 1 
    281272 
    … …  
    403394            after the other without any gap 
    404395        """ 
    405         self._v_allowReSchedule=False 
    406396        refDate=self.getStartDate('UTC') 
    407397        for entry in self._entries: 
    408398            entry.setStartDate(refDate) 
    409399            refDate=entry.getEndDate() 
    410         self._v_allowReSchedule=True 
    411400 
    412401    def moveUpEntry(self,entry,tz=None): 
    … …  
    588577        if len(entriesDay)<2: 
    589578            return 
    590         self._v_allowReSchedule=False 
    591579        entrypos = 0 
    592580        if entry in entriesDay: 
    … …  
    606594            entry.setStartDate(preventry.getStartDate(), check=0, moveEntries=1) 
    607595            preventry.setStartDate(entry.getEndDate(), check=0, moveEntries=1) 
    608         self._v_allowReSchedule=True 
    609596        self.reSchedule() 
    610597        self._p_changed = 1 
    … …  
    616603        if len(entriesDay)<2: 
    617604            return 
    618         self._v_allowReSchedule=False 
    619605        entrypos = 0 
    620606        if entry in entriesDay: 
    … …  
    633619            nextentry.setStartDate(entry.getStartDate(), check=0, moveEntries=1) 
    634620            entry.setStartDate(nextentry.getEndDate(), check=0, moveEntries=1) 
    635         self._v_allowReSchedule=True 
    636621        self.reSchedule() 
    637622        self._p_changed = 1 
    … …  
    777762        if len(entries)<2: 
    778763            return 
    779         self._v_allowReSchedule=False 
    780764        entrypos = 0 
    781765        if entry in entries: 
    … …  
    795779            entry.setStartDate(preventry.getStartDate(),check=0,moveEntries=1) 
    796780            preventry.setStartDate(entry.getEndDate(),check=0,moveEntries=1) 
    797         self._v_allowReSchedule=True 
    798781        self.reSchedule() 
    799782        self._p_changed = 1 
    … …  
    803786        if len(entries)<2: 
    804787            return 
    805         self._v_allowReSchedule=False 
    806788        entrypos = 0 
    807789        if entry in entries: 
    … …  
    820802            nextentry.setStartDate(entry.getStartDate(),check=0,moveEntries=1) 
    821803            entry.setStartDate(nextentry.getEndDate(),check=0,moveEntries=1) 
    822         self._v_allowReSchedule=True 
    823804        self.reSchedule() 
    824805        self._p_changed = 1 
    … …  
    868849 
    869850    def reSchedule(self): 
    870         try: 
    871             if self._v_allowReSchedule: 
    872                 pass 
    873         except AttributeError: 
    874             self._v_allowReSchedule=True 
    875         if self._v_allowReSchedule: 
    876             self._v_allowReSchedule=False 
    877             for e in self._entries: 
    878                 e.setStartDate(self.getStartDate()) 
    879             self._v_allowReSchedule=True 
     851        for e in self._entries: 
     852            e.setStartDate(self.getStartDate()) 
    880853 
    881854class SlotSchTypeFactory: 
Note: See TracChangeset for help on using the changeset viewer.