From 09dfdcf049ac7454df6625782535481d1ee8fd71 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Mar 27 2015 08:51:32 +0000 Subject: Do not publish fedmsg message on dependency change if ticket is private --- diff --git a/pagure/lib/__init__.py b/pagure/lib/__init__.py index 356dd4a..c8ad228 100644 --- a/pagure/lib/__init__.py +++ b/pagure/lib/__init__.py @@ -289,13 +289,14 @@ def add_issue_dependency(session, issue, issue_blocked, user, ticketfolder): #pagure.lib.notify.notify_assigned_issue(issue, user_obj) #pagure.lib.notify.notify_assigned_issue(issue_blocked, user_obj) - pagure.lib.notify.fedmsg_publish( - 'issue.dependency.added', - dict( - issue=issue.to_json(), - agent=user_obj.username, + if not issue.private: + pagure.lib.notify.fedmsg_publish( + 'issue.dependency.added', + dict( + issue=issue.to_json(), + agent=user_obj.username, + ) ) - ) return 'Dependency added' @@ -328,13 +329,14 @@ def remove_issue_dependency(session, issue, issue_blocked, user, ticketfolder): #pagure.lib.notify.notify_assigned_issue(issue, user_obj) #pagure.lib.notify.notify_assigned_issue(issue_blocked, user_obj) - pagure.lib.notify.fedmsg_publish( - 'issue.removed.added', - dict( - issue=issue.to_json(), - agent=user_obj.username, + if not issue.private: + pagure.lib.notify.fedmsg_publish( + 'issue.removed.added', + dict( + issue=issue.to_json(), + agent=user_obj.username, + ) ) - ) return 'Dependency removed'