Reverting constructor argument refactor

per code review by @jpwhite I'm changing the constructor arguments to
their original location and just changing the
`$pre_ingest_update_statements` to not have a default value. This has
been accounted for in it's usage in `Shredder.php`.

classes/DB/PDODBMultiIngestor.php

@@ -41,9 +41,9 @@ class PDODBMultiIngestor implements Ingestor
     function __construct(
         $dest_db,
         $source_db,
+        $pre_ingest_update_statements,
         $source_query,
         $insert_table,
-        $pre_ingest_update_statements = array(),
         $insert_fields = array(),
         $post_ingest_update_statements = array(),
         $delete_statement = null,

classes/DB/PDODBUnbufferedMultiIngestor.php

@@ -7,7 +7,7 @@ class PDODBUnbufferedMultiIngestor Extends PDODBMultiIngestor
     public function __construct(
         PDODB $dest_db,
         PDODB $source_db,
-        array $pre_ingest_update_statements = array(),
+        array $pre_ingest_update_statements,
         $source_query,
         $insert_table,
         array $insert_fields = array(),
@@ -40,9 +40,9 @@ public function __construct(
         parent::__construct(
             $dest_db,
             $source_db,
+            $pre_ingest_update_statements,
             $source_query,
             $insert_table,
-            $pre_ingest_update_statements,
             $insert_fields,
             $post_ingest_update_statements,
             $delete_statement,

classes/OpenXdmod/Shredder.php

@@ -563,9 +563,9 @@ public function getJobIngestor($ingestAll = false)
         $ingestor = new PDODBMultiIngestor(
             $this->db,
             $this->db,
+            array(),
             $sourceQuery,
             'shredded_job',
-            array(),
             $insertFields,
             array(),
             $deleteStatement