Skip Menu |
 

This queue is for tickets about the XML-RSS CPAN distribution.

Report information
The Basics
Id: 8866
Status: resolved
Priority: 0/
Queue: XML-RSS

People
Owner: Nobody in particular
Requestors: mmu [...] oce.nl
Cc:
AdminCc:

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



Subject: entity escaping makes error in specific cases eg. R&D;
Download (untitled) / with headers
text/plain 567b
Due to a logical error certain &'s are not escaped properly, for example in the string "R&D;" the ampersand is not escaped. I propose to fix this by changing the encode_text-sub to the following: sub encode_text { my $text = shift; $text =~ s/&($entities);/$entity{$1}/g; $text =~ s/&(?!(#[0-9]+|#x[0-9a-fA-F]+);)/&amp;/g; $text =~ s/</&lt;/g; return $text; } As you can see the known entities are replaced first now, after which only hexadecimal entities should exist and all the other ampersands can be escaped safely. Kind regards, Matthijs Mullender
This was fixed in 1.12 I believe.


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.