diff options
author | Christian Hammond <chipx86@chipx86.com> | 2009-09-22 18:02:24 -0700 |
---|---|---|
committer | Christian Hammond <chipx86@chipx86.com> | 2009-09-22 18:02:24 -0700 |
commit | c5ce0847175c5c8a03aea70d6623ee0960b6409d (patch) | |
tree | 830d5707093edbcdb7fcf0b5e9a150e81787bf50 | |
parent | 45a9029955e22404dba0687300b3d1bbfc0d03b5 (diff) |
Don't unconditionally publish the draft when setting a field.
When setting a field (such as target reviewers) from post-review, we were
saving the draft, always. This is bad enough, but we were doing this before
even updating the diff. This would result is a published draft and an extra
change description entry. We no longer save the draft, unless the user
uses --publish.
Fixes bug #811
-rwxr-xr-x | rbtools/postreview.py | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/rbtools/postreview.py b/rbtools/postreview.py index d56c06e..db91b82 100755 --- a/rbtools/postreview.py +++ b/rbtools/postreview.py @@ -2237,8 +2237,6 @@ def tempt_fate(server, tool, changenum, diff_content=None, a diff. On success, the review request path is displayed. """ try: - save_draft = False - if options.rid: review_request = server.get_review_request(options.rid) else: @@ -2247,40 +2245,30 @@ def tempt_fate(server, tool, changenum, diff_content=None, if options.target_groups: server.set_review_request_field(review_request, 'target_groups', options.target_groups) - save_draft = True if options.target_people: server.set_review_request_field(review_request, 'target_people', options.target_people) - save_draft = True if options.summary: server.set_review_request_field(review_request, 'summary', options.summary) - save_draft = True if options.branch: server.set_review_request_field(review_request, 'branch', options.branch) - save_draft = True if options.bugs_closed: server.set_review_request_field(review_request, 'bugs_closed', options.bugs_closed) - save_draft = True if options.description: server.set_review_request_field(review_request, 'description', options.description) - save_draft = True if options.testing_done: server.set_review_request_field(review_request, 'testing_done', options.testing_done) - save_draft = True - - if save_draft: - server.save_draft(review_request) except APIError, e: rsp, = e.args if rsp['err']['code'] == 103: # Not logged in |