[TASK] TceformsUpdateWizard: Throw exception if database query fails 93/29093/2
authorMarkus Klein <klein.t3@mfc-linz.at>
Wed, 2 Apr 2014 15:57:59 +0000 (17:57 +0200)
committerSteffen Müller <typo3@t3node.com>
Wed, 2 Apr 2014 19:33:42 +0000 (21:33 +0200)
Inform the user, if the database SELECT query fails during the wizard.

Resolves: #57572
Releases: 6.2
Change-Id: I0908aff5d58d6d309b471c810d924f26fc1ca577
Reviewed-on: https://review.typo3.org/29093
Reviewed-by: Alexander Opitz
Tested-by: Alexander Opitz
Reviewed-by: Steffen Müller
Tested-by: Steffen Müller
typo3/sysext/install/Classes/Updates/TceformsUpdateWizard.php

index 9550310..338955e 100644 (file)
@@ -224,6 +224,7 @@ class TceformsUpdateWizard extends AbstractUpdate {
         * @param string $fieldToMigrate
         * @param array $relationFields
         * @param int $limit Maximum number records to select
+        * @throws \RuntimeException
         * @return array
         */
        protected function getRecordsFromTable($table, $fieldToMigrate, $relationFields, $limit) {
@@ -235,7 +236,11 @@ class TceformsUpdateWizard extends AbstractUpdate {
                        . ' AND ' . $fieldToMigrate . ' != \'\''
                        . ' AND CAST(CAST(' . $fieldToMigrate . ' AS DECIMAL) AS CHAR) <> ' . $fieldToMigrate
                        . $deletedCheck;
-               return $this->database->exec_SELECTgetRows($fields, $table, $where, '', '', $limit);
+               $result = $this->database->exec_SELECTgetRows($fields, $table, $where, '', '', $limit);
+               if ($result === NULL) {
+                       throw new \RuntimeException('Database query failed. Error was: ' . $this->database->sql_error());
+               }
+               return $result;
        }
 
        /**