Skip Menu |
 

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

Report information
The Basics
Id: 38398
Status: rejected
Priority: 0/
Queue: HTML-Tree

People
Owner: Nobody in particular
Requestors: dimaniac83 [...] mail.ru
Cc:
AdminCc:

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



Subject: TreeBuilder can broke tree with correct nesting
Download (untitled) / with headers
text/plain 1.2k
first, sorry for my English :p My problem is: if we have HTML with correct nesting, but wrong by DTD policy, parsing that code without implicit_tags=0 can broke initial nesting. as example, i have HTML (correct as just a tree): <div id="some"> <div> <font> <div> <h1> <font> <p>!</p> </font> </h1> </div> </font> </div> 111 <table> <tr> <td>!!</td> </tr> </table> </div> parse it with implicit_tags ON, look_down tree for id='some' and output via as_HTML. What i get: <div id="some"><div><font><div><h1><font> </font></h1><p>!</p></div></ Show quoted text
font></div></div>
Because, as i understood, block element 'p' cant be placed under phrase element 'font'. So, 'font' force closed, and corresponding '/font' closing tag close 'div' and another, first 'font'. And table throwed out from div#some. Of course i can set implicit_tags OFF, but it make a lot of useful (and safe) work, such as table tags handling. Locally, i added implicit_tags=2 setting, which mean is kind of 'ON, but without tree nesting altering'. Maybe this should be added to next release?
Download (untitled) / with headers
text/plain 452b
On Mon, Aug 11, 2008 6:58:15 AM, skaurus wrote: Show quoted text
> Locally, i added implicit_tags=2 setting, which mean is kind of 'ON, > but without tree nesting altering'. Maybe this should be added to next > release?
If you're still interested in having this added to HTML-Tree, please fork my repo on GitHub (https://github.com/madsen/HTML-Tree) and send a pull request with your code. We'll look at it. Otherwise, I'm going to be closing this ticket. Thanks.
Download (untitled) / with headers
text/plain 160b
Since there's been no response, I'm closing this request. If you want this code incorporated, please send a pull request at https://github.com/madsen/HTML-Tree


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.