Abstracted use of exit() to make a cleaner solution easier to implement in the future.

module/VuFind/src/VuFind/CLI/Controller/AbstractBase.php

@@ -71,4 +71,26 @@ class AbstractBase extends AbstractActionController
             echo "Without it, inappropriate default settings may be loaded.\n\n";
         }
     }
+
+    /**
+     * Indicate failure.
+     *
+     * @return void
+     */
+    protected function getFailureResponse()
+    {
+        // TODO: better framework integration for response codes
+        exit(1);
+    }
+
+    /**
+     * Indicate success.
+     *
+     * @return void
+     */
+    protected function getSuccessResponse()
+    {
+        // TODO: better framework integration for response codes
+        exit(0);
+    }
 }
\ No newline at end of file

module/VuFind/src/VuFind/CLI/Controller/ImportController.php

@@ -72,7 +72,7 @@ class ImportController extends AbstractBase
                 . "SolrAuth to load authority records.\n\n"
                 . "Note: See vudl.properties and ojs.properties for configuration "
                 . "examples.\n";
-            exit(1);
+            return $this->getFailureResponse();
         }
 
         // Try to import the document if successful:
@@ -80,11 +80,11 @@ class ImportController extends AbstractBase
             Importer::save($argv[0], $argv[1], $index, $testMode);
         } catch (\Exception $e) {
             echo "Fatal error: " . $e->getMessage() . "\n";
-            exit(1);
+            return $this->getFailureResponse();
         }
         if (!$testMode) {
             echo "Successfully imported {$argv[0]}...\n";
         }
-        exit(0);
+        return $this->getSuccessResponse();
     }
 }