Bug 17127: Remove notes from detail.tt to only use MARCNOTES
authorFlorian Bontemps <florian.bontemps@biblibre.com>
Wed, 27 Oct 2021 12:50:42 +0000 (12:50 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 11 Jan 2022 22:38:35 +0000 (12:38 -1000)
commitf004ad000d288b6288c4cf131e61ba569ac9fba6
tree2432e2c7de1c0a7b631a5a2386476508aa7455c4
parentc780d61f90a385026f2b442767035211705fe997
Bug 17127: Remove notes from detail.tt to only use MARCNOTES

Notes come from the biblio table, but that table is already used to fill in MARCNOTES through Biblio.pm and get_marc_notes.
Get_marc_notes does check for NotesToHide and already read every note on the records, but .notes doesn't go through that same filter.
I don't see the point of keeping notes as a condition when MARCNOTES does the same job but better.

To test:
    1) Take any record, or create one
    2) Input something in the 500 field (or 300 in UNIMARC)
    3) In Systempreferences -> NotesToHide, fill in the number 500 (or 300).
    4) Save, then go look at the record detail in the OPAC and admin website.
    5) You should still see the 500 or 300 field under the Description tab.
    6) Apply patch.
    7) Reload the record detail page.
    8) Observe the error is gone.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/detail.tt
koha-tmpl/opac-tmpl/bootstrap/en/modules/opac-detail.tt