Bug 29496 fixed new issues mandatory subfields check. But there is still a problem with a supplement issue with mandatory subfields. I it nearly the same code missing class "input_marceditor"
Looks like it is the only one missing : > git grep '<select .*name="field_value"' koha-tmpl/intranet-tmpl/prog/en/modules/acqui/addorderiso2709.tt: <select name="field_value" class="input_marceditor"> koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt: <select name="field_value" class="select2 input_marceditor"> koha-tmpl/intranet-tmpl/prog/en/modules/serials/serials-edit.tt: <select name="field_value"> koha-tmpl/intranet-tmpl/prog/en/modules/services/itemrecorddisplay.tt: <select name="field_value" class="input_marceditor">
Created attachment 136405 [details] [review] Bug 29496: (follow-up) fix for supplemental issues The select field validation fails for supplemental issues in serials-edit.tt. Test plan inspired from Bug 29496 : Test plan: 0- Be sure to be in a version of koha where the patch that introduces the bug is present (it is present in master since Jul 8 2021 (it is present in 21.06.00.046) and will be pushed in 21.11.00) 1- Create (or find) a subscription for a biblio record and select the option "Create an item record when receiving this serial" 2- Be sure to have at least one mandatory subfield that is filed with a select input in the framework used by the biblio record. (ex: 995$b, 995$c or 995$e in unimarc; 952$a, 952$b or 952$c in marc21) 3- From the subscription-detail page click on "Receive" 4- Change the status to "Arrived" and fill the item form that appears. 5- Click on "Click to add item" below "Supplemental issue" and fill the item form that appears. 6- Click on "Save" 7- Check that an error box appear with the message " Form not submitted because of the following problem(s) 1 mandatory fields empty (highlighted)" (the number can be different according to the number of concerned subfields) 8- Apply the patch 9- Repeat step 3 to 6 10- Check that no error appear and that your item has been created
Created attachment 136406 [details] [review] Bug 31007: Fix CheckMandatorySubfields with select fields for supplement issue The select field validation fails for supplemental issues in serials-edit.tt. Test plan inspired from Bug 29496 : Test plan: 0- Be sure to be in a version of koha where the patch that introduces the bug is present (it is present in master since Jul 8 2021 (it is present in 21.06.00.046) and will be pushed in 21.11.00) 1- Create (or find) a subscription for a biblio record and select the option "Create an item record when receiving this serial" 2- Be sure to have at least one mandatory subfield that is filed with a select input in the framework used by the biblio record. (ex: 995$b, 995$c or 995$e in unimarc; 952$a, 952$b or 952$c in marc21) 3- From the subscription-detail page click on "Receive" 4- Change the status to "Arrived" and fill the item form that appears. 5- Click on "Click to add item" below "Supplemental issue" and fill the item form that appears. 6- Click on "Save" 7- Check that an error box appear with the message " Form not submitted because of the following problem(s) 1 mandatory fields empty (highlighted)" (the number can be different according to the number of concerned subfields) 8- Apply the patch 9- Repeat step 3 to 6 10- Check that no error appear and that your item has been created
Created attachment 136407 [details] [review] Bug 31007: Fix CheckMandatorySubfields with select fields for supplement issue The select field validation fails for supplemental issues in serials-edit.tt. Test plan inspired from Bug 29496 : Test plan: 0- Be sure to be in a version of koha where the patch that introduces the bug is present (it is present in master since Jul 8 2021 (it is present in 21.06.00.046) and will be pushed in 21.11.00) 1- Create (or find) a subscription for a biblio record and select the option "Create an item record when receiving this serial" 2- Be sure to have at least one mandatory subfield that is filed with a select input in the framework used by the biblio record. (ex: 995$b, 995$c or 995$e in unimarc; 952$a, 952$b or 952$c in marc21) 3- From the subscription-detail page click on "Receive" 4- Change the status to "Arrived" and fill the item form that appears. 5- Click on "Click to add item" below "Supplemental issue" and fill the item form that appears. 6- Click on "Save" 7- Check that an error box appear with the message " Form not submitted because of the following problem(s) 1 mandatory fields empty (highlighted)" (the number can be different according to the number of concerned subfields) 8- Apply the patch 9- Repeat step 3 to 6 10- Check that no error appear and that your item has been created https://bugs.koha-community.org/show_bug.cgi?id=31007 Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
I knew this one looked familiar, now I found out why, I had Signed Off the exact same patch a little earlier: bug 30658. Which one to pick? I can sign off here... but Janusz was a little earlier. *** This bug has been marked as a duplicate of bug 30658 ***