Bug 30159: Fix Check() to validate both mandatory and important fields
authorAdam Styles <adam.styles@esa.edu.au>
Tue, 22 Feb 2022 00:32:52 +0000 (11:32 +1100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Wed, 16 Mar 2022 08:30:50 +0000 (22:30 -1000)
commit0733ba2075badfca90477d0dbfa490f81e68f066
tree28414ddc508bf4adf6a69dc89446a841c4b07b50
parent0a6c8afd77b9d3204d949d3d2f0c95ede13f5990
Bug 30159: Fix Check() to validate both mandatory and important fields

This patch adds a check for both mandatory and important fields when
validating bibliographic records during cataloguing.

To test:

1. Go to Admin -> Biblio frameworks. View the MARC structure of your
   default framework.
2. Search for tag 082. Edit the subfields for this tag.
3. Check the 'important' checkbox for subfield a and Save.
4. Go to add or edit a new biblio record under Cataloguing.
5. Ensure the 082$a field is empty and hit Save.
6. Notice the confirmation pop-up is an unreadable display of HTML.
7. Click Cancel on the pop-up box.
8. Apply the patch and refresh the editor page.
9. Ensure the 082$a field is empty and hit Save.
10. Notice the confirmation pop-up is more helpful this time. Click
    Cancel on the pop-up box.
11. Notice the 082$a field is displayed at the top of the page as an
    important field that is missing information. Use the 'go to field'
    link and confirm this works as expected.
12. Leave 082$a empty and hit Save.
13. This time click OK on the confirmation pop-up. Confirm you are able
    to save your record as normal.

Sponsored-by: Education Services Australia SCIS
Signed-off-by: Owen Leonard <oleonard@myacpl.org>
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/cataloguing/addbiblio.tt