From 41e6fe45422e24c1b2af4f638d7113c109d7aff2 Mon Sep 17 00:00:00 2001
From: Jason Stephenson <jason@sigio.com>
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 <jason@sigio.com>
Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
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