Skip Menu |
 

This queue is for tickets about the HTML-Tree CPAN distribution.

Report information
The Basics
Id: 61809
Status: resolved
Priority: 0/
Queue: HTML-Tree

People
Owner: Jeff.Fearn [...] gmail.com
Requestors: tokuhirom+cpan [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 4.0
Fixed in: 4.1



Subject: allow XHTML's '/' as valid attribute name
Download (untitled) / with headers
text/plain 366b
XHTML's '/' is valid attribute name. This validation breaks web contents scrapers for XHTML. It was broken from: http://github.com/jfearn/HTML- Tree/commit/029b3569c198a8ba96834f62bd37675c32b82c4b please see this commit on github. http://github.com/tokuhirom/HTML- Tree/commit/6c40ccbcc5eae9dac6fe005b30c6bc1a8e7e8f47 I sent pull request to jfearn, few days ago.
Download (untitled) / with headers
text/plain 758b
On Fri Oct 01 19:15:32 2010, TOKUHIROM wrote: Show quoted text
> XHTML's '/' is valid attribute name. > This validation breaks web contents scrapers for XHTML. > > It was broken from: http://github.com/jfearn/HTML- > Tree/commit/029b3569c198a8ba96834f62bd37675c32b82c4b > > please see this commit on github. > http://github.com/tokuhirom/HTML- > Tree/commit/6c40ccbcc5eae9dac6fe005b30c6bc1a8e7e8f47 > > I sent pull request to jfearn, few days ago.
hmm I think my mail from CPAN is a little odd, I got the pull request from githum but could swear I never got an email about the RT ticket O_O I pulled in this fix. http://github.com/jfearn/HTML-Tree/commit/6c40ccbcc5eae9dac6fe005b30c6bc1a8e7e8f47 Thanks for the report and double thanks for the patch :) Cheers, Jeff.
Version 4.1 shipped with a fix for this bug.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.