From 41e6fe45422e24c1b2af4f638d7113c109d7aff2 Mon Sep 17 00:00:00 2001 From: Jason Stephenson Date: Sat, 2 Jun 2018 19:18:40 -0400 Subject: [PATCH] LP 1768715: Fix pingest.pl bug discovered by Dan Scott of Laurentian U. Reingest was not happening if all ingests were skipped, except for the display field reingest. This came down to a matter of two conditional statements not including checks for the $skip_display variable being true. Signed-off-by: Jason Stephenson Signed-off-by: Bill Erickson --- Open-ILS/src/support-scripts/pingest.pl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Open-ILS/src/support-scripts/pingest.pl b/Open-ILS/src/support-scripts/pingest.pl index fac993ce6c..2ddaac55ff 100755 --- a/Open-ILS/src/support-scripts/pingest.pl +++ b/Open-ILS/src/support-scripts/pingest.pl @@ -202,7 +202,7 @@ while ($count < $lists) { if (scalar(@lol) && scalar(@running) < $max_child && !$duration_expired) { # Reuse $records for the lulz. $records = shift(@lol); - if ($skip_search && $skip_facets && $skip_attrs) { + if ($skip_search && $skip_facets && $skip_attrs && $skip_display) { $count++; } else { reingest($records); @@ -271,7 +271,7 @@ sub reingest { $db_user, $db_password); reingest_attributes($dbh, $list) unless ($skip_attrs); reingest_field_entries($dbh, $list) - unless ($skip_facets && $skip_search); + unless ($skip_facets && $skip_search && $skip_display); $dbh->disconnect(); exit(0); } -- 2.11.0