diff --git a/C4/Scrubber.pm b/C4/Scrubber.pm index e265c8dcc0..ccfa830585 100644 --- a/C4/Scrubber.pm +++ b/C4/Scrubber.pm @@ -27,10 +27,10 @@ use HTML::Scrubber; use C4::Context; my %scrubbertypes = ( - default => {}, # place holder, default settings are below as fallbacks in call to constructor - tag => {}, # uses defaults + default => {}, # place holder, default settings are below as fallbacks in call to constructor + tag => {}, # uses defaults comment => { allow => [qw( br b i em big small strong )], }, - note => { allow => [qw[ p b i u hr br em big small strong span div ]] }, + note => { allow => [qw[ br b i em big small strong u hr span div p ]] }, staff => { default => [ 1 => { '*' => 1 } ], comment => 1, @@ -46,11 +46,11 @@ sub new { } my $settings = $scrubbertypes{$type}; my $scrubber = HTML::Scrubber->new( - allow => exists $settings->{allow} ? $settings->{allow} : [], - rules => exists $settings->{rules} ? $settings->{rules} : [], + allow => exists $settings->{allow} ? $settings->{allow} : [], + rules => exists $settings->{rules} ? $settings->{rules} : [], default => exists $settings->{default} ? $settings->{default} : [ 0 => { '*' => 0 } ], comment => exists $settings->{comment} ? $settings->{comment} : 0, - note => exists $settings->{note} ? $settings->{note} : 0, + note => exists $settings->{note} ? $settings->{note} : 0, process => 0, ); return $scrubber; diff --git a/t/Scrubber.t b/t/Scrubber.t index 3a43522ad0..e04fd2e0d9 100755 --- a/t/Scrubber.t +++ b/t/Scrubber.t @@ -3,7 +3,7 @@ use Modern::Perl; $| = 1; -use Test::More tests => 30; +use Test::More tests => 31; use Test::Warn; BEGIN { @@ -82,4 +82,12 @@ if ($@) { pass("Test should have failed on entry of 'Client' and it did. YAY!"); } -is( C4::Scrubber->new('note')->scrub('Allow span'), 'Allow span' ); +my $scrub_text = + '
bolditalemphaticembiggenshrinkstrongbad
under