Bug 22892: Remove warning from rebuild_es if --processes not passed
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Mon, 13 May 2019 00:46:58 +0000 (19:46 -0500)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 15 May 2019 12:10:53 +0000 (12:10 +0000)
Use of uninitialized value $processes in numeric lt (<) at
misc/search_tools/rebuild_elasticsearch.pl line 199.

We want the number of processes to be set to 1 by default, and then
assign it to $slice_count

Test plan:
Run the script with and without --processes and confirm that the warning
went away.

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

misc/search_tools/rebuild_elasticsearch.pl

index fa88a3c..1199b63 100755 (executable)
@@ -138,7 +138,7 @@ _verify_index_state($Koha::SearchEngine::Elasticsearch::BIBLIOS_INDEX, $delete)
 _verify_index_state($Koha::SearchEngine::Elasticsearch::AUTHORITIES_INDEX, $delete) if ($index_authorities);
 
 my $slice_index = 0;
-my $slice_count = $processes // 1;
+my $slice_count = ( $processes //= 1 );
 my %iterator_options;
 
 if ($slice_count > 1) {