From af2cfd4a46ef4503c39eb0f62ea6f76c950cc942 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Christian=20E=C3=9Fl?= <indy.essl@gmail.com>
Date: Fri, 11 Oct 2019 15:58:38 +0200
Subject: [PATCH] [BUGFIX] Do not try to persist fieldNames of type "none"
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
If a TCA field config of type="none" with the l10n_mode="exclude" is
present, the DataHandler will try to persist the field when saving a
translated version of the record, even though fields of type="none"
must not be persisted.
Resolves: #87781
Releases: master, 9.5
Change-Id: Ie99495ff95fcf1ceaac0c5d050653e21d3af2afd
Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/61960
Tested-by: TYPO3com <noreply@typo3.com>
Tested-by: Oliver Bartsch <bo@cedev.de>
Tested-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Oliver Bartsch <bo@cedev.de>
Reviewed-by: Guido Schmechel <guido.schmechel@brandung.de>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Jörg Bösche <typo3@joergboesche.de>
Reviewed-by: Henning Liebe <h.liebe@neusta.de>
Reviewed-by: Felix P. <f.pachowsky@neusta.de>
Reviewed-by: Steffen Frese <steffenf14@gmail.com>
Reviewed-by: Tobi Kretschmann <tobi@tobishome.de>
Reviewed-by: Sascha Rademacher <sascha.rademacher+typo3@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
---
.../Classes/DataHandling/Localization/DataMapProcessor.php | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/typo3/sysext/core/Classes/DataHandling/Localization/DataMapProcessor.php b/typo3/sysext/core/Classes/DataHandling/Localization/DataMapProcessor.php
index 1659835d38e6..89d06f48e2ec 100644
--- a/typo3/sysext/core/Classes/DataHandling/Localization/DataMapProcessor.php
+++ b/typo3/sysext/core/Classes/DataHandling/Localization/DataMapProcessor.php
@@ -1401,7 +1401,9 @@ class DataMapProcessor
}
foreach ($GLOBALS['TCA'][$tableName]['columns'] as $fieldName => $configuration) {
- if (($configuration['l10n_mode'] ?? null) === 'exclude') {
+ if (($configuration['l10n_mode'] ?? null) === 'exclude'
+ && ($configuration['config']['type'] ?? null) !== 'none'
+ ) {
$localizationExcludeFieldNames[] = $fieldName;
}
}
--
GitLab