[RELEASE] Version 11.0.1 with minor bug fixes

Author: vertexvaar

CHANGELOG.md

@@ -1,5 +1,22 @@
 # In2publish Core Change Log
 
+11.0.1:
+
+- [META] Set the EM conf version number to 11.0.1
+- [DOCS] Update changelog
+- [META] Set the EM conf version number to 10.2.2
+- [BUGFIX] Show the actual key for differences in site configs
+- [BUGFIX] Check the actual delete field of a table to determine if the record is deleted
+- [BUGFIX] Prevent access on undefined array key
+- [CLEANUP] Strip a superfluous function import as alias
+- [BUGFIX] Actually use the value formatted by its TCA eval field
+- [BUGFIX] Include array key existence check when looking for group/db MM tables
+- [BUGFIX] Format arrays as strings before comparing them with array_diff
+- [BUGFIX] Fix array key for registered tool actions
+- [BUGFIX] Check if deprecated config values are set before evaluation
+- [TYPO] Fix message when there are no envelopes to flush
+- [RELEASE] Version 11.0.0 with TYPO3 v11 compatibility
+
 11.0.0:
 
 - [META] Set the branch alias version number to 11.0.x-dev
@@ -102,6 +119,18 @@
 - [CLEANUP] Remove outdated compatibility class SignalSlotReplacement
 - [!!!][UPDATE] Require TYPO3 v11
 
+10.2.2:
+
+- [META] Set the EM conf version number to 10.2.2
+- [BUGFIX] Show the actual key for differences in site configs
+- [BUGFIX] Check the actual delete field of a table to determine if the record is deleted
+- [BUGFIX] Prevent access on undefined array key
+- [BUGFIX] Include array key existence check when looking for group/db MM tables
+- [BUGFIX] Format arrays as strings before comparing them with array_diff
+- [BUGFIX] Check if deprecated config values are set before evaluation
+- [TYPO] Fix message when there are no envelopes to flush
+- [RELEASE] Version 10.2.1 with a lot of bugfixes
+
 10.2.1:
 
 - [META] Set the EM conf version number to 10.2.1

Classes/Component/RecordHandling/DefaultRecordFinder.php

