Bug 32857 - Fix cataloguing/value_builder/unimarc_field_127.pl
Summary: Fix cataloguing/value_builder/unimarc_field_127.pl
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: Main
Hardware: All All
: P5 - low trivial (vote)
Assignee: Julian Maurice
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks: 30975
  Show dependency treegraph
 
Reported: 2023-02-03 13:34 UTC by Julian Maurice
Modified: 2023-12-28 20:44 UTC (History)
2 users (show)

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


Attachments
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.pl (1.48 KB, patch)
2023-02-03 13:34 UTC, Julian Maurice
Details | Diff | Splinter Review
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.pl (1.52 KB, patch)
2023-02-17 10:16 UTC, Julian Maurice
Details | Diff | Splinter Review
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.pl (1.57 KB, patch)
2023-03-26 17:29 UTC, David Nind
Details | Diff | Splinter Review
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.pl (1.62 KB, patch)
2023-03-29 08:35 UTC, Nick Clemens
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Julian Maurice 2023-02-03 13:34:23 UTC
This is part of the refactoring happening in bug 30975
Comment 1 Julian Maurice 2023-02-03 13:34:55 UTC
Created attachment 146122 [details] [review]
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.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
Comment 2 Julian Maurice 2023-02-17 10:16:06 UTC
Created attachment 146793 [details] [review]
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.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
Comment 3 David Nind 2023-03-26 17:29:29 UTC
Created attachment 148725 [details] [review]
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.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>
Comment 4 David Nind 2023-03-26 17:36:34 UTC
How I tested
~~~~~~~~~~~~

1. Add the unimarc_field_127.pl plugin to the 127$a subfield (Duration) for the default UNIMARC bibliographic framework in KTD.
2. Edit a bibliographic record.
3. Select the tag editor for the subfield.
4. A form is displayed for entering the values for hours, minutes, and seconds.
5. This should work the same before and after the patch is applied.

Description
~~~~~~~~~~~

Without this cataloguing plugin, the duration is entered manually into the 127$a subfield (Duration). With this plugin, a form is displayed that lets you enter the values for hours, minutes, and seconds.
Comment 5 Nick Clemens 2023-03-29 08:35:31 UTC
Created attachment 148874 [details] [review]
Bug 32857: Fix cataloguing/value_builder/unimarc_field_127.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>
Comment 6 Tomás Cohen Arazi 2023-03-30 10:32:19 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 7 Matt Blenkinsop 2023-04-11 11:12:28 UTC
Nice work everyone!

Pushed to stable for 22.11.x