]> git.koha-community.org Git - koha.git/commit
Bug 37655: Basic editor needs to HTML-escape the bib record title used as a heading
authorPhil Ringnalda <phil@chetcolibrary.org>
Thu, 15 Aug 2024 22:41:18 +0000 (15:41 -0700)
committerLucas Gass <lucas@bywatersolutions.com>
Mon, 30 Sep 2024 15:32:11 +0000 (15:32 +0000)
commit0b1c2ba4b86f9aac615625f0456d81c2cf0ab4d7
treedeb652343afc45e1162946db316e028052277ff9
parent1493c9b56ea3ca0f0e7d841068d015f086399e1f
Bug 37655: Basic editor needs to HTML-escape the bib record title used as a heading

We stick the title of a bib record you are editing in the basic editor into
an <h1> without escaping any HTML it might contain. We should instead escape
it.

Test plan:

1. Without the patch, search for any record in the catalog and click Edit
   record (if you are in the advanced editor, switch to the basic one)
2. Tab 2, Field 245, Subfield a, paste <script>alert('boo ❤')</script><h2>
   at the end of the subfield
3. Save, then from the record detail page select Edit - Edit record
4. You will have gotten an alert(), and the entire form will be the size
   of an <h2>. That's ugly, so go back to the detail page.
5. Apply patch, restart_all
6. Edit - Edit record
7. Now you should not get an alert, the whole title inluding the <script>
   should display in italics, and the "(Record number nnn)" after it should
   not be italicized.

Signed-off-by: David Cook <dcook@prosentient.com.au>
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt