Similar to bug 31238. I checked the rest of the form, seems to just be this field To test: 1. Edit e.g. default frameworks subfield 952$8 2. Add or change fields thesaurus value 3. Save your changes => note that on subfield listing there is no mention that 952$8 is linked to any thesaurus => if you edit 952$8 thesaurus input is empty
Created attachment 142147 [details] [review] Bug 31869: Unable to save thesaurus value to frameworks subfields This patch correct a typo authtypecodes for authtypecode To test: 1. Edit e.g. default frameworks subfield 100$a 2. Add or change fields thesaurus value 3. Save your changes => note that on subfield listing there is no mention that 100$a is linked to any thesaurus => if you edit 100$a thesaurus input is empty 4. Apply patch 5. Edit subfield 100$a and set a thesaurus value 6. Confirm it saves and displays correctly Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Self SO, straight to QA - bad bug, easy fix
Created attachment 142244 [details] [review] Bug 31869: Unable to save thesaurus value to frameworks subfields This patch correct a typo authtypecodes for authtypecode To test: 1. Edit e.g. default frameworks subfield 100$a 2. Add or change fields thesaurus value 3. Save your changes => note that on subfield listing there is no mention that 100$a is linked to any thesaurus => if you edit 100$a thesaurus input is empty 4. Apply patch 5. Edit subfield 100$a and set a thesaurus value 6. Confirm it saves and displays correctly Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 142477 [details] [review] Bug 31869: Unable to save thesaurus value to frameworks subfields This patch correct a typo authtypecodes for authtypecode To test: 1. Edit e.g. default frameworks subfield 100$a 2. Add or change fields thesaurus value 3. Save your changes => note that on subfield listing there is no mention that 100$a is linked to any thesaurus => if you edit 100$a thesaurus input is empty 4. Apply patch 5. Edit subfield 100$a and set a thesaurus value 6. Confirm it saves and displays correctly Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Pushed to master for 22.11. Nice work everyone, thanks!
Backported to 22.05.x for upcoming 22.05.08
this one conflicts when trying to apply to 21.11. Plus dependencies not met for some bugs. Won't backport.