Bug 4263

Summary: repeatable subfields in items
Product: Koha Reporter: Nahuel Angelinetti <nahuel.angelinetti>
Component: CatalogingAssignee: Henri-Damien LAURENT <henridamien>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: critical    
Priority: PATCH-Sent (DO NOT USE) CC: chris, colin.campbell, nengard
Version: unspecified   
Hardware: PC   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on:    
Bug Blocks: 5574    
Attachments: Fix for bug 4263
Patch to restore sort order

Description Chris Cormack 2010-05-21 01:25:06 UTC


---- Reported by nahuel.angelinetti@biblibre.com 2010-02-25 13:53:58 ----

Repeatable subfields in items have a problem, it concatenate all in one subfield separated with pipes, instead of create REAL subfields.
Then, edit items that have repeated subfields are not managed correctly, they "disappear".



--- Bug imported by chris@bigballofwax.co.nz 2010-05-21 01:25 UTC  ---

This bug was previously known as _bug_ 4263 at http://bugs.koha.org/cgi-bin/bugzilla3/show_bug.cgi?id=4263

Actual time not defined. Setting to 0.0
Setting qa contact to the default for this product.
   This bug either had no qa contact or an invalid one.

Comment 1 Chris Cormack 2011-02-02 23:20:13 UTC
Created attachment 3054 [details] [review]
Fix for bug 4263
Comment 2 Chris Cormack 2011-02-03 03:35:23 UTC
Signed off by Chris N, pushed please test
Comment 3 Nicole C. Engard 2011-02-08 02:39:54 UTC
The patch for this bug breaks the sorting of the subfields on the add/edit items form.
Comment 4 Colin Campbell 2011-02-09 16:39:35 UTC
As Nicole points out definitely causes the dispaly sorting to break.

What repeatable subfields are we expecting in items?
Comment 5 Colin Campbell 2011-02-10 16:54:27 UTC
Created attachment 3123 [details] [review]
Patch to restore sort order

Restore the broken order of item fields
Comment 6 Nicole C. Engard 2011-02-10 20:17:47 UTC
Signed off on follow up.
Comment 7 Chris Cormack 2011-02-13 20:17:23 UTC
Follow up patches pushed, please test