]> 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)
committerKatrin Fischer <katrin.fischer@bsz-bw.de>
Fri, 18 Oct 2024 10:07:45 +0000 (12:07 +0200)
commitea1d3f7a01fb6ed85c4c0d57c986a62e8d8fbb1c
tree8e523fd5ea697f529cfbdd1ab6323c30172ceb6c
parentbbd644444f0fb344922112dd447c18a4a86525d7
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>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
koha-tmpl/intranet-tmpl/prog/en/modules/cataloguing/addbiblio.tt