Bug 10148

Summary: 007 not filling in with existing values
Product: Koha Reporter: Nicole C. Engard <nengard>
Component: CatalogingAssignee: Mark Tompsett <mtompset>
Status: CLOSED FIXED QA Contact:
Severity: major    
Priority: P5 - low CC: elibris.helpdesk, f.demians, josef.moravec, julian.maurice, katrin.fischer, kyle, m.de.rooy, marjorie.barry-vila, mtompset, ztajoli
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=16856
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17072
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 10148: Marc21 field 007 builder improper reload values
Bug 10148: Marc21 field 007 builder improper reload values
[PASSED QA] Bug 10148: Marc21 field 007 builder improper reload values

Description Nicole C. Engard 2013-04-29 14:40:13 UTC
When you use the 007 builder to view the existing values in the 007 tag, Koha is not pulling these existing values into the builder window. Instead, Koha appears to be using defaults.


For example:

1. Open a record in the editor

2. The existing fixed field values in the 007 tag are: vd ||  ||

3. Using the tag editor, open the 007 tag builder window and you will see that Koha is not dsiplaying the existing values.

4.If you click the OK button Koha enters the following values: vc aa  ak


The problem also exists in the 006 tag as well.
Comment 1 Katrin Fischer 2015-01-06 20:06:51 UTC
Still valid.
Comment 2 Zeno Tajoli 2015-07-16 11:22:42 UTC
The bigger problem is on 007 for Electronic resource and Microform.
The plugin can't handle existing values when positions 06,07,08 must to be worked together.
Comment 3 Mark Tompsett 2016-07-05 20:20:01 UTC
The issue is in the template file. Anything that is 9595 lines long has got to have something wrong with it.
Comment 4 Mark Tompsett 2016-07-05 20:45:13 UTC Comment hidden (obsolete)
Comment 5 Mark Tompsett 2016-07-05 20:49:57 UTC
I can't duplicate the problem for 006.
Comment 6 Nick Clemens (kidclamp) 2016-08-06 18:34:45 UTC Comment hidden (obsolete)
Comment 7 Katrin Fischer 2016-08-07 09:12:25 UTC
This is a bad bug in my eyes as it's easy to lose data this way. Patch fixes 007, we still need a fix for 006. 
I am splitting 006 out into a separate bug 17072 so this one can move on.
Comment 8 Katrin Fischer 2016-08-07 09:15:35 UTC
Created attachment 54103 [details] [review]
[PASSED QA] Bug 10148: Marc21 field 007 builder improper reload values

The more correct solution is fix that template file.
However, in the mean time, this works.

TEST PLAN
---------
 1) find a record
 2) edit record
 3) click value builder for 007
 4) change everything to pipes as much as possible.
    (use Motion Picture to get all 00-22 values)
 5) save
    -- should save just fine.
 6) click the value builder again
    -- OOPS! Bad reload.
 7) prove t/db_dependent/FrameworkPlugin.t
    -- NOISY 007 messages.
 8) apply patch
 9) click the value builder again
    -- good reload
10) prove t/db_dependent/FrameworkPlugin.t
    -- No noise related to 007.
11) run koha qa test tools.

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 9 Kyle M Hall 2016-08-10 13:22:52 UTC
Pushed to master for 16.11, thanks Mark!
Comment 10 Frédéric Demians 2016-08-23 09:35:18 UTC
Pushed in 16.05. Will be in 16.05.03.
Comment 11 Julian Maurice 2016-08-23 15:28:42 UTC
Pushed to 3.22.x, will be in 3.22.10