From 8bb53ad5b48d3335f61d514358254709ccfce8fa Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Aug 08 2016 17:30:39 +0000 Subject: Fix configuring and activating the Pagure CI plugin in one action Thanks to @skrzepto for finding the bug --- diff --git a/pagure/hooks/pagure_ci.py b/pagure/hooks/pagure_ci.py index ceb0d6a..689a0cb 100644 --- a/pagure/hooks/pagure_ci.py +++ b/pagure/hooks/pagure_ci.py @@ -125,11 +125,7 @@ class PagureCi(BaseHook): ''' Install the generic post-receive hook that allow us to call multiple post-receive hooks as set per plugin. ''' - hook = project.ci_hook[0] - if not hook.pagure_ci_token: - hook.pagure_ci_token = pagure.lib.login.id_generator(32) - SESSION.add(project) - SESSION.commit() + pass @classmethod def install(cls, project, dbobj): @@ -139,7 +135,10 @@ class PagureCi(BaseHook): should be installed ''' - pass + if not dbobj.pagure_ci_token: + dbobj.pagure_ci_token = pagure.lib.login.id_generator(32) + SESSION.add(dbobj) + SESSION.commit() @classmethod def remove(cls, project):