From 0dea6038b065e125535fada719f7229a4a0d27a2 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 Content-Type: text/plain; charset=utf-8 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 Signed-off-by: Marcel de Rooy --- C4/Scrubber.pm | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/C4/Scrubber.pm b/C4/Scrubber.pm index f5201418e7..07561f3b50 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.39.5