[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=21800 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Fridolin SOMERS changed: What|Removed |Added Status|Pushed to Stable|RESOLVED Resolution|--- |FIXED --- Comment #34 from Fridolin SOMERS --- Depends on Bug 10306 not in 17.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #33 from Lucas Gass --- reverted and reapplied cleaner merge -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #32 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #31) > Nope, see > commit d6fe41657861666320383904b16165cc4820777a > Bug 21774: Cloned item subfields disappear when editing an item > > Actually my understanding of the problem was wrong, it's more a bad merge > conflict. Ah okay. Very messy merge.. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #31 from Jonathan Druart --- Nope, see commit d6fe41657861666320383904b16165cc4820777a Bug 21774: Cloned item subfields disappear when editing an item Actually my understanding of the problem was wrong, it's more a bad merge conflict. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #30 from Marcel de Rooy --- (In reply to Jonathan Druart from comment #29) > (In reply to Jesse Maseto from comment #28) > > Backported to 18.05 for 18.05.07 > > One test is failing: > koha_1 | # Failed test 'No tests run for subtest "Check > stockrotationitem relationship"' > koha_1 | # at t/db_dependent/Items.t line 860. > koha_1 | DBIx::Class::Schema::source(): Can't find source for > Stockrotationitem at /kohadevbox/koha/t/lib/TestBuilder.pm line 189 > koha_1 | # Looks like your test exited with 255 just after 13. > koha_1 | [20:33:35] t/db_dependent/Items.t > > In order to make it pass, a follow-up for 18.05 should be provided to use > another class than the new "Stockrotationitem" Wrong bug ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Jonathan Druart changed: What|Removed |Added CC||jonathan.dru...@bugs.koha-c ||ommunity.org --- Comment #29 from Jonathan Druart --- (In reply to Jesse Maseto from comment #28) > Backported to 18.05 for 18.05.07 One test is failing: koha_1 | # Failed test 'No tests run for subtest "Check stockrotationitem relationship"' koha_1 | # at t/db_dependent/Items.t line 860. koha_1 | DBIx::Class::Schema::source(): Can't find source for Stockrotationitem at /kohadevbox/koha/t/lib/TestBuilder.pm line 189 koha_1 | # Looks like your test exited with 255 just after 13. koha_1 | [20:33:35] t/db_dependent/Items.t In order to make it pass, a follow-up for 18.05 should be provided to use another class than the new "Stockrotationitem" -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Jesse Maseto changed: What|Removed |Added Status|Pushed to Master|Pushed to Stable CC||je...@bywatersolutions.com --- Comment #28 from Jesse Maseto --- Backported to 18.05 for 18.05.07 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #27 from Marcel de Rooy --- OK We are talking different languages but meaning the same: bug fixed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #26 from Marcel de Rooy --- (In reply to Tomás Cohen Arazi from comment #24) > The bug this is fixing is that in master, the values are stored as > pipe-separated values, but on the way back for rendering they are lost, and > then data is lost. No data is lost with me. > I also noticed that when the item subfields are repeatable they are > displayed as pipe-separated codes, and the patch also makes them show the > descriptions instead (as happens with single values). The pipe separated codes should not be in the editor ! Not sure what you mean with showing descriptions. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Nick Clemens changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #25 from Nick Clemens --- Awesome work all! Pushed to master for 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com --- Comment #24 from Tomás Cohen Arazi --- (In reply to Nick Clemens from comment #22) > (In reply to Marcel de Rooy from comment #21) > > Thx Tomas > > I just want to make sure this is expected behvaiour: > 1 - Make an item field repeatable > 2 - Edit item, repeat a field > 3 - Save the item > 4 - The repeated field is stored as "VALUEA | VALUEB" > 5 - Edit the item > 6 - Field is loaded as a single pipe delimited field > > I worry about the use of pipe in regards to SIP, but can push if this is > confirmed as working as expected and it is restoring previous behvaiour The bug this is fixing is that in master, the values are stored as pipe-separated values, but on the way back for rendering they are lost, and then data is lost. I also noticed that when the item subfields are repeatable they are displayed as pipe-separated codes, and the patch also makes them show the descriptions instead (as happens with single values). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #23 from Marcel de Rooy --- (In reply to Nick Clemens from comment #22) > I just want to make sure this is expected behvaiour: > 1 - Make an item field repeatable > 2 - Edit item, repeat a field > 3 - Save the item > 4 - The repeated field is stored as "VALUEA | VALUEB" > 5 - Edit the item > 6 - Field is loaded as a single pipe delimited field No, in step 6 you should see two separate subfields. Check framework? > I worry about the use of pipe in regards to SIP, but can push if this is > confirmed as working as expected and it is restoring previous behvaiour This is not introducing something new. Just what we did before. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #22 from Nick Clemens --- (In reply to Marcel de Rooy from comment #21) > Thx Tomas I just want to make sure this is expected behvaiour: 1 - Make an item field repeatable 2 - Edit item, repeat a field 3 - Save the item 4 - The repeated field is stored as "VALUEA | VALUEB" 5 - Edit the item 6 - Field is loaded as a single pipe delimited field I worry about the use of pipe in regards to SIP, but can push if this is confirmed as working as expected and it is restoring previous behvaiour -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #21 from Marcel de Rooy --- Thx Tomas -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Tomás Cohen Arazi changed: What|Removed |Added Attachment #82390|0 |1 is obsolete|| --- Comment #20 from Tomás Cohen Arazi --- Created attachment 82543 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82543=edit Bug 21774: Cloned item subfields disappear when editing an item Bug 10306 changed behavior on cloning item subfields by no longer splitting constructions like 'A | B' in item fields like ccode. If it is really recommended to clone item subfields, I am not so sure about. But this patch at least restores the possibility to do so while we discuss if we should ;) Test plan: [1] Run Items.t [2] Make an item subfield repeatable in framework. And test edit items. Signed-off-by: Marcel de Rooy Signed-off-by: Fridolin Somers Signed-off-by: Tomas Cohen Arazi Works as expected. Also fixes the display of collections on the items table (on editing items). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Tomás Cohen Arazi changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #19 from Marcel de Rooy --- (In reply to Fridolin SOMERS from comment #18) > Works well. > Very old bug I think. > Looks like nearly nobody uses repeatable subfields in items field ;) Thanks for testing. No old bug btw. See above comments. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Fridolin SOMERS changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #18 from Fridolin SOMERS --- Works well. Very old bug I think. Looks like nearly nobody uses repeatable subfields in items field ;) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Fridolin SOMERS changed: What|Removed |Added Attachment #82080|0 |1 is obsolete|| --- Comment #17 from Fridolin SOMERS --- Created attachment 82390 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82390=edit Bug 21774: Cloned item subfields disappear when editing an item Bug 10306 changed behavior on cloning item subfields by no longer splitting constructions like 'A | B' in item fields like ccode. If it is really recommended to clone item subfields, I am not so sure about. But this patch at least restores the possibility to do so while we discuss if we should ;) Test plan: [1] Run Items.t [2] Make an item subfield repeatable in framework. And test edit items. Signed-off-by: Marcel de Rooy Signed-off-by: Fridolin Somers -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Fridolin SOMERS changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added CC||n...@bywatersolutions.com Keywords||rel_18_11_candidate --- Comment #16 from Marcel de Rooy --- Nick we need this one too in 18.11 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #15 from Marcel de Rooy --- Opened bug 21800 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #14 from Marcel de Rooy --- (In reply to M. Tompsett from comment #12) > I could sign this off, but then we haven't really solved the enforcing of > non-repeatables. Why do you think frameworkcode was passed before it was > removed in bug 19096? It is what would determine if it is repeatable, not > the default framework. This patch allows anything and all things to repeat. > Perhaps I'm reading the code wrong? *shrug* You make a point, but frameworkcode was unfortunately not used to check if the field was repeatable. There was just a split on pipe. Report 19096 removed the framework in order to make Koha consistent in its handling of kohafield: always use the default. But it did not look at repeatability just as the former code did not. So now we face the challenge of resolving this issue. The fix here is simple and can still go into 18.11 and yes it might repeat a bit too much (like before). But benefits are far greater. And I would still prefer to solve the repeatability issue on another report, perhaps making its way into 18.11.01 which would be a fix for something that we did never check before. (Just tested this again in 16.11: put a pipe in your barcode field.) So hope you signoff ! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added Severity|major |critical --- Comment #13 from Marcel de Rooy --- (In reply to Katrin Fischer from comment #11) > Just noting: we are using repeatable subfields for notes and custom item > fields. It would be bad for us if this feature was broken. I am raising the severity again. Thx for feedback. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 M. Tompsett changed: What|Removed |Added CC||mtomp...@hotmail.com --- Comment #12 from M. Tompsett --- (In reply to Marcel de Rooy from comment #10) > Split or not based on the frameworkcode or repeatableness of the subfield is > imo an enhancement request; this was no part of earlier code. It wasn't part of the earlier code, because they did no_split as a stop gap measure to fix it. The fix that made it in was wrong. This wouldn't be an enhancement, because it would actually be fixing the problem correctly. I could sign this off, but then we haven't really solved the enforcing of non-repeatables. Why do you think frameworkcode was passed before it was removed in bug 19096? It is what would determine if it is repeatable, not the default framework. This patch allows anything and all things to repeat. Perhaps I'm reading the code wrong? *shrug* > ... I could also think of having the option to use > another separator than pipe. But they should imo be handled on a new report. Pipe is already used as a separator in other places. I don't see a need for separator flexibility. But yes, that would be a separate issue. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de --- Comment #11 from Katrin Fischer --- Just noting: we are using repeatable subfields for notes and custom item fields. It would be bad for us if this feature was broken. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #10 from Marcel de Rooy --- (In reply to M. Tompsett from comment #9) > I'm not sure no_split logic was the correct solution, so I have no issue > with this removal. However, I think they may be tried to use that to split > or not split based on the frameworkcode, because the field might not be > repeatable, and so it is a little larger than this change alone. Just my > thoughts on this. I could be off on a different tangent. Hi Mark, Split or not based on the frameworkcode or repeatableness of the subfield is imo an enhancement request; this was no part of earlier code. If feel it could be useful though. I could also think of having the option to use another separator than pipe. But they should imo be handled on a new report. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774
--- Comment #9 from M. Tompsett ---
Comment on attachment 82080
--> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82080
Bug 21774: Cloned item subfields disappear when editing an item
Review of attachment 82080:
-->
(https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html=21774=82080)
-
::: C4/Items.pm
@@ +1342,4 @@
> } keys %{ $itemrecord }
> };
> my $framework = C4::Biblio::GetFrameworkCode( $biblionumber );
> +my $itemmarc = C4::Biblio::TransformKohaToMarc( $mungeditem ); # Bug
> 21774: no_split parameter removed to allow cloned subfields
I'm not sure no_split logic was the correct solution, so I have no issue with
this removal. However, I think they may be tried to use that to split or not
split based on the frameworkcode, because the field might not be repeatable,
and so it is a little larger than this change alone. Just my thoughts on this.
I could be off on a different tangent.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774
--- Comment #8 from je...@medios.es ---
> Replace
> my $itemmarc = C4::Biblio::TransformKohaToMarc(
> $mungeditem, { no_split => 1},
> );
> by
> my $itemmarc = C4::Biblio::TransformKohaToMarc($mungeditem);
>
Making this change solved the bug for us in 18.05.05-1
Thx.
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added Attachment #82078|0 |1 is obsolete|| --- Comment #7 from Marcel de Rooy --- Created attachment 82080 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82080=edit Bug 21774: Cloned item subfields disappear when editing an item Bug 10306 changed behavior on cloning item subfields by no longer splitting constructions like 'A | B' in item fields like ccode. If it is really recommended to clone item subfields, I am not so sure about. But this patch at least restores the possibility to do so while we discuss if we should ;) Test plan: [1] Run Items.t [2] Make an item subfield repeatable in framework. And test edit items. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added Depends on||10306 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10306 [Bug 10306] Koha to MARC mappings (Part 1): Allow multiple mappings per kohafield (for say 260/RDA 264) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added Status|ASSIGNED|Needs Signoff Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 --- Comment #6 from Marcel de Rooy --- Created attachment 82078 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82078=edit Bug 21774: Cloned item subfields disappear when editing an item Bug 10306 changed behavior on cloning item subfields by no longer splitting constructions like 'A | B' in item fields like ccode. If it is really recommended to clone item subfields, I am not so sure about. But this patch at least restores the possibility to do so while we discuss if we should ;) Test plan: [1] Run Items.t [2] Make an item subfield repeatable in framework. And test edit items. Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 21774] Cloned item subfields disappear when editing an item
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21774 Marcel de Rooy changed: What|Removed |Added Summary|Subfield clone fields |Cloned item subfields |dissapear when editing a|disappear when editing an |record |item -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
