Hi,

On Sat, Apr 18, 2009 at 10:57 AM, Nahuel ANGELINETTI
<[email protected]> wrote:
> From: Henri-Damien LAURENT <[email protected]>

It would be better to replace much of the code added by this patch
with a call to C4::Biblio::_koha_marc_update_bib_ids().  This will
avoid copy and paste and work better: see below:

> +               if ($tag>010){

This condition is wrong in two different ways:

1. "010" is an *octal* literal, so has value 8.
2. MARC tag 010 can have subfields, so the condition should be $tag >= 10.

Regards,

Galen
-- 
Galen Charlton
VP, Research & Development, LibLime
[email protected]
p: 1-888-564-2457 x709
skype: gmcharlt
_______________________________________________
Koha-patches mailing list
[email protected]
http://lists.koha.org/mailman/listinfo/koha-patches

Reply via email to