From a0ebdab046bc647b89d8c95ada12a28087b6d5b8 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Wed, 22 Sep 2021 14:06:07 +0000 Subject: [PATCH] Bug 29078: Resolve division by zero in touch_all scripts Illegal division by zero at misc/maintenance/touch_all_biblios.pl line 102. Trivial fix. Funny that the percentage is not even shown without verbose flag ;) Test plan: Run misc/maintenance/touch_all_biblios.pl -where 'biblionumber<0' You should now see: Good: 0, Bad: 0 (of 0) in 0 seconds Signed-off-by: Marcel de Rooy Signed-off-by: Martin Renvoize Signed-off-by: Jonathan Druart --- misc/maintenance/touch_all_biblios.pl | 2 +- misc/maintenance/touch_all_items.pl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/misc/maintenance/touch_all_biblios.pl b/misc/maintenance/touch_all_biblios.pl index be349cfb19..2c725ae5e8 100755 --- a/misc/maintenance/touch_all_biblios.pl +++ b/misc/maintenance/touch_all_biblios.pl @@ -101,7 +101,7 @@ close($fh); # Benchmarking my $endtime = time(); my $time = $endtime-$startime; -my $accuracy = ($goodcount / $totalcount) * 100; # this is a percentage +my $accuracy = $totalcount ? ($goodcount / $totalcount) * 100 : 0; # this is a percentage my $averagetime = 0; $averagetime = $time / $totalcount if $totalcount; print "Good: $goodcount, Bad: $badcount (of $totalcount) in $time seconds\n"; diff --git a/misc/maintenance/touch_all_items.pl b/misc/maintenance/touch_all_items.pl index 087914498e..c253959959 100755 --- a/misc/maintenance/touch_all_items.pl +++ b/misc/maintenance/touch_all_items.pl @@ -109,7 +109,7 @@ close($fh); # Benchmarking my $endtime = time(); my $time = $endtime-$startime; -my $accuracy = ($goodcount / $totalcount) * 100; # this is a percentage +my $accuracy = $totalcount ? ($goodcount / $totalcount) * 100 : 0; # this is a percentage my $averagetime = 0; $averagetime = $time / $totalcount if $totalcount; print "Good: $goodcount, Bad: $badcount (of $totalcount) in $time seconds\n"; -- 2.39.5