From e1b2f9aea0988b4448efa507da92e657f3638d83 Mon Sep 17 00:00:00 2001
From: Jon Ciesla <limburgher@gmail.com>
Date: Aug 17 2012 11:04:25 +0000
Subject: Even more robust flag checking.


---

diff --git a/scripts/process-git-requests/process-git-requests b/scripts/process-git-requests/process-git-requests
index afceae8..f631c22 100755
--- a/scripts/process-git-requests/process-git-requests
+++ b/scripts/process-git-requests/process-git-requests
@@ -485,16 +485,16 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc
         warned = True
 
     frflag = bug.get_flags('fedora-review')
-    if not frflag[0]['status']:
+    if not frflag:
         warnings.append("WARNING: fedora-review flag not set")
         warned = True
     else:
         if frflag[0]['status'] != '+':
             warnings.append("WARNING: fedora-review flag not set to '+'")
             warned = True
-    if frflag[0]['setter_email'] == bug.reporter:
-        warnings.append("WARNING: fedora-review flag set by review submitter!  Verify that review was approved by reviewer!")
-        warned = True
+        if frflag[0]['setter_email'] == bug.reporter:
+            warnings.append("WARNING: fedora-review flag set by review submitter!  Verify that review was approved by reviewer!")
+            warned = True
 
     m=re.search('Review Request:\s+([a-zA-Z0-9_+.-]+)\s+', bug.summary, re.I)
     if not m: