Please strip also "]" (and others) in bug arguments

Bug #235597 reported by Daniel Hahler
2
Affects Status Importance Assigned to Milestone
five-a-day
Fix Released
Undecided
Unassigned

Bug Description

When double clicking the bug from the subject line in Thunderbird, you get e.g. "235207]", so please strip also "]" from the bug argument list on the command line.

Additionally, I think "[", "(" and ")" should get stripped, too.

I'm providing the patch inline, because it's so simple:
=== modified file 'fiveaday/files.py'
--- fiveaday/files.py 2008-05-27 08:49:53 +0000
+++ fiveaday/files.py 2008-05-28 20:18:14 +0000
@@ -105,7 +105,7 @@
         bugs = []
         for entry in entries:
             for i in entry.split(","):
- n = i.strip(" #")
+ n = i.strip(" #[]()")
                 if n:
                     try:
                         number = int(n)

Revision history for this message
Daniel Hahler (blueyed) wrote :
Revision history for this message
Daniel Holbach (dholbach) wrote :

 five-a-day (0.41) intrepid; urgency=low
 .
   * fiveaday/files.py: also strip "[]()" from bug arguments - thanks a lot
     Daniel Hahler <email address hidden> for the patch. (LP: #235597)
   * fiveaday/bzr.py: check for branch.last_revision() to avoid bzr crashing in
     corner cases. Thanks a lot Daniel Hahler <email address hidden> for the
     patch. (LP: #234890)

Changed in five-a-day:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.