Skip Menu |
 

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

Report information
The Basics
Id: 402
Status: open
Priority: 0/
Queue: HTML-Clean

People
Owner: Nobody in particular
Requestors: tjmather [...] tjmather.com
Cc:
AdminCc:

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



Subject: Removes newline after linktag
Download (untitled) / with headers
text/plain 756b
I'm using HTML::Clean and I find it useful for reducing the size of HTML downloads. However, I do have a small problem with it. If I have the following text: <a href="http://www.yahoo.com">Yahoo</a> <font size=-1>(link to yahoo)</font> It gets transformed to something like <a href="http://www.yahoo.com">Yahoo</a><font size=-1>(link to yahoo)</font> It gets displayed as Yahoo(link to yahoo) But instead I want it to be transformed to <a href="http://www.yahoo.com">Yahoo</a> <font size=-1>(link to yahoo)</font> With a *space* between the "Yahoo" link and the "(link to yahoo)" text. That is, Yahoo (link to yahoo) I was able to fix it easily so that it puts a space between tags. Please see the attached patch.
Download HTML-Clean.patch
text/x-diff 475b
--- Clean.pm.old Wed Jan 17 06:01:37 2001 +++ Clean.pm Wed Jan 17 06:01:45 2001 @@ -431,7 +431,7 @@ $$h =~ s,\n\s+<,\n<,sg; # space before tag $$h =~ s,\n\s+,\n ,sg; # other spaces - $$h =~ s,>\n\s*<,><,sg; # LF/spaces between tags.. + $$h =~ s,>\n\s*<,> <,sg; # LF/spaces between tags.. # Remove excess spaces within tags.. note, we could parse out the elements # and rewrite for excess spaces between elements. perhaps next version.
Download (untitled) / with headers
text/plain 206b
I came on here to report a similar bug only to find it was reported 10 *YEARS* ago and is still outstanding. Can this either be fixed or can the module be taken over by someone who will support it, please?


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.