[CLEANUP] Use correct type hint for messages in install tool updates 24/51624/5
authorMarkus Klein <markus.klein@typo3.org>
Thu, 9 Feb 2017 11:35:03 +0000 (12:35 +0100)
committerAndreas Fernandez <typo3@scripting-base.de>
Fri, 10 Feb 2017 19:04:04 +0000 (20:04 +0100)
The data type of customMessage has been changed to string a while ago
but the method signatures still have a mixed-type.
Normalize this over all updates to make things consistent again.

Resolves: #79739
Releases: master
Change-Id: I840cc6d217dafb1a83f0af5b7a978fdc60318d5d
Reviewed-on: https://review.typo3.org/51624
Reviewed-by: Sascha Egerer <sascha@sascha-egerer.de>
Tested-by: Sascha Egerer <sascha@sascha-egerer.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
34 files changed:
typo3/sysext/install/Classes/Updates/AbstractDownloadExtensionUpdate.php
typo3/sysext/install/Classes/Updates/AbstractUpdate.php
typo3/sysext/install/Classes/Updates/AccessRightParametersUpdate.php
typo3/sysext/install/Classes/Updates/BackendUserStartModuleUpdate.php
typo3/sysext/install/Classes/Updates/BulletContentElementUpdate.php
typo3/sysext/install/Classes/Updates/Compatibility6ExtractionUpdate.php
typo3/sysext/install/Classes/Updates/ContentTypesToTextMediaUpdate.php
typo3/sysext/install/Classes/Updates/DatabaseCharsetUpdate.php
typo3/sysext/install/Classes/Updates/DatabaseRowsUpdateWizard.php
typo3/sysext/install/Classes/Updates/DbalAndAdodbExtractionUpdate.php
typo3/sysext/install/Classes/Updates/ExtensionManagerTables.php
typo3/sysext/install/Classes/Updates/FileListIsStartModuleUpdate.php
typo3/sysext/install/Classes/Updates/FilesReplacePermissionUpdate.php
typo3/sysext/install/Classes/Updates/FillTranslationSourceField.php
typo3/sysext/install/Classes/Updates/FinalDatabaseSchemaUpdate.php
typo3/sysext/install/Classes/Updates/FrontendUserImageUpdateWizard.php
typo3/sysext/install/Classes/Updates/InitialDatabaseSchemaUpdate.php
typo3/sysext/install/Classes/Updates/LanguageIsoCodeUpdate.php
typo3/sysext/install/Classes/Updates/MediaceExtractionUpdate.php
typo3/sysext/install/Classes/Updates/MigrateCscStaticTemplateUpdate.php
typo3/sysext/install/Classes/Updates/MigrateFscStaticTemplateUpdate.php
typo3/sysext/install/Classes/Updates/MigrateMediaToAssetsForTextMediaCe.php
typo3/sysext/install/Classes/Updates/MigrateShortcutUrlsAgainUpdate.php
typo3/sysext/install/Classes/Updates/OpenidExtractionUpdate.php
typo3/sysext/install/Classes/Updates/PageShortcutParentUpdate.php
typo3/sysext/install/Classes/Updates/ProcessedFileChecksumUpdate.php
typo3/sysext/install/Classes/Updates/SectionFrameToFrameClassUpdate.php
typo3/sysext/install/Classes/Updates/SplitMenusUpdate.php
typo3/sysext/install/Classes/Updates/TableFlexFormToTtContentFieldsUpdate.php
typo3/sysext/install/Classes/Updates/UploadContentElementUpdate.php
typo3/sysext/install/Classes/Updates/WizardDoneToRegistry.php
typo3/sysext/install/Classes/Updates/WorkspacesNotificationSettingsUpdate.php
typo3/sysext/rtehtmlarea/Classes/Hook/Install/DeprecatedRteProperties.php
typo3/sysext/rtehtmlarea/Classes/Hook/Install/RteAcronymButtonRenamedToAbbreviation.php

