Fixes #110: This does not really fix the problem. Log about attaching is

valid only in case really attach. Signed-off-by: Petr Hracek <phracek@redhat.com>
fedora-infra · Mar 11, 2016 · a99c1fb · a99c1fb
1 parent d71096d
commit a99c1fb
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/hotness/bz.py b/hotness/bz.py
@@ -123,10 +123,10 @@ def follow_up(self, text, bug):
         self.log.info("Followed up on bug: %s" % bug.weburl)
 
     def attach_patch(self, filename, description, bug):
-        self.log.debug("Attaching patch to bug %r" % bug.bug_id)
         if os.path.exists(filename) and os.path.getsize(filename) != 0:
+            self.log.debug("Attaching patch to bug %r" % bug.bug_id)
             self.bugzilla.attachfile(bug.bug_id, filename, description,
-                is_patch=True)
+                    is_patch=True)
             self.log.info("Attached patch to bug: %s" % bug.weburl)
 
     def ftbfs_bugs(self, name):