urosm
/
ikiwiki
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

use HTML::Entities

Joey Hess 2008-11-17 14:27:11 -05:00
parent 43039d7d86
commit 181bdbe1a9
3 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
IkiWiki/Plugin/htmlbalance.pm
View File

@ -11,7 +11,7 @@ use warnings;
use strict;
use IkiWiki 2.00;
use HTML::TreeBuilder;
use XML::Atom::Util qw(encode_xml);
use HTML::Entities;
sub import { #{{{
hook(type => "getsetup", id => "htmlbalance", call => \&getsetup);
@ -39,7 +39,7 @@ sub sanitize (@) { #{{{
$node->delete();
}
else {
$ret .= encode_xml($node);
$ret .= encode_entities($node);
}
}
$tree->delete();

2
doc/plugins/htmlbalance/discussion.mdwn
View File

@ -2,6 +2,8 @@ Would it be possible to use [[!cpan HTML::Entities]] rather than
`XML::Atom::Util` for encoding entities? The former is already an ikiwiki
dependency (via [[!cpan HTML::Parser]]).
> Now switched to HTML::Entities --[[Joey]]
I also wonder if there's any benefit to using this plugin aside from with
aggregate. Perhaps a small one but aggregate seems like the main case..
wondering if it would be better to just have aggregate balanace the html

3
t/htmlbalance.t
View File

@ -5,10 +5,9 @@ use strict;
BEGIN {
eval q{
use HTML::TreeBuilder;
use XML::Atom::Util qw(encode_xml);
};
if ($@) {
eval q{use Test::More skip_all => "HTML::TreeBuilder or XML::Atom::Util not available"};
eval q{use Test::More skip_all => "HTML::TreeBuilder not available"};
}
else {
eval q{use Test::More tests => 7};