Bug 31007 - Can't save a supplement issue with mandatory subfields
Summary: Can't save a supplement issue with mandatory subfields
Status: RESOLVED DUPLICATE of bug 30658
Alias: None
Product: Koha
Classification: Unclassified
Component: Serials (show other bugs)
Version: master
Hardware: All All
: P5 - low major (vote)
Assignee: Johanna Räisä
QA Contact: Testopia
URL:
Keywords:
Depends on: 29496
Blocks:
  Show dependency treegraph
 
Reported: 2022-06-21 22:30 UTC by Fridolin Somers
Modified: 2022-06-25 21:24 UTC (History)
0 users

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 29496: (follow-up) fix for supplemental issues (2.60 KB, patch)
2022-06-21 22:44 UTC, Fridolin Somers
Details | Diff | Splinter Review
Bug 31007: Fix CheckMandatorySubfields with select fields for supplement issue (2.63 KB, patch)
2022-06-21 22:48 UTC, Fridolin Somers
Details | Diff | Splinter Review
Bug 31007: Fix CheckMandatorySubfields with select fields for supplement issue (2.69 KB, patch)
2022-06-21 22:52 UTC, Fridolin Somers
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Fridolin Somers 2022-06-21 22:30:48 UTC
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"
Comment 1 Fridolin Somers 2022-06-21 22:37:25 UTC
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">
Comment 2 Fridolin Somers 2022-06-21 22:44:50 UTC
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
Comment 3 Fridolin Somers 2022-06-21 22:48:27 UTC
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
Comment 4 Fridolin Somers 2022-06-21 22:52:28 UTC
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>
Comment 5 Katrin Fischer 2022-06-25 21:24:19 UTC
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 ***