Bug #61809 for HTML-Tree: allow XHTML's '/' as valid attribute name

Fri Oct 01 05:15:32 2010 tokuhirom+cpan [...] gmail.com - Ticket created

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.

Tue Oct 12 08:31:03 2010 Jeff.Fearn [...] gmail.com - Taken

Tue Oct 12 08:33:11 2010 Jeff.Fearn [...] gmail.com - Correspondence added

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.

Tue Oct 12 08:33:12 2010 The RT System itself - Status changed from 'new' to 'open'

Tue Oct 12 08:33:12 2010 Jeff.Fearn [...] gmail.com - Status changed from 'open' to 'patched'

Tue Oct 26 04:04:46 2010 Jeff.Fearn [...] gmail.com - Fixed in 4.1 added

Tue Oct 26 04:04:46 2010 Jeff.Fearn [...] gmail.com - Correspondence added

Version 4.1 shipped with a fix for this bug.

Tue Oct 26 04:04:47 2010 The RT System itself - Status changed from 'patched' to 'open'

Tue Oct 26 04:04:47 2010 Jeff.Fearn [...] gmail.com - Status changed from 'open' to 'resolved'