http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7729
Paul Poulain <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] --- Comment #7 from Paul Poulain <[email protected]> --- QA comment: another question: The code says: + my $fullrecord = $record->clone(); then a few lines later: - foreach my $field ($record->field($itemtag)) { + foreach my $field ( $record->field($itemtag) ) { $record->delete_field($field); } then + foreach my $field ( $fullrecord->field($itemtag) ) { + my $one_item_record = $record->clone(); + $one_item_record->add_fields($field); + ModItemFromMarc( $one_item_record, $biblionumber, + $field->subfield($itemsubfield) ); + } so= we clone the record, remove items from old record if there are, then reintroduce item in the new record. Shouldn't we have - foreach my $field ($record->field($itemtag)) { + foreach my $field ( $fullrecord->field($itemtag) ) { $record->delete_field($field); } ? -- 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/
