[TASK] Improve core update error messages 06/67506/2
authorOliver Bartsch <bo@cedev.de>
Fri, 22 Jan 2021 13:32:41 +0000 (14:32 +0100)
committerGeorg Ringer <georg.ringer@gmail.com>
Wed, 27 Jan 2021 13:18:37 +0000 (14:18 +0100)
Performing a core update in the install tool requires
a couple of checks, regarding directory structure
and permissions. On failure, the corresponding error
messages do now contain the information where to
check the directory status within the install tool.

Resolves: #91630
Releases: master, 10.4
Change-Id: I750203c61a0f697838f55b3fd114e5e2326e6df0
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/67506
Tested-by: TYPO3com <noreply@typo3.com>
Tested-by: Jochen <rothjochen@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Jochen <rothjochen@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
typo3/sysext/install/Classes/Service/CoreUpdateService.php

index 82ae2c2..b95fd98 100644 (file)
@@ -141,8 +141,9 @@ class CoreUpdateService
             $success = false;
             $this->messages->enqueue(new FlashMessage(
                 'To perform an update, the folder structure of this TYPO3 CMS instance must'
-                    . ' stick to the conventions, or the update process could lead to unexpected'
-                    . ' results and may be hazardous to your system',
+                . ' stick to the conventions, or the update process could lead to unexpected results'
+                . ' and may be hazardous to your system. Please check your directory status in the'
+                . ' “Environment” module under “Directory Status”.',
                 'Automatic TYPO3 CMS core update not possible: Folder structure has errors or warnings',
                 FlashMessage::ERROR
             ));
@@ -165,7 +166,8 @@ class CoreUpdateService
             if (!$result) {
                 $success = false;
                 $this->messages->enqueue(new FlashMessage(
-                    'Could not write a file in path "' . Environment::getPublicPath() . '/"!',
+                    'Could not write a file in path "' . Environment::getPublicPath() . '/"!'
+                    . ' Please check your directory status in the “Environment” module under “Directory Status”.',
                     'Automatic TYPO3 CMS core update not possible: No write access to document root',
                     FlashMessage::ERROR
                 ));
@@ -176,7 +178,8 @@ class CoreUpdateService
                 if (!is_link($link)) {
                     $success = false;
                     $this->messages->enqueue(new FlashMessage(
-                        'Could not create a symbolic link in path "' . Environment::getPublicPath() . '/"!',
+                        'Could not create a symbolic link in path "' . Environment::getPublicPath() . '/"!'
+                        . ' Please check your directory status in the “Environment” module under “Directory Status”.',
                         'Automatic TYPO3 CMS core update not possible: No symlink creation possible',
                         FlashMessage::ERROR
                     ));
@@ -194,7 +197,8 @@ class CoreUpdateService
                 if (!$result) {
                     $success = false;
                     $this->messages->enqueue(new FlashMessage(
-                        'New TYPO3 CMS core should be installed in "' . $coreLocation . '", but this directory is not writable!',
+                        'New TYPO3 CMS core should be installed in "' . $coreLocation . '", but this directory is not writable!'
+                        . ' Please check your directory status in the “Environment” module under “Directory Status”.',
                         'Automatic TYPO3 CMS core update not possible: No write access to TYPO3 CMS core location',
                         FlashMessage::ERROR
                     ));