From 9c9ebb23212c22b288ad7206ecf972dae2cecaa4 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 Signed-off-by: David Nind --- C4/Scrubber.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/C4/Scrubber.pm b/C4/Scrubber.pm index f5201418e74..07561f3b507 100644 --- a/C4/Scrubber.pm +++ b/C4/Scrubber.pm @@ -29,7 +29,7 @@ use C4::Context; my %scrubbertypes = ( default => {}, # place holder, default settings are below as fallbacks in call to constructor 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 ol ul li ]] }, ); sub new { @@ -44,7 +44,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