Bug 32321 - 006 field not correctly prepopulated in Advanced cataloging editor
Summary: 006 field not correctly prepopulated in Advanced cataloging editor
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Cataloging (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Nick Clemens
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks: 32325
  Show dependency treegraph
 
Reported: 2022-11-22 14:04 UTC by Nick Clemens
Modified: 2023-12-28 20:44 UTC (History)
6 users (show)

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


Attachments
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor (1.79 KB, patch)
2022-11-22 14:07 UTC, Nick Clemens
Details | Diff | Splinter Review
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor (1.85 KB, patch)
2022-11-22 17:42 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor (1.94 KB, patch)
2023-01-27 09:00 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Nick Clemens 2022-11-22 14:04:56 UTC
The 006 helper should load the correct values based on the first position of the field - this is not happening

To recreate:
1 - Edit a record in advanced cataloging editor with field helpers enabled
2 - Choose a material type, not BKS, from dropdown
3 - Set the first position to something
4 - Save the record
5 - 006 shows 'BKS'
Comment 1 Nick Clemens 2022-11-22 14:07:25 UTC
Created attachment 144154 [details] [review]
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor

To test"
1 - Edit a record in advanced cataloging editor with field helpers enabled
2 - Choose a material type, not BKS, from dropdown
3 - Set the first position to something
4 - Save the record
5 - 006 shows 'BKS'
6 - Apply patch
7 - Reload
8 - Record should show correct value
9 - Edit 006 and confirm the first position correctly determines material type
Comment 2 ByWater Sandboxes 2022-11-22 17:42:55 UTC
Created attachment 144172 [details] [review]
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor

To test"
1 - Edit a record in advanced cataloging editor with field helpers enabled
2 - Choose a material type, not BKS, from dropdown
3 - Set the first position to something
4 - Save the record
5 - 006 shows 'BKS'
6 - Apply patch
7 - Reload
8 - Record should show correct value
9 - Edit 006 and confirm the first position correctly determines material type

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Comment 3 Marcel de Rooy 2023-01-27 09:00:14 UTC
Created attachment 145719 [details] [review]
Bug 32321: Pre-populate material type for 006 in advanced cataloging editor

To test"
1 - Edit a record in advanced cataloging editor with field helpers enabled
2 - Choose a material type, not BKS, from dropdown
3 - Set the first position to something
4 - Save the record
5 - 006 shows 'BKS'
6 - Apply patch
7 - Reload
8 - Record should show correct value
9 - Edit 006 and confirm the first position correctly determines material type

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 4 Tomás Cohen Arazi 2023-01-27 18:21:39 UTC
Pushed to master for 23.05.

Nice work everyone, thanks!
Comment 5 Matt Blenkinsop 2023-01-31 14:16:07 UTC
Nice work everyone!

Pushed to stable for 22.11.x
Comment 6 Lucas Gass 2023-02-01 16:07:14 UTC
Backported to 22.05.x for upcoming 22.05.10
Comment 7 Arthur Suzuki 2023-02-16 15:03:53 UTC
applied to 21.11.x for 21.11.16
Comment 8 wainuiwitikapark 2023-03-15 01:40:40 UTC
Not backported to 21.05.x