This is part of the refactoring happening in bug 30975
Created attachment 146124 [details] [review] Bug 32859: Fix cataloguing/value_builder/unimarc_field_128b.pl Use event parameter This is part of the refactoring happening in bug 30975 Test plan: 1. Verify that the plugin continues to work as before
Created attachment 146795 [details] [review] Bug 32859: Fix cataloguing/value_builder/unimarc_field_128b.pl Use event parameter This is part of the refactoring happening in bug 30975 Test plan: 1. Verify that the plugin continues to work as before
Created attachment 148727 [details] [review] Bug 32859: Fix cataloguing/value_builder/unimarc_field_128b.pl Use event parameter This is part of the refactoring happening in bug 30975 Test plan: 1. Verify that the plugin continues to work as before Signed-off-by: David Nind <david@davidnind.com>
How I tested ~~~~~~~~~~~~ 1. Add the unimarc_field_128b.pl plugin to subfield 128$b for the default UNIMARC bibliographic framework in KTD. 2. Change 'Managed in tab' from 'Ignore' to 1'1'. 3. Edit a bibliographic record. 4. Select the tag editor for the subfield. 5. Options and values should be the same before and after the patch is applied. Description ~~~~~~~~~~~ This is a standard UNIMARC value builder for a subfield - it displays a form to select the values that make up the subfield. Note: Subfield 128$b is obsolete ([Obsolete] Instruments or Voices for Ensembles).
Created attachment 148877 [details] [review] Bug 32859: Fix cataloguing/value_builder/unimarc_field_128b.pl Use event parameter This is part of the refactoring happening in bug 30975 Test plan: 1. Verify that the plugin continues to work as before Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to stable for 22.11.x