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

Report information
The Basics
Id:
61809
Status:
resolved
Priority:
Low/Low
Queue:

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

BugTracker
Severity:
(no value)
Broken in:
4.0
Fixed in:
4.1



Subject: allow XHTML's '/' as valid attribute name
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.
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 runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

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