http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=12586

--- Comment #9 from David Cook <[email protected]> ---
Comment on attachment 46380
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=46380
Bug 12586 - Record matching rules - Required match checks doesn't work with
MARCXML

Review of attachment 46380:
 --> 
(http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=12586&attachment=46380)
-----------------------------------------------------------------

::: C4/Matcher.pm
@@ +703,5 @@
> +                foreach my $matched ( @{$searchresults} ) {
> +                    my $record = C4::Search::new_record_from_zebra( 
> 'biblioserver', $matched );
> +                    $matches{$record}->{score} += $matchpoint->{'score'}; 
> #Using $record HASH string representation as the key :)
> +                    $matches{$record}->{record} = $record;
> +                }

Actually, looking at this again, there's no point repeating the loop... it
would've made more sense to have 1 foreach loop and done the record_type checks
within it...

Not a big deal though really

-- 
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/

Reply via email to