From e1f7dfb912b4bbb55d88f2be906c62fb3f7e3e20 Mon Sep 17 00:00:00 2001
From: Pavel Raiskup <praiskup@redhat.com>
Date: Aug 06 2019 08:05:48 +0000
Subject: frontend: listens_for decorator is nicer than manual listen()


---

diff --git a/frontend/coprs_frontend/coprs/logic/coprs_logic.py b/frontend/coprs_frontend/coprs/logic/coprs_logic.py
index 640819a..a225c64 100644
--- a/frontend/coprs_frontend/coprs/logic/coprs_logic.py
+++ b/frontend/coprs_frontend/coprs/logic/coprs_logic.py
@@ -5,7 +5,7 @@ import datetime
 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 @@ class CoprDirsLogic(object):
             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 @@ def on_auto_createrepo_change(target_copr, value_acr, old_value_acr, initiator):
         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):