[Reportbug-commits] [SCM] Reportbug - reports bugs in the Debian distribution branch, master, updated. 6.0-52-ged954c2
Sandro Tosi
morph at debian.org
Sat Sep 10 21:15:07 UTC 2011
The following commit has been merged in the master branch:
commit ad03768811d13373aa67ec48fccc0f9a3606a550
Author: Sandro Tosi <morph at debian.org>
Date: Sat Sep 10 22:55:39 2011 +0200
fix a crash after the bugs list, when filtering the result set, viewing a report and then wanting to follow it up; thanks to Michal Suchanek for the report; Closes: #640518
The problem is that search_bugs() can return either a string or a Bugreport
instance, so we need to check the return value type.
diff --git a/debian/changelog b/debian/changelog
index 3783420..55ac59a 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -11,8 +11,11 @@ reportbug (6.3) UNRELEASED; urgency=low
* reportbug/ui/text_ui.py
- fix a crash after the bugs list, when selecting 'y' and then entering the
item number; thanks to Jakub Wilki for the report; Closes: #640196
+ - fix a crash after the bugs list, when filtering the result set, viewing a
+ report and then wanting to follow it up; thanks to Michal Suchanek for the
+ report; Closes: #640518
- -- Sandro Tosi <morph at debian.org> Sat, 10 Sep 2011 22:49:51 +0200
+ -- Sandro Tosi <morph at debian.org> Sat, 10 Sep 2011 22:54:04 +0200
reportbug (6.2) unstable; urgency=low
diff --git a/reportbug/ui/text_ui.py b/reportbug/ui/text_ui.py
index 35c34b8..65570bd 100644
--- a/reportbug/ui/text_ui.py
+++ b/reportbug/ui/text_ui.py
@@ -704,7 +704,8 @@ def browse_bugs(hierarchy, count, bugs, bts, queryonly, mirrors,
retval = search_bugs(hierarchy,bts, queryonly, mirrors,
http_proxy, timeout, screen, title,
package, mbox_reader_cmd)
- if retval in ["FilterEnd", "Top"]:
+ if isinstance(retval, basestring) and \
+ retval in ["FilterEnd", "Top"]:
continue
else:
return retval
--
Reportbug - reports bugs in the Debian distribution
More information about the Reportbug-commits
mailing list