View | Details | Raw Unified | Return to bug 20447
Collapse All | Expand All

(-)a/Koha/Holding.pm (-3 / +3 lines)
Lines 321-333 sub marc_to_koha_fields { Link Here
321
            my $sub = $field->{tagsubfield};
321
            my $sub = $field->{tagsubfield};
322
            foreach my $fld ($record->field($tag)) {
322
            foreach my $fld ($record->field($tag)) {
323
                if( $sub eq '@' || $fld->is_control_field ) {
323
                if( $sub eq '@' || $fld->is_control_field ) {
324
                    push @values, $fld->data if $fld->data;
324
                    push @values, $fld->data;
325
                } else {
325
                } else {
326
                    push @values, grep { $_ } $fld->subfield($sub);
326
                    push @values, $fld->subfield($sub);
327
                }
327
                }
328
            }
328
            }
329
        }
329
        }
330
        $result->{$column} = join(' | ', @values) if (@values);
330
        $result->{$column} = scalar(@values) ? join(' | ', @values) : undef;
331
    }
331
    }
332
    return $result;
332
    return $result;
333
}
333
}
(-)a/t/db_dependent/Koha/Holding.t (-1 / +6 lines)
Lines 139-144 subtest 'Koha::Holding tests' => sub { Link Here
139
        {
139
        {
140
            holdingbranch => 'ABC',
140
            holdingbranch => 'ABC',
141
            location => 'DEF',
141
            location => 'DEF',
142
            ccode => undef,
143
            indexes => undef,
144
            public_note => undef,
145
            callnumber => undef,
146
            summary => undef,
147
            supplements => undef,
142
            suppress => 1,
148
            suppress => 1,
143
            holding_id => $new_holding->holding_id()
149
            holding_id => $new_holding->holding_id()
144
        },
150
        },
145
- 

Return to bug 20447