diff --git a/scripts/process-git-requests/process-git-requests b/scripts/process-git-requests/process-git-requests index 3f06298..f631c22 100755 --- a/scripts/process-git-requests/process-git-requests +++ b/scripts/process-git-requests/process-git-requests @@ -485,12 +485,16 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc warned = True frflag = bug.get_flags('fedora-review') - 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!") + 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 m=re.search('Review Request:\s+([a-zA-Z0-9_+.-]+)\s+', bug.summary, re.I) if not m: