From: Jason Stephenson <jason@sigio.com>
Date: Sat, 2 Jun 2018 23:18:40 +0000 (-0400)
Subject: LP 1768715: Fix pingest.pl bug discovered by Dan Scott of Laurentian U.
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=41e6fe45422e24c1b2af4f638d7113c109d7aff2;p=evergreen%2Fpines.git

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>
---

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);
     }