@@ -1212,7 +1212,7 @@ protected function fetchRelatedRecordsByGroupTypeDb(
             if (in_array($tableName, $excludedTableNames, true)) {
                 return $records;
             }
-            if ($columnConfiguration['MM']) {
+            if (!empty($columnConfiguration['MM'])) {
                 // skip if this record is not the owning side of the relation
                 if (!empty($columnConfiguration['MM_oppositeUsage'])) {
                     return $records;
@@ -1804,7 +1804,7 @@ protected function fetchOriginalRecordsForInlineRecord(
      */
     protected function isIgnoredRecord(array $localProperties, array $foreignProperties, string $tableName): bool
     {
-        if ($this->isDeletedAndUnchangedRecord($localProperties, $foreignProperties)) {
+        if ($this->isDeletedAndUnchangedRecord($localProperties, $foreignProperties, $tableName)) {
             return true;
         }
 
@@ -1851,12 +1851,20 @@ protected function isRemovedAndDeletedRecord(array $localProps, array $foreignPr
      *
      * @param array $localProperties
      * @param array $foreignProperties
+     * @param string $tableName
      *
      * @return bool
      */
-    protected function isDeletedAndUnchangedRecord(array $localProperties, array $foreignProperties): bool
-    {
-        return $localProperties['deleted'] === '1' && !count(array_diff($localProperties, $foreignProperties));
+    protected function isDeletedAndUnchangedRecord(
+        array $localProperties,
+        array $foreignProperties,
+        string $tableName
+    ): bool {
+        $deleteField = $GLOBALS['TCA'][$tableName]['ctrl']['delete'] ?? null;
+        return null !== $deleteField
+               && array_key_exists($deleteField, $localProperties)
+               && $localProperties[$deleteField]
+               && !count(array_diff($localProperties, $foreignProperties));
     }
 
     /**

Classes/Features/AdminTools/Service/ToolsRegistry.php

@@ -84,7 +84,7 @@ public function getEntries(): array
             if ($this->evaluateCondition($config)) {
                 $controller = $config['controller'];
                 $processedTools[$key] = $config;
-                $actions = GeneralUtility::trimExplode(',', $processedTools[$key]['actions']);
+                $actions = GeneralUtility::trimExplode(',', $processedTools[$key]['action']);
                 $processedTools[$key]['initialAction'] = $actions[0];
                 if (isset($controllerConfig[$controller]['alias'])) {
                     $processedTools[$key]['alias'] = $controllerConfig[$controller]['alias'];

Classes/Features/SimplifiedOverviewAndPublishing/Config/Migration/SimplifiedOverviewAndPublishingMigration.php

@@ -43,11 +43,17 @@ class SimplifiedOverviewAndPublishingMigration extends AbstractMigration
     public function migrate(array $config): array
     {
         $enable = false;
-        if ($config['factory']['simpleOverviewAndAjax']) {
+        if (
+            isset($config['factory']['simpleOverviewAndAjax'])
+            && $config['factory']['simpleOverviewAndAjax']
+        ) {
             $this->addMessage(sprintf(self::MESSAGE, 'factory.simpleOverviewAndAjax'));
             $enable = true;
         }
-        if ($config['features']['simplePublishing']['enable']) {
+        if (
+            isset($config['features']['simplePublishing']['enable'])
+            && $config['features']['simplePublishing']['enable']
+        ) {
             $this->addMessage(sprintf(self::MESSAGE, 'features.simplePublishing.enable'));
             $enable = true;
         }

Classes/Testing/Tests/Application/SiteConfigurationTest.php

@@ -42,6 +42,7 @@
 use function array_keys;
 use function array_merge;
 use function array_unique;
+use function json_encode;
 use function sprintf;
 
 class SiteConfigurationTest implements TestCaseInterface
@@ -107,7 +108,7 @@ public function run(): TestResult
                             'direction' => $localLanguage->getDirection(),
                             'typo3Language' => $localLanguage->getTypo3Language(),
                             'fallbackType' => $localLanguage->getFallbackType(),
-                            'fallbackLanguageIds' => $localLanguage->getFallbackLanguageIds(),
+                            'fallbackLanguageIds' => json_encode($localLanguage->getFallbackLanguageIds()),
                             'enabled' => $localLanguage->isEnabled(),
                         ];
 
@@ -120,15 +121,16 @@ public function run(): TestResult
                             'direction' => $foreignLanguage->getDirection(),
                             'typo3Language' => $foreignLanguage->getTypo3Language(),
                             'fallbackType' => $foreignLanguage->getFallbackType(),
-                            'fallbackLanguageIds' => $foreignLanguage->getFallbackLanguageIds(),
+                            'fallbackLanguageIds' => json_encode($foreignLanguage->getFallbackLanguageIds()),
                             'enabled' => $foreignLanguage->isEnabled(),
                         ];
 
                         $differences = array_keys(array_diff($localLanguageArray, $foreignLanguageArray));
                         foreach ($differences as $difference) {
                             $siteErrors[] = sprintf(
-                                'Site %s differences: %s <> %s',
+                                'Site %s differences in %s: %s <> %s',
                                 $siteIdentifier,
+                                $difference,
                                 $localLanguageArray[$difference],
                                 $foreignLanguageArray[$difference]
                             );

Classes/Utility/UriUtility.php

@@ -40,10 +40,9 @@ public static function normalizeUri(UriInterface $uri): UriInterface
         }
         if (empty($uri->getHost())) {
             $path = $uri->getPath();
-            [$host, $path] = explode('/', $path, 2);
-            if (empty($path)) {
-                $path = '';
-            }
+            $parts = explode('/', $path, 2);
+            $host = $parts[0];
+            $path = $parts[1] ?? '';
             $uri = $uri->withHost($host)->withPath($path);
         }
         return $uri;

Classes/ViewHelpers/Tca/FormatPropertyByTcaDefinitionViewHelper.php

@@ -31,7 +31,7 @@
 use TYPO3\CMS\Core\Utility\GeneralUtility;
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
 
-use function nl2br as nl2br1;
+use function nl2br;
 use function strftime;
 use function trim;
 
@@ -63,10 +63,10 @@ public function render(): string
 
         switch ($this->tableConfiguration['config']['type']) {
             case 'input':
-                $this->changeValueForTypeInput($value);
+                $value = $this->changeValueForTypeInput($value);
                 break;
             case 'text':
-                $value = nl2br1($value);
+                $value = nl2br($value);
                 break;
             default:
         }
@@ -80,12 +80,17 @@ public function render(): string
 
     protected function changeValueForTypeInput(string $value): string
     {
-        if (GeneralUtility::inList($this->tableConfiguration['config']['eval'], 'password')) {
+        $eval = $this->tableConfiguration['config']['eval'] ?? null;
+        if (null === $eval) {
+            return $value;
+        }
+
+        if (GeneralUtility::inList($eval, 'password')) {
             return '*******';
         }
         if (
-            GeneralUtility::inList($this->tableConfiguration['config']['eval'], 'datetime')
-            || GeneralUtility::inList($this->tableConfiguration['config']['eval'], 'date')
+            GeneralUtility::inList($eval, 'datetime')
+            || GeneralUtility::inList($eval, 'date')
         ) {
             if ($value !== '0') {
                 $value = strftime('%d.%m.%Y', (int)$value);

Resources/Private/Language/de.locallang.xlf

@@ -395,7 +395,7 @@
 				<target state="translated">Überflüssigen Envelopes löschen.</target>
 			</trans-unit>
 			<trans-unit id="module.m4.nothing_to_flush">
-				<source>There are not envelopes to flush.</source>
+				<source>There are no envelopes to flush.</source>
 				<target state="translated">Es gibt keine Envelopes zum löschen.</target>
 			</trans-unit>
 			<trans-unit id="module.m4.flush_registry">

Resources/Private/Language/locallang.xlf

@@ -299,7 +299,7 @@
 				<source>Flush superfluous envelopes.</source>
 			</trans-unit>
 			<trans-unit id="module.m4.nothing_to_flush">
-				<source>There are not envelopes to flush.</source>
+				<source>There are no envelopes to flush.</source>
 			</trans-unit>
 			<trans-unit id="module.m4.flush_registry">
 				<source>Clear registry entries.</source>

ext_emconf.php

@@ -9,7 +9,7 @@
     'title' => 'in2publish Core',
     'description' => 'Content publishing extension to connect stage and production server',
     'category' => 'plugin',
-    'version' => '11.0.0',
+    'version' => '11.0.1',
     'state' => 'stable',
     'clearCacheOnLoad' => true,
     'author' => 'Alex Kellner, Oliver Eglseder, Thomas Scheibitz, Stefan Busemann',