[TASK] Add missing breaking changes documentation 14/42814/9
authorAlexander Schnitzler <git@alexanderschnitzler.de>
Fri, 21 Aug 2015 14:04:19 +0000 (16:04 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Tue, 25 Aug 2015 17:32:08 +0000 (19:32 +0200)
This patch adds missing documentation for the breaking changes
introduced in https://review.typo3.org/42644.

It also adds an early return for the FileListIsStartModuleUpdate
to prevent unnecessary database operations.

Releases: master
Resolves: #69237
Change-Id: Iaa113741919013b8a0a9513c1dcca2e5db094b04
Reviewed-on: http://review.typo3.org/42814
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/core/Documentation/Changelog/master/Breaking-69083-RenamedIdentifierForFilenameModule.rst [new file with mode: 0644]
typo3/sysext/install/Classes/Updates/FileListIsStartModuleUpdate.php

diff --git a/typo3/sysext/core/Documentation/Changelog/master/Breaking-69083-RenamedIdentifierForFilenameModule.rst b/typo3/sysext/core/Documentation/Changelog/master/Breaking-69083-RenamedIdentifierForFilenameModule.rst
new file mode 100644 (file)
index 0000000..fbd0f73
--- /dev/null
@@ -0,0 +1,28 @@
+=========================================================
+Breaking: #69083 - Renamed identifier for filelist module
+=========================================================
+
+Description
+===========
+
+The filelist module was rewritten based on extbase. Therefore the module identifier was changed
+from ``file_list`` to ``file_FilelistList``.
+
+
+Impact
+======
+
+All links pointing to the filelist module using the old identifier will break.
+
+
+Affected Installations
+======================
+
+All installations that reference the filelist module by its old name.
+
+
+Migration
+=========
+
+There is a upgrade wizard to change the backend user settings of users whose start module is the filelist module.
+All other links to the module have to be changed manually to use ``file_FilelistList`` as module identifier.
index e6d0ea6..44bb1d5 100644 (file)
 namespace TYPO3\CMS\Install\Updates;
 
 /**
- * Class FileListIsStartModuleUpdate
  * Update backend user setting startModule if set to "file_list"
- *
- * @package TYPO3\CMS\Install\Updates
  */
 class FileListIsStartModuleUpdate extends AbstractUpdate {
 
@@ -34,8 +31,11 @@ class FileListIsStartModuleUpdate extends AbstractUpdate {
         * @return bool Whether an update is needed (TRUE) or not (FALSE)
         */
        public function checkForUpdate(&$description) {
-               $backendUsersCount = $this->getDatabaseConnection()->exec_SELECTcountRows('uid', 'be_users');
-               if ($this->isWizardDone() || $backendUsersCount === 0) {
+               if ($this->isWizardDone()) {
+                       return FALSE;
+               }
+
+               if ($this->getDatabaseConnection()->exec_SELECTcountRows('uid', 'be_users') === 0) {
                        return FALSE;
                }