diff options
author | David Pursehouse <david.pursehouse@gmail.com> | 2013-10-17 07:52:06 -0700 |
---|---|---|
committer | David Pursehouse <david.pursehouse@gmail.com> | 2013-10-17 07:52:06 -0700 |
commit | 59c46e1c7b8181c3d2afdf91270ce9e26d982b68 (patch) | |
tree | 48d8788098ab48b3c3451cd4e60705341bbe99b7 | |
parent | cff9ece4a3e2742458c9e0744f2f5f36c7bcf0fb (diff) | |
parent | 69649921b58472db5176c8476d9e0fedaba0de55 (diff) | |
download | pygerrit-59c46e1c7b8181c3d2afdf91270ce9e26d982b68.tar.gz |
Merge pull request #14 from EnJens/fix-missing-reason
Reason is optional in abandon and restore changes
-rw-r--r-- | pygerrit/events.py | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/pygerrit/events.py b/pygerrit/events.py index ebbb327..000387e 100644 --- a/pygerrit/events.py +++ b/pygerrit/events.py @@ -220,7 +220,8 @@ class MergeFailedEvent(GerritEvent): self.change = Change(json_data["change"]) self.patchset = Patchset(json_data["patchSet"]) self.submitter = Account(json_data["submitter"]) - self.reason = json_data["reason"] + if 'reason' in json_data: + self.reason = json_data["reason"] except KeyError as e: raise GerritError("MergeFailedEvent: %s" % e) @@ -241,7 +242,8 @@ class ChangeAbandonedEvent(GerritEvent): self.change = Change(json_data["change"]) self.patchset = Patchset.from_json(json_data) self.abandoner = Account(json_data["abandoner"]) - self.reason = json_data["reason"] + if 'reason' in json_data: + self.reason = json_data["reason"] except KeyError as e: raise GerritError("ChangeAbandonedEvent: %s" % e) @@ -262,7 +264,8 @@ class ChangeRestoredEvent(GerritEvent): self.change = Change(json_data["change"]) self.patchset = Patchset.from_json(json_data) self.restorer = Account(json_data["restorer"]) - self.reason = json_data["reason"] + if 'reason' in json_data: + self.reason = json_data["reason"] except KeyError as e: raise GerritError("ChangeRestoredEvent: %s" % e) |