[Apt-listbugs-commits] [SCM] apt-listbugs development tree branch, master, updated. 0.1.0-5-gf8a888c

[Ryan Niebur]

The following commit has been merged in the master branch:
commit 0ca67de284c11926221959a0ee0237dd0c3bd84d
Author: Francesco Poli (t1000) <frx at firenze.linux.it>
Date:   Mon Aug 24 23:48:53 2009 +0200

    drop '\r' characters from bug subjects
    
    Sometimes bug reports have '\r' characters in their subjects, causing
    weird and incorrect display of bug descriptions in apt-listbugs:
    dropping these spurious characters from bug subjects in the BTS SOAP
    reply parsing code fixes this issue.

diff --git a/debian/changelog b/debian/changelog
index 2b93fe5..74b2ae8 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,8 +1,8 @@
 apt-listbugs (0.1.1) UNRELEASED; urgency=low
 
-  * 
+  * enhance BTS SOAP reply parsing: drop '\r' characters from bug subjects
 
- -- Francesco Poli (t1000) <frx at firenze.linux.it>  Thu, 06 Aug 2009 00:22:41 +0200
+ -- Francesco Poli (t1000) <frx at firenze.linux.it>  Sun, 23 Aug 2009 23:18:37 +0200
 
 apt-listbugs (0.1.0) unstable; urgency=low
 
diff --git a/lib/debian/btssoap.rb b/lib/debian/btssoap.rb
index bb6cf07..16ac4be 100644
--- a/lib/debian/btssoap.rb
+++ b/lib/debian/btssoap.rb
@@ -1,5 +1,6 @@
 # btssoap.rb - ruby interface for Debian BTS SOAP engine
-# Copyright (c) 2006-2008 Junichi Uekawa
+# Copyright (c) 2006-2008 Junichi Uekawa <dancer at debian.org>
+# Copyright (c) 2009      Francesco Poli <frx at firenze.linux.it>
 #
 # This program is free software; you can redistribute it and/or modify
 # it under the terms of the GNU General Public License as published by
@@ -64,7 +65,7 @@ module Debian
                                      bugnum.to_s, 
                                      res.severity,
                                      res.pending, 
-                                     res.subject,
+                                     res.subject.gsub(/\r/,''),
                                      res.tags.split(" "),
                                      res.mergedwith.to_s.split(" "),
                                      Time::at(res.date.to_i))

-- 
apt-listbugs development tree