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

Bug#516737: No popcon in PTS for packages with special chars



tags 516737 patch
thanks

Escaping special chars is just fine in PTS's XSL, but because PTS page
links to http://qa.debian.org/developer.php?popcon=$PACKAGE which
returns HTTP level redirection (where it loses escaping) to
http://qa.debian.org/popcon.php?package=$PACKAGE, this escaping is lost.

PTS should be fixed by giving direct URL to popcon:

Index: www/xsl/pts.xsl
===================================================================
--- www/xsl/pts.xsl	(revision 2220)
+++ www/xsl/pts.xsl	(working copy)
@@ -930,7 +930,7 @@
 	  </li>
 	</xsl:if>
 	<li>
-          <a href="http://qa.debian.org/developer.php?popcon={$escaped-package}";>Popcon stats</a>
+          <a href="http://qa.debian.org/popcon.php?package={$escaped-package}";>Popcon stats</a>
 	</li>
         <xsl:if test="$other/@svnbuildstat='yes'">
           <li>
Also developer.php (which does not seem to be part of PTS) can be
fixed to preserve escaping of special chars in redirection pages.

-- 
Tommi Vainikainen

Reply to: