Skip Menu |

This queue is for tickets about the SVN-Web CPAN distribution.

Report information
The Basics
Id: 33905
Status: resolved
Priority: 0/
Queue: SVN-Web

People
Owner: Nobody in particular
Requestors: DAVECROSS [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.53
Fixed in: (no value)



Subject: Generating Invalid HTML
I'm getting failures in the HTML validity tests in t/2basic.t. For example: # Failed test ' and is valid HTML' # in t/2basic.t at line 65. # Messages: and is valid HTML t/2basic.............NOK 198# t/2basic.t (54:33) Warning: <a> escaping malformed URI reference # t/2basic.t (55:33) Warning: <a> escaping malformed URI reference # t/2basic.t (55:120) Warning: <a> escaping malformed URI reference # t/2basic.t (55:163) Warning: <a> escaping malformed URI reference [ snip until line 54 ] # <div class="add"></div> <a href="http://localhost/svnweb/repos/browse/E F?rev=3">/E F</a> </li> <li> # <div class="add"></div> <a href="http://localhost/svnweb/repos/view/E F/test file?rev=3">/E F/test file</a> (<a href="#diff__E F_test file">Diff</a>) (<a href="http://localhost/svnweb/repos/checkout/E F/test file?rev=3">Checkout</a>) </li> </ul> The problem is the spaces in the URLs (e.g. repos/view/E F/) which should be encoded as %20 or '+'. Not sure if this has been picked up because I've just installed a newer version of tidy. $ rpm -qa | grep tidy libtidy-0.99.0-16.20070615.fc8 tidy-0.99.0-16.20070615.fc8 libtidy-devel-0.99.0-16.20070615.fc8 This is Perl 5.8.8 on Fedora 8 (kernel 2.6.23.15-137.fc8).
Thanks for the report Dave. Fixed in http://jc.ngo.org.uk:80/svnweb/jc/revision/?rev=1307