From 1f20255880a9334ccf23b54b866453781b8191e3 Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Thu, 5 Jun 2025 10:29:41 +0100 Subject: [PATCH] Bug 40079: Add ul/ol and li to allowed tags in notes profile This patch add the unordered list, ordered list and list item html tags to the list of allowed tags in C4::Scrubber. Test plan: 1) Add a course reserve and include the following in the public note field: 2) View the course details page and note "First item Second item" all appears on the same line (the ul and li tags have been stripped) 3) View the course reserves page and note the same issue in the public notes field of the table display 4) Apply this patch 5) Repeat 2 and 3 and note you now have a bullet pointed list displayed in both places 6) The display of the note on the details page may look mis-aligned.. that will be dealt with on it's own bug --- C4/Scrubber.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/C4/Scrubber.pm b/C4/Scrubber.pm index 855524e2b6d..996512417af 100644 --- a/C4/Scrubber.pm +++ b/C4/Scrubber.pm @@ -30,7 +30,7 @@ my %scrubbertypes = ( 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[ br b i em big small strong u hr span div p ]] }, + note => { allow => [qw[ br b i em big small strong u hr span div p ul li ]] }, staff => { default => [ 1 => { '*' => 1 } ], comment => 1, @@ -49,7 +49,6 @@ sub new { 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, process => 0, ); return $scrubber; -- 2.49.0