Bug 13531: QA follow up
authorThomas <tomsStudy@gmail.com>
Wed, 14 Jan 2015 20:20:36 +0000 (20:20 +0000)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Tue, 31 Mar 2015 13:56:14 +0000 (10:56 -0300)
commit002b79c200f03adc162a5f59b988e8eb6c65e229
tree49dd7d0c4f2366c1be586ab18edc779b81e4ee03
parente1cdb4ebfa9e00850566a01de2719ff8daa9aee0
Bug 13531: QA follow up

A minor QA comment.

::: misc/migration_tools/bulkmarcimport.pl
@@ +271,5 @@
>          my ( $error, $results, $totalhits ) = C4::Search::SimpleSearch( $query, 0, 3, [$server] );
> +        # changed to warn so able to continue with one broken record
> +        if ( defined $error ) {
> +            warn "unable to search the database for duplicates : $error";
> +            next;

For consistency with the rest of the script, should this perhaps be:

next RECORD;

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
misc/migration_tools/bulkmarcimport.pl