Bug 36328 - C4::Scrubber should allow more html tags
Summary: C4::Scrubber should allow more html tags
Status: Passed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 19613
  Show dependency treegraph
 
Reported: 2024-03-15 10:38 UTC by Jonathan Druart
Modified: 2024-04-01 16:11 UTC (History)
2 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 36328: Add p span div to Scrubber (909 bytes, patch)
2024-03-15 10:41 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 36328: Add test (812 bytes, patch)
2024-03-15 10:41 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 36328: Add a separate 'note' profile (1.79 KB, patch)
2024-03-20 07:34 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 36328: Add p span div to Scrubber (962 bytes, patch)
2024-03-21 00:42 UTC, David Cook
Details | Diff | Splinter Review
Bug 36328: Add test (865 bytes, patch)
2024-03-21 00:42 UTC, David Cook
Details | Diff | Splinter Review
Bug 36328: Add a separate 'note' profile (1.85 KB, patch)
2024-03-21 00:42 UTC, David Cook
Details | Diff | Splinter Review
Bug 36328: Add p span div to Scrubber (1018 bytes, patch)
2024-04-01 16:11 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 36328: Add test (921 bytes, patch)
2024-04-01 16:11 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 36328: Add a separate 'note' profile (1.90 KB, patch)
2024-04-01 16:11 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 36328: (QA follow-up) Expand tests and reorder elements to clarify differences (3.05 KB, patch)
2024-04-01 16:11 UTC, Nick Clemens
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2024-03-15 10:38:59 UTC
Should not we allow span, div, etc. as well as existing "br b i em big small strong"?
Comment 1 Jonathan Druart 2024-03-15 10:41:33 UTC
Created attachment 163192 [details] [review]
Bug 36328: Add p span div to Scrubber
Comment 2 Jonathan Druart 2024-03-15 10:41:36 UTC
Created attachment 163193 [details] [review]
Bug 36328: Add test
Comment 3 David Cook 2024-03-17 23:35:57 UTC
I think it probably makes sense to have a few different profiles. 

For the "comment" scrubber, we probably want to keep it fairly minimal I think.

Whereas "borrowernotes" and "opacnote" could have more elaborate structures. 

Bug 31934 has a profile with a fairly comprehensive list (although I wouldn't use it for "borrowernotes" or "opacnote"). 

I think an argument could be made for also including headings, table elements, fieldset/legend, ul/ol/li.
Comment 4 Martin Renvoize 2024-03-19 12:58:00 UTC
We could really do with getting bug 31934 moving again I think..
Comment 5 Jonathan Druart 2024-03-20 07:34:23 UTC
Created attachment 163497 [details] [review]
Bug 36328: Add a separate 'note' profile
Comment 6 Jonathan Druart 2024-03-20 07:34:50 UTC
Would this patch work for you? Here I am trying to tackle down bug 19613.
Comment 7 David Cook 2024-03-21 00:10:03 UTC
(In reply to Jonathan Druart from comment #6)
> Would this patch work for you? Here I am trying to tackle down bug 19613.

Looks good. I'll look at signing this off pronto...
Comment 8 David Cook 2024-03-21 00:42:35 UTC
Created attachment 163582 [details] [review]
Bug 36328: Add p span div to Scrubber

Signed-off-by: David Cook <dcook@prosentient.com.au>
Comment 9 David Cook 2024-03-21 00:42:37 UTC
Created attachment 163583 [details] [review]
Bug 36328: Add test

Signed-off-by: David Cook <dcook@prosentient.com.au>
Comment 10 David Cook 2024-03-21 00:42:40 UTC
Created attachment 163584 [details] [review]
Bug 36328: Add a separate 'note' profile

Signed-off-by: David Cook <dcook@prosentient.com.au>
Comment 11 Nick Clemens 2024-04-01 16:11:39 UTC
Created attachment 164206 [details] [review]
Bug 36328: Add p span div to Scrubber

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 12 Nick Clemens 2024-04-01 16:11:41 UTC
Created attachment 164207 [details] [review]
Bug 36328: Add test

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 13 Nick Clemens 2024-04-01 16:11:43 UTC
Created attachment 164208 [details] [review]
Bug 36328: Add a separate 'note' profile

Signed-off-by: David Cook <dcook@prosentient.com.au>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Comment 14 Nick Clemens 2024-04-01 16:11:45 UTC
Created attachment 164209 [details] [review]
Bug 36328: (QA follow-up) Expand tests and reorder elements to clarify differences

Also tidy

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>