index cf0a636..340b3f1 100644 (file)
@@ -47,10 +47,10 @@ abstract class AbstractDownloadExtensionUpdate extends AbstractUpdate
      * (e.g. installing in extList, respecting priority, etc.)
      *
      * @param string $extensionKey
-     * @param mixed $customMessages
+     * @param string $customMessage
      * @return bool whether the installation worked or not
      */
-    protected function installExtension($extensionKey, &$customMessages)
+    protected function installExtension($extensionKey, &$customMessage)
     {
         $updateSuccessful = true;
         /** @var $objectManager ObjectManager */
@@ -69,17 +69,17 @@ abstract class AbstractDownloadExtensionUpdate extends AbstractUpdate
             $extensionDetails = $this->getExtensionDetails($extensionKey);
             if (empty($extensionDetails)) {
                 $updateSuccessful = false;
-                $customMessages .= 'No version information for extension ' . $extensionKey . ' found. Can not install the extension.';
+                $customMessage .= 'No version information for extension ' . $extensionKey . ' found. Can not install the extension.';
             }
             $t3xContent = $this->fetchExtension($extensionKey, $extensionDetails['versionString']);
             if (empty($t3xContent)) {
                 $updateSuccessful = false;
-                $customMessages .= 'The extension ' . $extensionKey . ' could not be downloaded.';
+                $customMessage .= 'The extension ' . $extensionKey . ' could not be downloaded.';
             }
             $t3xExtracted = $extensionTerUtility->decodeExchangeData($t3xContent);
             if (empty($t3xExtracted) || !is_array($t3xExtracted) || empty($t3xExtracted['extKey'])) {
                 $updateSuccessful = false;
-                $customMessages .= 'The extension ' . $extensionKey . ' could not be extracted.';
+                $customMessage .= 'The extension ' . $extensionKey . ' could not be extracted.';
             }
 
             /** @var $extensionFileHandlingUtility FileHandlingUtility */
index 26a2576..dceb110 100644 (file)
@@ -168,10 +168,10 @@ abstract class AbstractUpdate
      * Performs the accordant updates.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool Whether everything went smoothly or not
      */
-    abstract public function performUpdate(array &$dbQueries, &$customMessages);
+    abstract public function performUpdate(array &$dbQueries, &$customMessage);
 
     /**
      * This method can be called to install extensions following all proper processes
index 3f7d9ba..a49389a 100644 (file)
@@ -69,10 +69,10 @@ class AccessRightParametersUpdate extends AbstractUpdate
      * Performs the configuration update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         /** @var ConfigurationManager $configurationManager */
         $configurationManager = GeneralUtility::makeInstance(ConfigurationManager::class);
index dae45d6..aa565fb 100644 (file)
@@ -62,10 +62,10 @@ class BackendUserStartModuleUpdate extends AbstractUpdate
      * "help_aboutmodules" or "help_AboutmodulesAboutmodules"
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('be_users');
         $statement = $queryBuilder->select('uid', 'uc')->from('be_users')->execute();
index d695be7..8c910f5 100644 (file)
@@ -58,10 +58,10 @@ class BulletContentElementUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $queryBuilder = $connection->createQueryBuilder();
index ba448bd..a39da25 100644 (file)
@@ -94,10 +94,10 @@ class Compatibility6ExtractionUpdate extends AbstractDownloadExtensionUpdate
      * Performs the update if EXT:compatibility6 should be installed.
      *
      * @param array $databaseQueries Queries done in this update
-     * @param mixed $customMessages Custom messages
+     * @param string $customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $requestParams = GeneralUtility::_GP('install');
         if (!isset($requestParams['values']['compatibility6Extension']['install'])) {
@@ -107,7 +107,7 @@ class Compatibility6ExtractionUpdate extends AbstractDownloadExtensionUpdate
 
         if ($install === 1) {
             // user decided to install extension, install and mark wizard as done
-            $updateSuccessful = $this->installExtension($this->extensionKey, $customMessages);
+            $updateSuccessful = $this->installExtension($this->extensionKey, $customMessage);
             if ($updateSuccessful) {
                 $this->markWizardAsDone();
                 return true;
index a8e3b98..743ab2d 100644 (file)
@@ -76,10 +76,10 @@ class ContentTypesToTextMediaUpdate extends AbstractUpdate
      * Performs the database update if old CTypes are available
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $ttContentConnection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $falConnection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('sys_file_reference');
index 6f03e86..1d48a2f 100644 (file)
@@ -61,12 +61,12 @@ class DatabaseCharsetUpdate extends AbstractUpdate
      * Performs the accordant updates.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool Whether everything went smoothly or not
      * @throws \InvalidArgumentException
      * @throws \Doctrine\DBAL\DBALException
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)
             ->getConnectionByName(ConnectionPool::DEFAULT_CONNECTION_NAME);
@@ -75,7 +75,7 @@ class DatabaseCharsetUpdate extends AbstractUpdate
         try {
             $connection->exec($sql);
         } catch (DBALException $e) {
-            $customMessages = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
+            $customMessage = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
             return false;
         }
         $dbQueries[] = $sql;
index 595f74f..1f3e889 100644 (file)
@@ -90,12 +90,12 @@ class DatabaseRowsUpdateWizard extends AbstractUpdate
      * Performs the configuration update.
      *
      * @param array &$databaseQueries Queries done in this update - not filled for this updater
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      * @throws \Doctrine\DBAL\ConnectionException
      * @throws \Exception
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $registry = GeneralUtility::makeInstance(Registry::class);
 
index 6589ec7..906d0c7 100644 (file)
@@ -94,10 +94,10 @@ class DbalAndAdodbExtractionUpdate extends AbstractDownloadExtensionUpdate
      * Fetch and enable ext:adodb and ext:dbal
      *
      * @param array $databaseQueries Queries done in this update
-     * @param mixed $customMessages Custom messages
+     * @param string $customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $requestParams = GeneralUtility::_GP('install');
         if (!isset($requestParams['values']['TYPO3\CMS\Install\Updates\DbalAndAdodbExtractionUpdate']['install'])) {
@@ -107,8 +107,8 @@ class DbalAndAdodbExtractionUpdate extends AbstractDownloadExtensionUpdate
 
         if ($install === 1) {
             // user decided to install extensions, install and mark wizard as done
-            $adodbSuccessful = $this->installExtension('adodb', $customMessages);
-            $dbalSuccessful = $this->installExtension('dbal', $customMessages);
+            $adodbSuccessful = $this->installExtension('adodb', $customMessage);
+            $dbalSuccessful = $this->installExtension('dbal', $customMessage);
             if ($adodbSuccessful && $dbalSuccessful) {
                 $this->markWizardAsDone();
                 return true;
index ba85185..bf75a7a 100644 (file)
@@ -115,19 +115,10 @@ class ExtensionManagerTables extends AbstractUpdate
      * Performs the database update.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool Whether it worked (TRUE) or not (FALSE)
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotReturnException
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\UnexpectedSignalReturnValueTypeException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\StatementException
-     * @throws \RuntimeException
-     * @throws \Doctrine\DBAL\Schema\SchemaException
-     * @throws \Doctrine\DBAL\DBALException
-     * @throws \InvalidArgumentException
-     * @throws \BadFunctionCallException
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
         $result = true;
         $sqlReader = GeneralUtility::makeInstance(SqlReader::class);
@@ -150,7 +141,7 @@ class ExtensionManagerTables extends AbstractUpdate
             $dbQueries[] = $statement;
             if ($errorMessage) {
                 $result = false;
-                $customMessages .= '<br /><br />SQL-ERROR: ' . htmlspecialchars($errorMessage);
+                $customMessage .= '<br /><br />SQL-ERROR: ' . htmlspecialchars($errorMessage);
             }
         }
 
index 9601e98..7029eb0 100644 (file)
@@ -66,10 +66,10 @@ class FileListIsStartModuleUpdate extends AbstractUpdate
      * Performs the database update if backend user's startmodule is file_list
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
             ->getQueryBuilderForTable('be_users');
index d6618db..ba6e137 100644 (file)
@@ -81,10 +81,10 @@ class FilesReplacePermissionUpdate extends AbstractUpdate
      * Performs the accordant updates.
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool Whether everything went smoothly or not
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         foreach ($this->tablesToProcess as $table) {
             $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable($table);
index 7abde09..08ff40c 100644 (file)
@@ -60,10 +60,10 @@ class FillTranslationSourceField extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)
             ->getConnectionForTable('tt_content');
index cff63a7..383f761 100644 (file)
@@ -109,18 +109,10 @@ class FinalDatabaseSchemaUpdate extends AbstractDatabaseSchemaUpdate
      * Performs the database update.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool TRUE on success, FALSE on error
-     * @throws \Doctrine\DBAL\DBALException
-     * @throws \Doctrine\DBAL\Schema\SchemaException
-     * @throws \InvalidArgumentException
-     * @throws \RuntimeException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\UnexpectedSignalReturnValueTypeException
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotException
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotReturnException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\StatementException
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
         $statements = $this->getDatabaseDefinition();
         $result = $this->schemaMigrationService->install($statements, false);
@@ -131,7 +123,7 @@ class FinalDatabaseSchemaUpdate extends AbstractDatabaseSchemaUpdate
         // Only keep error messages
         $result = array_filter($result);
 
-        $customMessages = implode(
+        $customMessage = implode(
             LF,
             array_map(
                 function (string $message) {
index 3ceb470..bc51c1e 100644 (file)
@@ -136,12 +136,12 @@ class FrontendUserImageUpdateWizard extends AbstractUpdate
      * Performs the database update.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
-     *
+     * @param string &$customMessage Custom message
      * @return bool TRUE on success, FALSE on error
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
+        $customMessage = '';
         try {
             $this->init();
 
@@ -153,7 +153,7 @@ class FrontendUserImageUpdateWizard extends AbstractUpdate
                 $limit = $this->recordOffset[$this->table] . ',' . self::RECORDS_PER_QUERY;
                 $records = $this->getRecordsFromTable($limit, $dbQueries);
                 foreach ($records as $record) {
-                    $this->migrateField($record, $customMessages, $dbQueries);
+                    $this->migrateField($record, $customMessage, $dbQueries);
                 }
                 $this->registry->set($this->registryNamespace, 'recordOffset', $this->recordOffset);
             } while (count($records) === self::RECORDS_PER_QUERY);
@@ -161,10 +161,10 @@ class FrontendUserImageUpdateWizard extends AbstractUpdate
             $this->markWizardAsDone();
             $this->registry->remove($this->registryNamespace, 'recordOffset');
         } catch (\Exception $e) {
-            $customMessages .= PHP_EOL . $e->getMessage();
+            $customMessage .= PHP_EOL . $e->getMessage();
         }
 
-        return empty($customMessages);
+        return empty($customMessage);
     }
 
     /**
@@ -221,12 +221,12 @@ class FrontendUserImageUpdateWizard extends AbstractUpdate
      * Migrates a single field.
      *
      * @param array $row
-     * @param string $customMessages
+     * @param string $customMessage
      * @param array $dbQueries
      *
      * @throws \Exception
      */
-    protected function migrateField($row, &$customMessages, &$dbQueries)
+    protected function migrateField($row, &$customMessage, &$dbQueries)
     {
         $fieldItems = GeneralUtility::trimExplode(',', $row[$this->fieldToMigrate], true);
         if (empty($fieldItems) || is_numeric($row[$this->fieldToMigrate])) {
@@ -302,7 +302,7 @@ class FrontendUserImageUpdateWizard extends AbstractUpdate
                     $format = 'File \'%s\' does not exist. Referencing field: %s.%d.%s. The reference was not migrated.';
                     $message = sprintf($format, $this->sourcePath . $item, $this->table,
                         $row['uid'], $this->fieldToMigrate);
-                    $customMessages .= PHP_EOL . $message;
+                    $customMessage .= PHP_EOL . $message;
                     continue;
                 }
             }
index 644cb59..7e5dc9c 100644 (file)
@@ -110,18 +110,10 @@ class InitialDatabaseSchemaUpdate extends AbstractDatabaseSchemaUpdate
      * Performs the database update.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool TRUE on success, FALSE on error
-     * @throws \Doctrine\DBAL\DBALException
-     * @throws \Doctrine\DBAL\Schema\SchemaException
-     * @throws \InvalidArgumentException
-     * @throws \RuntimeException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\UnexpectedSignalReturnValueTypeException
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotException
-     * @throws \TYPO3\CMS\Extbase\SignalSlot\Exception\InvalidSlotReturnException
-     * @throws \TYPO3\CMS\Core\Database\Schema\Exception\StatementException
      */
-    public function performUpdate(array &$dbQueries, &$customMessages): bool
+    public function performUpdate(array &$dbQueries, &$customMessage): bool
     {
         $statements = $this->getDatabaseDefinition();
         $result = $this->schemaMigrationService->install($statements, true);
@@ -132,7 +124,7 @@ class InitialDatabaseSchemaUpdate extends AbstractDatabaseSchemaUpdate
         // Only keep error messages
         $result = array_filter($result);
 
-        $customMessages = implode(
+        $customMessage = implode(
             LF,
             array_map(
                 function (string $message) {
index 0670b52..a3b326e 100644 (file)
@@ -13,6 +13,7 @@ namespace TYPO3\CMS\Install\Updates;
  *
  * The TYPO3 project - inspiring people to share!
  */
+
 use TYPO3\CMS\Core\Database\ConnectionPool;
 use TYPO3\CMS\Core\Utility\ExtensionManagementUtility;
 use TYPO3\CMS\Core\Utility\GeneralUtility;
@@ -64,10 +65,10 @@ class LanguageIsoCodeUpdate extends AbstractUpdate
      * data of the old relation.
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('sys_language');
         $statement = $queryBuilder->select('uid', 'language_isocode', 'static_lang_isocode')
index ad1a06b..f684900 100644 (file)
@@ -88,12 +88,12 @@ class MediaceExtractionUpdate extends AbstractDownloadExtensionUpdate
      * Performs the database update if media CTypes are available.
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
-        $updateSuccessful = $this->installExtension($this->extensionKey, $customMessages);
+        $updateSuccessful = $this->installExtension($this->extensionKey, $customMessage);
         if ($updateSuccessful) {
             $this->markWizardAsDone();
         }
index 5f0b4db..f55a3e3 100644 (file)
@@ -82,10 +82,10 @@ class MigrateCscStaticTemplateUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('sys_template');
         $queryBuilder = $connection->createQueryBuilder();
index 7e68e1d..cbe28d4 100644 (file)
@@ -70,10 +70,10 @@ class MigrateFscStaticTemplateUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('sys_template');
         $queryBuilder = $connection->createQueryBuilder();
index be1d79b..c8c5bef 100644 (file)
@@ -68,10 +68,10 @@ class MigrateMediaToAssetsForTextMediaCe extends AbstractUpdate
      * Performs the database update if old mediafile references are available
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
             ->getQueryBuilderForTable('tt_content');
@@ -104,7 +104,7 @@ class MigrateMediaToAssetsForTextMediaCe extends AbstractUpdate
             try {
                 $queryBuilder->execute();
             } catch (DBALException $e) {
-                $customMessages = 'MySQL-Error: ' . $queryBuilder->getConnection()->errorInfo();
+                $customMessage = 'MySQL-Error: ' . $queryBuilder->getConnection()->errorInfo();
                 return false;
             }
 
@@ -130,7 +130,7 @@ class MigrateMediaToAssetsForTextMediaCe extends AbstractUpdate
             try {
                 $queryBuilder->execute();
             } catch (DBALException $e) {
-                $customMessages = 'MySQL-Error: ' . $queryBuilder->getConnection()->errorInfo();
+                $customMessage = 'MySQL-Error: ' . $queryBuilder->getConnection()->errorInfo();
                 // if the second query is not successful but the first was we'll have
                 // to get back to a consistent state by rolling back the first query.
                 $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
index ee26241..5d756a1 100644 (file)
@@ -51,10 +51,10 @@ class MigrateShortcutUrlsAgainUpdate extends AbstractUpdate
      * Performs the database update if shortcuts are available
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('sys_be_shortcuts');
         $statement = $connection->select(['uid', 'url'], 'sys_be_shortcuts', []);
index 17b32be..f83c2cd 100644 (file)
@@ -85,12 +85,12 @@ class OpenidExtractionUpdate extends AbstractDownloadExtensionUpdate
      * Performs the update if EXT:openid is used.
      *
      * @param array $databaseQueries Queries done in this update
-     * @param mixed $customMessages Custom messages
+     * @param string $customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
-        $updateSuccessful = $this->installExtension($this->extensionKey, $customMessages);
+        $updateSuccessful = $this->installExtension($this->extensionKey, $customMessage);
         if ($updateSuccessful) {
             $this->markWizardAsDone();
         }
index ea1fc5f..43f4fa5 100644 (file)
@@ -69,10 +69,10 @@ class PageShortcutParentUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('pages');
         $queryBuilder->update('pages')
index fb649b7..923f4bd 100644 (file)
@@ -82,10 +82,10 @@ class ProcessedFileChecksumUpdate extends AbstractUpdate
      * Performs the update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $registry = GeneralUtility::makeInstance(Registry::class);
         $factory = GeneralUtility::makeInstance(ResourceFactory::class);
index 7390162..54fb322 100644 (file)
@@ -65,10 +65,10 @@ class SectionFrameToFrameClassUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $queryBuilder = $connection->createQueryBuilder();
index a415d28..1813bd2 100644 (file)
@@ -59,10 +59,10 @@ class SplitMenusUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $queryBuilder = $connection->createQueryBuilder();
index f1081bd..e24bca1 100644 (file)
@@ -63,10 +63,10 @@ class TableFlexFormToTtContentFieldsUpdate extends AbstractUpdate
      * Performs the database update if CType 'table' still has content in pi_flexform
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $queryBuilder = $connection->createQueryBuilder();
index 134b784..48574b8 100644 (file)
@@ -58,10 +58,10 @@ class UploadContentElementUpdate extends AbstractUpdate
      * Performs the database update
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $connection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('tt_content');
         $queryBuilder = $connection->createQueryBuilder();
index c8a8ff6..e25a19e 100644 (file)
@@ -55,10 +55,10 @@ class WizardDoneToRegistry extends AbstractUpdate
      * Performs the accordant updates.
      *
      * @param array &$dbQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool Whether everything went smoothly or not
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
         $configurationManager = GeneralUtility::makeInstance(ConfigurationManager::class);
         $registry = GeneralUtility::makeInstance(Registry::class);
index b92d7ad..91719a4 100644 (file)
@@ -71,10 +71,10 @@ class WorkspacesNotificationSettingsUpdate extends AbstractUpdate
      * Perform the database updates for workspace records
      *
      * @param array &$databaseQueries Queries done in this update
-     * @param mixed &$customMessages Custom messages
+     * @param string &$customMessage Custom message
      * @return bool
      */
-    public function performUpdate(array &$databaseQueries, &$customMessages)
+    public function performUpdate(array &$databaseQueries, &$customMessage)
     {
         $workspaceConnection = GeneralUtility::makeInstance(ConnectionPool::class)->getConnectionForTable('sys_workspace');
         $queryBuilder = $workspaceConnection->createQueryBuilder();
index 2ede937..27586b4 100644 (file)
@@ -162,49 +162,49 @@ class DeprecatedRteProperties extends AbstractUpdate
      * Performs the update itself
      *
      * @param array $dbQueries Pointer where to insert all DB queries made, so they can be shown to the user if wanted
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom messages
      * @return bool TRUE if update succeeded, FALSE otherwise
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
-        $customMessages = '';
-        $pages = $this->getPagesWithDeprecatedRteProperties($customMessages);
-        if (empty($customMessages)) {
+        $customMessage = '';
+        $pages = $this->getPagesWithDeprecatedRteProperties($customMessage);
+        if (empty($customMessage)) {
             $pagesCount = count($pages);
             if ($pagesCount) {
                 $updateablePages = $this->findUpdateablePagesWithDeprecatedRteProperties($pages);
                 if (!empty($updateablePages)) {
-                    $this->updatePages($updateablePages, $dbQueries, $customMessages);
+                    $this->updatePages($updateablePages, $dbQueries, $customMessage);
                     // If the update was successful
-                    if (empty($customMessages)) {
+                    if (empty($customMessage)) {
                         // If all pages were updated, we query again to check if any deprecated properties are still present.
                         if (count($updateablePages) === $pagesCount) {
-                            $pagesAfter = $this->getPagesWithDeprecatedRteProperties($customMessages);
-                            if (empty($customMessages)) {
+                            $pagesAfter = $this->getPagesWithDeprecatedRteProperties($customMessage);
+                            if (empty($customMessage)) {
                                 if (!empty($pagesAfter)) {
-                                    $customMessages = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
+                                    $customMessage = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
                                 }
                             }
                         } else {
-                            $customMessages = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
+                            $customMessage = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
                         }
                     }
                 } else {
-                    $customMessages = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace any of the deprecated properties found. These properties will have to be replaced manually.';
+                    $customMessage = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace any of the deprecated properties found. These properties will have to be replaced manually.';
                 }
             }
         }
         $this->markWizardAsDone();
-        return empty($customMessages);
+        return empty($customMessage);
     }
 
     /**
      * Gets the pages with deprecated RTE properties in TSconfig column
      *
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom message
      * @return array uid and inclusion string for the pages with deprecated RTE properties in TSconfig column
      */
-    protected function getPagesWithDeprecatedRteProperties(&$customMessages)
+    protected function getPagesWithDeprecatedRteProperties(&$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('pages');
         $queryBuilder->getRestrictions()->removeAll();
@@ -261,7 +261,7 @@ class DeprecatedRteProperties extends AbstractUpdate
                 ->execute()
                 ->fetchAll();
         } catch (DBALException $e) {
-            $customMessages = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
+            $customMessage = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
         }
 
         return [];
@@ -293,9 +293,9 @@ class DeprecatedRteProperties extends AbstractUpdate
      *
      * @param array $pages Page records to update, fetched by getTemplates() and filtered by
      * @param array $dbQueries Pointer where to insert all DB queries made, so they can be shown to the user if wanted
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom message
      */
-    protected function updatePages($pages, &$dbQueries, &$customMessages)
+    protected function updatePages($pages, &$dbQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('pages');
         foreach ($pages as $page) {
@@ -310,7 +310,7 @@ class DeprecatedRteProperties extends AbstractUpdate
                     ->set('TSconfig', $page['TSconfig'])
                     ->execute();
             } catch (DBALException $e) {
-                $customMessages .= 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage()) . LF . LF;
+                $customMessage .= 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage()) . LF . LF;
             }
             $dbQueries[] = str_replace(LF, ' ', $queryBuilder->getSQL());
         }
index ef58bba..e6d21c2 100644 (file)
@@ -40,7 +40,7 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate
     {
         $result = false;
 
-        $pages = $this->getPagesWithDeprecatedRteProperties($customMessages);
+        $pages = $this->getPagesWithDeprecatedRteProperties($customMessage);
         $pagesCount = count($pages);
         $description = '<p>The RTE "acronym" button is deprecated and replaced by the "abbreviation" button since TYPO3 CMS 7.0.</p>' . LF . '<p>Page TSconfig currently includes the string "acronym" on <strong>' . strval($pagesCount) . '&nbsp;pages</strong>  (including deleted and hidden pages).</p>' . LF;
         if ($pagesCount) {
@@ -75,41 +75,41 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate
      * Performs the update itself
      *
      * @param array $dbQueries Pointer where to insert all DB queries made, so they can be shown to the user if wanted
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom message
      * @return bool TRUE if update succeeded, FALSE otherwise
      */
-    public function performUpdate(array &$dbQueries, &$customMessages)
+    public function performUpdate(array &$dbQueries, &$customMessage)
     {
-        $customMessages = '';
-        $pages = $this->getPagesWithDeprecatedRteProperties($customMessages);
-        if (empty($customMessages)) {
+        $customMessage = '';
+        $pages = $this->getPagesWithDeprecatedRteProperties($customMessage);
+        if (empty($customMessage)) {
             $pagesCount = count($pages);
             if ($pagesCount) {
                 $updateablePages = $this->findUpdateablePagesWithDeprecatedRteProperties($pages);
                 if (!empty($updateablePages)) {
-                    $this->updatePages($updateablePages, $dbQueries, $customMessages);
+                    $this->updatePages($updateablePages, $dbQueries, $customMessage);
                     // If the update was successful
-                    if (empty($customMessages)) {
+                    if (empty($customMessage)) {
                         if (count($updateablePages) !== $pagesCount) {
-                            $customMessages = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
+                            $customMessage = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace all the deprecated properties found. Some properties will have to be replaced manually.';
                         }
                     }
                 } else {
-                    $customMessages = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace any of the deprecated properties found. These properties will have to be replaced manually.';
+                    $customMessage = 'Some deprecated Page TSconfig properties were found. However, the wizard was unable to automatically replace any of the deprecated properties found. These properties will have to be replaced manually.';
                 }
             }
         }
         $this->markWizardAsDone();
-        return empty($customMessages);
+        return empty($customMessage);
     }
 
     /**
      * Gets the pages with deprecated RTE properties in TSconfig column
      *
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom message
      * @return array uid and inclusion string for the pages with deprecated RTE properties in TSconfig column
      */
-    protected function getPagesWithDeprecatedRteProperties(&$customMessages)
+    protected function getPagesWithDeprecatedRteProperties(&$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('pages');
         $queryBuilder->getRestrictions()->removeAll();
@@ -136,7 +136,7 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate
                 ->execute()
                 ->fetchAll();
         } catch (DBALException $e) {
-            $customMessages = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
+            $customMessage = 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage());
         }
 
         return [];
@@ -166,9 +166,9 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate
      *
      * @param array $pages Page records to update, fetched by getTemplates() and filtered by
      * @param array $dbQueries Pointer where to insert all DB queries made, so they can be shown to the user if wanted
-     * @param string $customMessages Pointer to output custom messages
+     * @param string $customMessage Pointer to output custom message
      */
-    protected function updatePages($pages, &$dbQueries, &$customMessages)
+    protected function updatePages($pages, &$dbQueries, &$customMessage)
     {
         $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('pages');
         foreach ($pages as $page) {
@@ -183,7 +183,7 @@ class RteAcronymButtonRenamedToAbbreviation extends AbstractUpdate
                     ->set('TSconfig', $page['TSconfig'])
                     ->execute();
             } catch (DBALException $e) {
-                $customMessages .= 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage()) . LF . LF;
+                $customMessage .= 'SQL-ERROR: ' . htmlspecialchars($e->getPrevious()->getMessage()) . LF . LF;
             }
             $dbQueries[] = str_replace(LF, ' ', $queryBuilder->getSQL());
         }