[TASK] Mention table and record uid in DataHandler localize command log 72/58072/2
authorTymoteusz Motylewski <t.motylewski@gmail.com>
Wed, 29 Aug 2018 15:49:24 +0000 (17:49 +0200)
committerMarkus Klein <markus.klein@typo3.org>
Wed, 29 Aug 2018 17:50:50 +0000 (19:50 +0200)
Resolves: #86029
Releases: master, 8.7
Change-Id: I75c5bf23537a987fbe16c9a6fb2464a853d12135
Reviewed-on: https://review.typo3.org/58072
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
typo3/sysext/core/Classes/DataHandling/DataHandler.php

index 49dc4fc..aa6fbc8 100644 (file)
@@ -4745,7 +4745,7 @@ class DataHandler
                 || $table === 'pages_language_overlay')
             && $table !== 'pages') {
             if ($this->enableLogging) {
-                $this->newlog('Localization failed; "languageField" and "transOrigPointerField" must be defined for the table!', 1);
+                $this->newlog('Localization failed; "languageField" and "transOrigPointerField" must be defined for the table ' . $table, 1);
             }
             return false;
         }
@@ -4759,7 +4759,7 @@ class DataHandler
 
         if (!$this->doesRecordExist($table, $uid, 'show')) {
             if ($this->enableLogging) {
-                $this->newlog('Attempt to localize record without permission', 1);
+                $this->newlog('Attempt to localize record ' . $table . ':' . $uid . ' without permission.', 1);
             }
             return false;
         }
@@ -4768,7 +4768,7 @@ class DataHandler
         $row = BackendUtility::getRecordWSOL($table, $uid);
         if (!is_array($row)) {
             if ($this->enableLogging) {
-                $this->newlog('Attempt to localize record that did not exist!', 1);
+                $this->newlog('Attempt to localize record ' . $table . ':' . $uid . ' that did not exist!', 1);
             }
             return false;
         }
@@ -4784,7 +4784,7 @@ class DataHandler
             );
             if ((int)$localizationParentRecord[$GLOBALS['TCA'][$table]['ctrl']['languageField']] !== 0) {
                 if ($this->enableLogging) {
-                    $this->newlog('Localization failed; Source record contained a reference to an original record that is not a default record (which is strange)!', 1);
+                    $this->newlog('Localization failed; Source record ' . $table . ':' . $localizationParentRecord['uid'] . ' contained a reference to an original record that is not a default record (which is strange)!', 1);
                 }
                 return false;
             }
@@ -4795,7 +4795,7 @@ class DataHandler
             && (int)$row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] === 0
             && $table !== 'pages') {
             if ($this->enableLogging) {
-                $this->newlog('Localization failed; Source record contained a reference to an original default record but is a default record itself (which is strange)!', 1);
+                $this->newlog('Localization failed; Source record ' . $table . ':' . $row['uid'] . ' contained a reference to an original default record but is a default record itself (which is strange)!', 1);
             }
             return false;
         }
@@ -4832,7 +4832,7 @@ class DataHandler
 
         if (!$pass) {
             if ($this->enableLogging) {
-                $this->newlog('Localization failed; There already was a localization for this language of the record!', 1);
+                $this->newlog('Localization failed; There already was a localization for this language of the record ' . $table . ':' . $uid . '!', 1);
             }
             return false;
         }