[TASK] Improve TranslateViewHelper compiling
[Packages/TYPO3.CMS.git] / typo3 / sysext / fluid / Classes / ViewHelpers / TranslateViewHelper.php
index dedb94b..0f56911 100644 (file)
@@ -16,9 +16,9 @@ namespace TYPO3\CMS\Fluid\ViewHelpers;
 
 use TYPO3\CMS\Extbase\Utility\LocalizationUtility;
 use TYPO3\CMS\Fluid\Core\ViewHelper\Exception\InvalidVariableException;
-use TYPO3Fluid\Fluid\Core\Rendering\RenderingContextInterface;
+use TYPO3Fluid\Fluid\Core\Compiler\TemplateCompiler;
+use TYPO3Fluid\Fluid\Core\Parser\SyntaxTree\ViewHelperNode;
 use TYPO3Fluid\Fluid\Core\ViewHelper\AbstractViewHelper;
-use TYPO3Fluid\Fluid\Core\ViewHelper\Traits\CompileWithRenderStatic;
 
 /**
  * Translate a key from locallang. The files are loaded from the folder
@@ -73,8 +73,6 @@ use TYPO3Fluid\Fluid\Core\ViewHelper\Traits\CompileWithRenderStatic;
  */
 class TranslateViewHelper extends AbstractViewHelper
 {
-    use CompileWithRenderStatic;
-
     /**
      * Output is escaped already. We must not escape children, to avoid double encoding.
      *
@@ -99,43 +97,65 @@ class TranslateViewHelper extends AbstractViewHelper
     }
 
     /**
-     * Return array element by key.
-     *
-     * @param array $arguments
-     * @param \Closure $renderChildrenClosure
-     * @param RenderingContextInterface $renderingContext
-     * @throws InvalidVariableException
+     * @param string $argumentsName
+     * @param string $closureName
+     * @param string $initializationPhpCode
+     * @param ViewHelperNode $node
+     * @param TemplateCompiler $compiler
      * @return string
      */
-    public static function renderStatic(array $arguments, \Closure $renderChildrenClosure, RenderingContextInterface $renderingContext)
+    public function compile($argumentsName, $closureName, &$initializationPhpCode, ViewHelperNode $node, TemplateCompiler $compiler)
+    {
+        return sprintf(
+            '\\%s::translate(%s[\'key\'] ?? %s[\'id\'], %s[\'extensionName\'] ?? $renderingContext->getControllerContext()->getRequest()->getControllerExtensionName(), %s[\'arguments\'], %s[\'languageKey\'], %s[\'alternativeLanguageKeys\']) ?? %s[\'default\'] ?? %s()',
+            LocalizationUtility::class,
+            $argumentsName,
+            $argumentsName,
+            $argumentsName,
+            $argumentsName,
+            $argumentsName,
+            $argumentsName,
+            $argumentsName,
+            $closureName
+        );
+    }
+
+    /**
+     * @return string|null
+     */
+    public function render()
     {
-        $key = $arguments['key'];
-        $id = $arguments['id'];
-        $default = $arguments['default'];
-        $extensionName = $arguments['extensionName'];
-        $translateArguments = $arguments['arguments'];
+        $key = $this->arguments['key'];
+        $id = $this->arguments['id'];
+        $default = $this->arguments['default'];
+        $extensionName = $this->arguments['extensionName'];
+        $arguments = $this->arguments['arguments'];
+        $languageKey = $this->arguments['languageKey'];
+        $alternativeLanguageKeys = $this->arguments['alternativeLanguageKeys'];
+
+        if (empty($id) && empty($key)) {
+            throw new InvalidVariableException('Either "key" or "id" must be provided for f:translate', 1351584844);
+        }
 
         // Wrapper including a compatibility layer for TYPO3 Flow Translation
         if ($id === null) {
             $id = $key;
         }
 
-        if ((string)$id === '') {
-            throw new InvalidVariableException('An argument "key" or "id" has to be provided', 1351584844);
-        }
-
-        $request = $renderingContext->getControllerContext()->getRequest();
-        $extensionName = $extensionName ?? $request->getControllerExtensionName();
+        $request = $this->renderingContext->getControllerContext()->getRequest();
         try {
-            $value = static::translate($id, $extensionName, $translateArguments, $arguments['languageKey'], $arguments['alternativeLanguageKeys']);
+            $value = $this->translate(
+                $key ?? $id,
+                $extensionName ?? $request->getControllerExtensionName(),
+                $arguments,
+                $languageKey,
+                $alternativeLanguageKeys
+            );
         } catch (\InvalidArgumentException $e) {
             $value = null;
         }
         if ($value === null) {
-            $value = $default ?? $renderChildrenClosure();
-            if (!empty($translateArguments)) {
-                $value = vsprintf($value, $translateArguments);
-            }
+            $value = $default !== null ? $default : $this->renderChildren();
         }
         return $value;
     }
@@ -151,7 +171,7 @@ class TranslateViewHelper extends AbstractViewHelper
      *
      * @return string|null
      */
-    protected static function translate($id, $extensionName, $arguments, $languageKey, $alternativeLanguageKeys)
+    protected function translate($id, $extensionName, $arguments, $languageKey, $alternativeLanguageKeys)
     {
         return LocalizationUtility::translate($id, $extensionName, $arguments, $languageKey, $alternativeLanguageKeys);
     }