#906 frontend: listens_for decorator is nicer than manual listen()
Merged 4 years ago by praiskup. Opened 4 years ago by praiskup.
Unknown source listens_for  into  master

@@ -5,7 +5,7 @@

  from sqlalchemy import and_

  from sqlalchemy.sql import func

  from sqlalchemy import asc, desc

- from sqlalchemy.event import listen

+ from sqlalchemy.event import listens_for

  from sqlalchemy.orm.attributes import NEVER_SET

  from sqlalchemy.orm.exc import NoResultFound

  from sqlalchemy.orm.attributes import get_history
@@ -552,6 +552,7 @@

              db.session.delete(copr_dir)

  

  

+ @listens_for(models.Copr.auto_createrepo, 'set')

  def on_auto_createrepo_change(target_copr, value_acr, old_value_acr, initiator):

      """ Emit createrepo action when auto_createrepo re-enabled"""

      if old_value_acr == NEVER_SET:
@@ -562,10 +563,6 @@

          ActionsLogic.send_createrepo(target_copr)

  

  

- listen(models.Copr.auto_createrepo, 'set', on_auto_createrepo_change,

-        active_history=True, retval=False)

- 

- 

  class BranchesLogic(object):

      @classmethod

      def get_or_create(cls, name, session=db.session):

no initial comment

rebased onto c1addd4e0b95ab887a5c505b7b358b8199b16015

4 years ago

listens_for decorator is nicer than manual listen()

It definitely is, +1

rebased onto e1f7dfb

4 years ago

Pull-Request has been merged by praiskup

4 years ago