From 545a5ec11095ae2088a49b5d8ae9cc7010d1d0ea Mon Sep 17 00:00:00 2001 From: Jon Ciesla Date: Thu, 5 Jul 2012 10:30:19 -0500 Subject: [PATCH] Let's make just one call to BZ for the fedora-review flag. --- scripts/process-git-requests/process-git-requests | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/scripts/process-git-requests/process-git-requests b/scripts/process-git-requests/process-git-requests index 79ffb6c..3f06298 100755 --- a/scripts/process-git-requests/process-git-requests +++ b/scripts/process-git-requests/process-git-requests @@ -484,11 +484,10 @@ def process_new_request(bug, comment, allcomments, firstfound, pkgdb, fas, branc warnings.append("WARNING: Ticket is not assigned to anyone.") warned = True - if bug.flags.find('fedora-review+') < 0: + frflag = bug.get_flags('fedora-review') + if frflag[0]['status'] != '+': warnings.append("WARNING: fedora-review flag not set to '+'") warned = True - - frflag = bug.get_flags('fedora-review') 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