[BUGFIX] Install tool: Empty messages in TCA migration 42/53742/3
authorChristian Kuhn <lolli@schwarzbu.ch>
Sun, 20 Aug 2017 08:43:10 +0000 (10:43 +0200)
committerChristian Kuhn <lolli@schwarzbu.ch>
Mon, 21 Aug 2017 09:36:42 +0000 (11:36 +0200)
The JsonView should return the detail message text as
'message' array and not as 'status'.

Change-Id: I492d31ee8fe3895c97e0229369c5f3ca09f7b713
Resolves: #82137
Related: #76084
Releases: master
Reviewed-on: https://review.typo3.org/53742
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Reviewed-by: Sascha Egerer <sascha@sascha-egerer.de>
Reviewed-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Tested-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Tested-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
typo3/sysext/install/Classes/Status/AbstractStatus.php
typo3/sysext/install/Resources/Public/JavaScript/Modules/Presets.js
typo3/sysext/install/Tests/Unit/Status/AbstractStatusTest.php

index 15586be..4fc971d 100644 (file)
@@ -110,7 +110,7 @@ abstract class AbstractStatus implements StatusInterface
         return [
             'severity' => $this->getSeverityNumber(),
             'title' => $this->getTitle(),
-            'status' => $this->getMessage()
+            'message' => $this->getMessage(),
         ];
     }
 
index f1ec0a6..64befb3 100644 (file)
@@ -64,7 +64,7 @@ define(['jquery', 'TYPO3/CMS/Install/FlashMessage', 'TYPO3/CMS/Install/ProgressB
                                        $outputContainer.empty();
                                        if (data.success === true && Array.isArray(data.status)) {
                                                data.status.forEach(function(element) {
-                                                       var message = InfoBox.render(element.severity, element.title, element.status);
+                                                       var message = InfoBox.render(element.severity, element.title, element.message);
                                                        $outputContainer.append(message);
                                                });
                                        }
index 41cca39..7c9f0f4 100644 (file)
@@ -43,7 +43,7 @@ class AbstractStatusTest extends UnitTestCase
         $expected = [
             'severity' => 1,
             'title' => 'aTitle',
-            'status' => 'aMessage',
+            'message' => 'aMessage',
         ];
         $this->assertEquals($expected, $status->jsonSerialize());
     }