diff options
author | Will Woods <wwoods@redhat.com> | 2008-01-02 17:26:13 -0500 |
---|---|---|
committer | Will Woods <wwoods@redhat.com> | 2008-01-02 17:26:13 -0500 |
commit | aa4b541b181eb76ff1b40c686d6e8e7193a33e53 (patch) | |
tree | 9464d792d4df24d37be30f5a19c7d79e65cfd9c2 | |
parent | 8aff1a6def091d3b269d1444cd21c3c8867ccd04 (diff) | |
download | python-bugzilla-aa4b541b181eb76ff1b40c686d6e8e7193a33e53.tar.gz python-bugzilla-aa4b541b181eb76ff1b40c686d6e8e7193a33e53.tar.xz python-bugzilla-aa4b541b181eb76ff1b40c686d6e8e7193a33e53.zip |
add some comments and refactor setassignee a little bit
-rw-r--r-- | bugzilla.py | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/bugzilla.py b/bugzilla.py index 8eaccd5..27fdc4e 100644 --- a/bugzilla.py +++ b/bugzilla.py @@ -711,6 +711,9 @@ class Bug(object): def __str__(self): '''Return a simple string representation of this bug''' + # XXX Not really sure why we get short_desc sometimes and + # short_short_desc other times. I feel like I'm working around + # a bug here, so keep an eye on this. if 'short_short_desc' in self.__dict__: desc = self.short_short_desc else: @@ -750,8 +753,10 @@ class Bug(object): # XXX is ValueError the right thing to throw here? raise ValueError, "You must set one of assigned_to, reporter, or qa_contact" # empty fields are ignored, so it's OK to send 'em - return self.bugzilla._setassignee(self.bug_id,assigned_to=assigned_to, + r = self.bugzilla._setassignee(self.bug_id,assigned_to=assigned_to, reporter=reporter,qa_contact=qa_contact,comment=comment) + # FIXME reload bug data here + return r def addcomment(self,comment,private=False,timestamp='',worktime='',bz_gid=''): '''Add the given comment to this bug. Set private to True to mark this @@ -788,6 +793,8 @@ class Bug(object): (append,prepend,overwrite) with the given text on the given whiteboard for the given bug.''' self.bugzilla._updatewhiteboard(self.bug_id,text,which,action) + # FIXME reload bug data here + def getwhiteboard(self,which='status'): '''Get the current value of the whiteboard specified by 'which'. Known whiteboard names: 'status','internal','devel','qa'. |