Summary: | Unable to save authorised value to frameworks subfields | ||
---|---|---|---|
Product: | Koha | Reporter: | Emmi Takkinen <emmi.takkinen> |
Component: | MARC Bibliographic data support | Assignee: | Emmi Takkinen <emmi.takkinen> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | major | ||
Priority: | P5 - low | CC: | arthur.suzuki, e.betemps, lucas, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
22.11.00, 22.05.06
|
|
Circulation function: | |||
Bug Depends on: | 29001 | ||
Bug Blocks: | 31869 | ||
Attachments: |
Bug 31238: Fix typo in marc_subfields_structure.pl
Bug 31238: Fix typo in marc_subfields_structure.pl Bug 31238: Fix typo in marc_subfields_structure.pl |
Description
Emmi Takkinen
2022-07-26 08:49:17 UTC
There seems to be a minor typo on marc_subfields_structure.pl, >$input->param("authorised_values_$id") should be >$input->param("authorised_value_$id") Created attachment 138119 [details] [review] Bug 31238: Fix typo in marc_subfields_structure.pl After bug 29001 one is unable to save authorised value to frameworks subfields. This is caused by minor typo in marc_subfields_structure.pl. To test: 1. Edit e.g. default frameworks subfield 952$8 2. Add or change fields authorised value 3. Save your changes => note that on subfield listing there is no mention that 952$8 is linked to any authorised value => if you edit 952$8 authorised value input is empty 4. Apply this patch. 5. Repeat steps 1.-3. => authorised value should now be saved Sponsored-by: Koha-Suomi Oy Created attachment 138125 [details] [review] Bug 31238: Fix typo in marc_subfields_structure.pl After bug 29001 one is unable to save authorised value to frameworks subfields. This is caused by minor typo in marc_subfields_structure.pl. To test: 1. Edit e.g. default frameworks subfield 952$8 2. Add or change fields authorised value 3. Save your changes => note that on subfield listing there is no mention that 952$8 is linked to any authorised value => if you edit 952$8 authorised value input is empty 4. Apply this patch. 5. Repeat steps 1.-3. => authorised value should now be saved Sponsored-by: Koha-Suomi Oy Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Created attachment 138231 [details] [review] Bug 31238: Fix typo in marc_subfields_structure.pl After bug 29001 one is unable to save authorised value to frameworks subfields. This is caused by minor typo in marc_subfields_structure.pl. To test: 1. Edit e.g. default frameworks subfield 952$8 2. Add or change fields authorised value 3. Save your changes => note that on subfield listing there is no mention that 952$8 is linked to any authorised value => if you edit 952$8 authorised value input is empty 4. Apply this patch. 5. Repeat steps 1.-3. => authorised value should now be saved Sponsored-by: Koha-Suomi Oy Signed-off-by: Emmanuel Bétemps <e.betemps@gmail.com> Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Works, QA script happy, code looks good, passing QA :) Severity upped to major. Feel free to change if that's too much. Surely that's at least normal severity. Pushed to master for 22.11. Nice work everyone, thanks! Lucas, can you please check if this is needed in 22.05 (see comment on bug 29001? Backported to 22.05.x for 22.05.06 I don't reproduce the issue on 21.11.x branch. Also bz29001 was not applied there so maybe that one is not needed for backport. Won't backport unless asked to. |