Revert "[BUGFIX] Properly handle translation file detection" 66/37166/2
authorMarkus Klein <klein.t3@reelworx.at>
Tue, 24 Feb 2015 14:21:24 +0000 (15:21 +0100)
committerMarkus Klein <klein.t3@reelworx.at>
Tue, 24 Feb 2015 14:22:45 +0000 (15:22 +0100)
This reverts commit 81ec05a3fc39845426f302412d53ef6480fc7c3a.

The patch is reverted as it was merged in a merge-freeze
by accident.
Backport has not been merged yet.

Change-Id: I805393f32e9ddf0c635d492a298ec0c3d2d50132
Releases: master
Reviewed-on: http://review.typo3.org/37166
Reviewed-by: Markus Klein <klein.t3@reelworx.at>
Tested-by: Markus Klein <klein.t3@reelworx.at>
typo3/sysext/core/Classes/Localization/Parser/LocallangXmlParser.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php

index c1f3edb..2d5cff5 100644 (file)
@@ -14,9 +14,6 @@ namespace TYPO3\CMS\Core\Localization\Parser;
  * The TYPO3 project - inspiring people to share!
  */
 
-use TYPO3\CMS\Core\Localization\Exception\InvalidXmlFileException;
-use TYPO3\CMS\Core\Utility\GeneralUtility;
-
 /**
  * Parser for XML locallang file.
  *
@@ -46,21 +43,11 @@ class LocallangXmlParser extends AbstractXmlParser {
                // Parse source
                $parsedSource = $this->parseXmlFile();
                // Parse target
-               $targetPath = $this->sourcePath;
-               if ($this->languageKey !== 'default') {
-                       $localizedTargetPath = GeneralUtility::getFileAbsFileName(GeneralUtility::llXmlAutoFileName($this->sourcePath, $this->languageKey));
-                       if (!@is_file($localizedTargetPath)) {
-                               // Global localization is not available, try split localization file
-                               $localizedTargetPath = GeneralUtility::getFileAbsFileName(GeneralUtility::llXmlAutoFileName($this->sourcePath, $this->languageKey, TRUE));
-                       }
-                       if (!@is_file($localizedTargetPath)) {
-                               $localizedTargetPath = $this->sourcePath;
-                       }
-                       $targetPath = $localizedTargetPath;
-               }
+               $localizedTargetPath = \TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName(\TYPO3\CMS\Core\Utility\GeneralUtility::llXmlAutoFileName($this->sourcePath, $this->languageKey));
+               $targetPath = $this->languageKey !== 'default' && @is_file($localizedTargetPath) ? $localizedTargetPath : $this->sourcePath;
                try {
                        $parsedTarget = $this->getParsedTargetData($targetPath);
-               } catch (InvalidXmlFileException $e) {
+               } catch (\TYPO3\CMS\Core\Localization\Exception\InvalidXmlFileException $e) {
                        $parsedTarget = $this->getParsedTargetData($this->sourcePath);
                }
                $LOCAL_LANG = array();
@@ -72,7 +59,7 @@ class LocallangXmlParser extends AbstractXmlParser {
        /**
         * Returns array representation of XLIFF data, starting from a root node.
         *
-        * @param \SimpleXMLElement $root XML root element
+        * @param SimpleXMLElement $root XML root element
         * @param string $element Target or Source
         * @return array
         */
@@ -112,7 +99,7 @@ class LocallangXmlParser extends AbstractXmlParser {
                        // <languageKey index="fr">EXT:yourext/path/to/localized/locallang.xml</languageKey>
                        $reference = sprintf('%s', $bodyOfFileTag);
                        if (substr($reference, -4) === '.xml') {
-                               return $this->getParsedTargetData(GeneralUtility::getFileAbsFileName($reference));
+                               return $this->getParsedTargetData(\TYPO3\CMS\Core\Utility\GeneralUtility::getFileAbsFileName($reference));
                        }
                }
                /** @var \SimpleXMLElement $translationElement */
@@ -167,7 +154,7 @@ class LocallangXmlParser extends AbstractXmlParser {
         *
         * @param string $targetPath Path of the target file
         * @return array
-        * @throws InvalidXmlFileException
+        * @throws \TYPO3\CMS\Core\Localization\Exception\InvalidXmlFileException
         */
        protected function parseXmlTargetFile($targetPath) {
                $rootXmlNode = FALSE;
@@ -175,7 +162,7 @@ class LocallangXmlParser extends AbstractXmlParser {
                        $rootXmlNode = simplexml_load_file($targetPath, 'SimpleXmlElement', \LIBXML_NOWARNING);
                }
                if (!isset($rootXmlNode) || $rootXmlNode === FALSE) {
-                       throw new InvalidXmlFileException('The path provided does not point to existing and accessible well-formed XML file (' . $targetPath . ').', 1278155987);
+                       throw new \TYPO3\CMS\Core\Localization\Exception\InvalidXmlFileException('The path provided does not point to existing and accessible well-formed XML file (' . $targetPath . ').', 1278155987);
                }
                return $this->doParsingTargetFromRoot($rootXmlNode);
        }
index f662a8a..43e3069 100755 (executable)
@@ -3979,24 +3979,18 @@ Connection: close
        /**
         * Returns auto-filename for locallang-XML localizations.
         *
-        * @param string $fileRef Absolute file reference to locallang-XML file.
+        * @param string $fileRef Absolute file reference to locallang-XML file. Must be inside system/global/local extension
         * @param string $language Language key
         * @param bool $sameLocation if TRUE, then locallang-XML localization file name will be returned with same directory as $fileRef
-        * @return string Returns the filename reference for the language unless error occurred in which case it will be NULL
+        * @return string Returns the filename reference for the language unless error occurred (or local mode is used) in which case it will be NULL
         */
        static public function llXmlAutoFileName($fileRef, $language, $sameLocation = FALSE) {
-               // If $fileRef is already prefixed with "[language key]" then we should return it as this
-               $fileName = basename($fileRef);
-               if (self::isFirstPartOfStr($fileName, $language . '.')) {
-                       return $fileRef;
-               }
-
                if ($sameLocation) {
-                       return str_replace($fileName, $language . '.' . $fileName, $fileRef);
+                       $location = 'EXT:';
+               } else {
+                       // Default location of translations
+                       $location = 'typo3conf/l10n/' . $language . '/';
                }
-
-               // Default location of translations
-               $location = 'typo3conf/l10n/' . $language . '/';
                // Analyse file reference:
                // Is system:
                if (self::isFirstPartOfStr($fileRef, PATH_typo3 . 'sysext/')) {
@@ -4023,6 +4017,10 @@ Connection: close
                        }
                        // Add empty first-entry if not there.
                        list($file_extPath, $file_fileName) = $temp;
+                       // If $fileRef is already prefix with "[language key]" then we should return it as this
+                       if (substr($file_fileName, 0, strlen($language) + 1) === $language . '.') {
+                               return $fileRef;
+                       }
                        // The filename is prefixed with "[language key]." because it prevents the llxmltranslate tool from detecting it.
                        return $location . $file_extKey . '/' . ($file_extPath ? $file_extPath . '/' : '') . $language . '.' . $file_fileName;
                } else {