http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9472
M. de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Failed QA --- Comment #6 from M. de Rooy <[email protected]> --- (In reply to Jonathan Druart from comment #5) > Yes, I am aware of that. > But actually, against master, the display is: > all subfields, alphabetically sorted, present in the record > followed by: > all subfields, alphabetically sorted, non present in the record. > > (search for foreach my $subfield ( sort( keys %{ $tagslib->{$tag} } ) ) ~ > l.600-700) > > This patch just suggests to display all subfields, alphabetically sorted. > Which makes more sense. > > If my dsecription of how it works on master is wrong, my patch is certainly > wrong. Yes, I am still afraid that your patch is actually "wrong" ;) I am not sure if you understood my point too. Hope the following clarifies: I tested this: I had a tag 100 with a 9 4 q d e (4 and e were empty) Without your patch, I look in catalogue marc detail, I see: 100 a q d. Fine! Without your patch, I look in opac marc detail, I see: 100 a 9 q d. Fine too. Now with your patch, I edit the record: The order is now: 4-9-a-d-e-q Catalogue marc detail gives: a-d-q CHANGED ! Opac marc detail: 9-a-d-q CHANGED AGAIN ! Failed QA -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] 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/
