[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#433894: qa.debian.org: "Updating foo introduces new bugs:" links only to the first link



Package: qa.debian.org
Severity: minor
Tags: patch

When a package introduces new bugs to testing, the "Updating foo
introduces new bugs:" links only link to one bug of the whole lot.

An example is here:

http://packages.qa.debian.org/p/php4-ps.html

I've attached a quick and dirty and _untested_ patch that may work. I'm
not sure it is the right approach, making all lines with links in them
do something similar might be a better way to go so that all links in
the update_excuses.html file are preserved on the PTS page. If that is
wanted I can rework it.

I disclaim all copyright on this 11 line patch :)

-- 
bye,
pabs

http://wiki.debian.org/PaulWise
--- excuses_to_xml.py~	2007-07-20 07:09:44.000000000 +0000
+++ excuses_to_xml.py	2007-07-20 07:39:02.000000000 +0000
@@ -99,6 +99,17 @@
                 sub_elt = doc.createElement("item")
                 sub_elt.appendChild(doc.createTextNode(subline))
                 main_elt.appendChild(sub_elt) 
+            elif subline.find("introduces new bugs:") != -1:
+                subline = striphtml(subline)
+                subline = re.split('(#[0-9]+)', subline)
+                for item in subline:
+                    if len(item) and item[0] == '#':
+                        sub_elt = doc.createElement("item")
+                        sub_elt.setAttribute("url", "http://bugs.debian.org/"; + item[1:])
+	                sub_elt.appendChild(doc.createTextNode(item))
+                    else:
+                        sub_elt = doc.createTextNode(item)
+                    main_elt.appendChild(sub_elt) 
             else:
                 sub_elt = doc.createElement("item")
                 start = subline.find('href="http://')

Attachment: signature.asc
Description: This is a digitally signed message part


Reply to: