[BUGFIX] Do not use array assignment for string variable 08/51608/6
authorSascha Egerer <s.egerer@syzygy.de>
Thu, 9 Feb 2017 11:35:03 +0000 (12:35 +0100)
committerMarkus Klein <markus.klein@typo3.org>
Fri, 10 Feb 2017 09:46:45 +0000 (10:46 +0100)
In PHP 7.1 it is not possible anymore to assign a value to
a variable of type string with `[]` appended. In older PHP
versions the variables was silently converted to an array but
in PHP 7.1 a warning is thrown.
Beside that the variable must be of type string anyway and not
an array.

Resolves: #79709
Releases: master
Change-Id: Ie36dcf89cff23e7837d1f559cd05b3f9f448360e
Reviewed-on: https://review.typo3.org/51608
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Johannes Kasberger <johannes.kasberger@reelworx.at>
Tested-by: Johannes Kasberger <johannes.kasberger@reelworx.at>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
typo3/sysext/install/Classes/Updates/CommandLineBackendUserRemovalUpdate.php

index 540d03a..9492922 100644 (file)
@@ -63,10 +63,10 @@ class CommandLineBackendUserRemovalUpdate extends AbstractUpdate
      * Performs the database update to set all be_users starting with _CLI_* to deleted
      *
      * @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)
     {
         $usersFound = $this->getUnneededCommandLineUsers();
         foreach ($usersFound as $userUid => $username) {
@@ -84,7 +84,7 @@ class CommandLineBackendUserRemovalUpdate extends AbstractUpdate
                 ->execute();
             $databaseQueries[] = $queryBuilder->getSQL();
         }
-        $customMessages[] = '<p>The following backend users have been deleted:</p><ul><li>' . implode('</li><li>', $usersFound) . '</li></ul>';
+        $customMessage = '<p>The following backend users have been deleted:</p><ul><li>' . implode('</li><li>', $usersFound) . '</li></ul>';
         $this->markWizardAsDone();
         return true;
     }