diff options
authorLuis Araujo <>2016-05-25 15:10:28 (GMT)
committerLuis Araujo <>2016-06-01 09:36:20 (GMT)
commitbb3cd6748765cacb698dae78c01ff4a4359468df (patch)
parentd0794058a9a54f82bcf9d30ea1349f85f51633e7 (diff)
Avoid updating the bug if the status won't be updatedHEADmaster
This commit avoids that qa-bughandler updates the bug unless the status is moving forward. Reviewed-by: Héctor Orón Martínez <> Signed-off-by: Luis Araujo <> Differential Revision:
1 files changed, 5 insertions, 2 deletions
diff --git a/ b/
index 514a17d..1adffd1 100755
--- a/
+++ b/
@@ -276,12 +276,15 @@ def do_main(settings):
# Only update the status if it moved us forward towards CLOSED and only
# update resolution when also doing a status update
if settings.cli.args.status is not None:
- if compare_status(bug["status"], settings.cli.args.status) >= 0:
+ if compare_status(bug["status"], settings.cli.args.status) > 0:
fields['status'] = settings.cli.args.status
# only handle resolution if a status is given
if settings.cli.args.resolution is not None:
fields['resolution'] = settings.cli.args.resolution
+ else:
+ # Return and do nothing if the status is not moving forward or it is
+ # the same already set.
+ return
# handle additional fields
if settings.cli.args.fields is not None: