Fixed translation configuration

ibexa · Aug 22, 2023 · f19d065 · f19d065
1 parent d946511
commit f19d065
Show file tree

Hide file tree

Showing 11 changed files with 227 additions and 357 deletions.
diff --git a/src/bundle/Core/DependencyInjection/IbexaCoreExtension.php b/src/bundle/Core/DependencyInjection/IbexaCoreExtension.php
@@ -1021,10 +1021,10 @@ private function prependJMSTranslation(ContainerBuilder $container): void
             'configs' => [
                 'ibexa_core' => [
                     'dirs' => [
-                        __DIR__ . '/../',
+                        __DIR__ . '/../../../',
                     ],
                     'output_dir' => __DIR__ . '/../Resources/translations/',
-                    'output_format' => 'xliff',
+                    'output_format' => 'xlf',
                     'excluded_dirs' => ['Behat', 'Tests', 'node_modules', 'Features'],
                 ],
             ],

diff --git a/src/bundle/Core/Resources/translations/content_fields.en.xlf b/src/bundle/Core/Resources/translations/content_fields.en.xlf
@@ -8,12 +8,12 @@
     <body>
       <trans-unit id="f337a8fd9042dbc4de8b5d305b6d8df3c0c374c4" resname="content-field.latitude.not_set">
         <source>Not set</source>
-        <target state="new">Not set</target>
+        <target>Not set</target>
         <note>key: content-field.latitude.not_set</note>
       </trans-unit>
       <trans-unit id="687973ca20983f27a99c50243e364e346c9c4814" resname="content-field.longitude.not_set">
         <source>Not set</source>
-        <target state="new">Not set</target>
+        <target>Not set</target>
         <note>key: content-field.longitude.not_set</note>
       </trans-unit>
     </body>

diff --git a/src/bundle/Core/Resources/translations/fielddefinition.en.xlf b/src/bundle/Core/Resources/translations/fielddefinition.en.xlf
diff --git a/...ore/Resources/translations/forms.en.xliff → .../Core/Resources/translations/forms.en.xlf b/...ore/Resources/translations/forms.en.xliff → .../Core/Resources/translations/forms.en.xlf
diff --git a/src/bundle/Core/Resources/translations/messages.en.xlf b/src/bundle/Core/Resources/translations/messages.en.xlf
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <xliff xmlns="urn:oasis:names:tc:xliff:document:1.2" xmlns:jms="urn:jms:translation" version="1.2">
-  <file date="2017-01-26T14:55:58Z" source-language="en" target-language="en" datatype="plaintext" original="not.available">
+  <file source-language="en" target-language="en" datatype="plaintext" original="not.available">
     <header>
       <tool tool-id="JMSTranslationBundle" tool-name="JMSTranslationBundle" tool-version="1.1.0-DEV"/>
       <note>The source node in most cases contains the sample message as written by the developer. If it looks like a dot-delimitted string such as "form.label.firstname", then the developer has not provided a default message.</note>
@@ -10,32 +10,26 @@
         <source>Enter login or email</source>
         <target>Enter login or email</target>
         <note>key: Enter login or email</note>
-        <jms:reference-file line="18">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
       </trans-unit>
       <trans-unit id="570591c060a999e9ab1592763c74d881b7654ee1" resname="Enter password">
         <source>Enter password</source>
         <target>Enter password</target>
         <note>key: Enter password</note>
-        <jms:reference-file line="22">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
       </trans-unit>
       <trans-unit id="4e5a2893bdcc7d239c1db72e4c4ffbe4bea73174" resname="Login">
         <source>Login</source>
         <target>Login</target>
         <note>key: Login</note>
-        <jms:reference-file line="32">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
-        <jms:reference-file line="6">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
       </trans-unit>
       <trans-unit id="be81ab98cb9ccf753975f50fa70cd32581a821fc" resname="Password:">
         <source>Password:</source>
         <target>Password:</target>
         <note>key: Password:</note>
-        <jms:reference-file line="21">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
       </trans-unit>
       <trans-unit id="17dfb031e086e3415d71f3928e6ce08d6b1c1d4d" resname="Username:">
         <source>Username:</source>
         <target>Username:</target>
         <note>key: Username:</note>
-        <jms:reference-file line="17">src/bundle/Core/Resources/views/Security/login.html.twig</jms:reference-file>
       </trans-unit>
     </body>
   </file>

diff --git a/src/bundle/Core/Resources/translations/repository_exceptions.en.xlf b/src/bundle/Core/Resources/translations/repository_exceptions.en.xlf
diff --git a/src/lib/Base/Exceptions/ContentTypeValidationException.php b/src/lib/Base/Exceptions/ContentTypeValidationException.php
@@ -9,6 +9,7 @@
 use Ibexa\Contracts\Core\Repository\Exceptions\ContentTypeValidationException as APIContentTypeValidationException;
 use Ibexa\Core\Base\Translatable;
 use Ibexa\Core\Base\TranslatableBase;
+use JMS\TranslationBundle\Annotation\Ignore;
 
 /**
  * This Exception is thrown on create or update content type when content type is not valid.
@@ -25,8 +26,7 @@ class ContentTypeValidationException extends APIContentTypeValidationException i
      */
     public function __construct($messageTemplate, array $parameters = [])
     {
-        /** @Ignore */
-        $this->setMessageTemplate($messageTemplate);
+        $this->setMessageTemplate(/** @Ignore */$messageTemplate);
         $this->setParameters($parameters);
 
         parent::__construct($this->getBaseTranslation());

diff --git a/src/lib/Base/Exceptions/ContentValidationException.php b/src/lib/Base/Exceptions/ContentValidationException.php
@@ -9,6 +9,7 @@
 use Ibexa\Contracts\Core\Repository\Exceptions\ContentValidationException as APIContentValidationException;
 use Ibexa\Core\Base\Translatable;
 use Ibexa\Core\Base\TranslatableBase;
+use JMS\TranslationBundle\Annotation\Ignore;
 
 /**
  * This Exception is thrown on create or update content one or more given fields are not valid.
@@ -25,8 +26,7 @@ class ContentValidationException extends APIContentValidationException implement
      */
     public function __construct($messageTemplate, array $parameters = [])
     {
-        /** @Ignore */
-        $this->setMessageTemplate($messageTemplate);
+        $this->setMessageTemplate(/** @Ignore */$messageTemplate);
         $this->setParameters($parameters);
 
         parent::__construct($this->getBaseTranslation());

diff --git a/src/lib/Helper/FieldsGroups/ArrayTranslatorFieldsGroupsList.php b/src/lib/Helper/FieldsGroups/ArrayTranslatorFieldsGroupsList.php
@@ -7,6 +7,7 @@
 namespace Ibexa\Core\Helper\FieldsGroups;
 
 use Ibexa\Contracts\Core\Repository\Values\ContentType\FieldDefinition;
+use JMS\TranslationBundle\Annotation\Ignore;
 use Symfony\Contracts\Translation\TranslatorInterface;
 
 /**
@@ -39,6 +40,7 @@ public function getGroups()
 
         foreach ($this->groups as $groupIdentifier) {
             $translatedGroups[$groupIdentifier] = $this->translator->trans(
+                /** @Ignore */
                 $groupIdentifier,
                 [],
                 'ezplatform_fields_groups'

diff --git a/src/lib/MVC/Symfony/Templating/Twig/Extension/FileSizeExtension.php b/src/lib/MVC/Symfony/Templating/Twig/Extension/FileSizeExtension.php
@@ -8,6 +8,7 @@
 
 use Ibexa\Contracts\Core\SiteAccess\ConfigResolverInterface;
 use Ibexa\Core\MVC\Symfony\Locale\LocaleConverterInterface;
+use JMS\TranslationBundle\Annotation\Ignore;
 use Locale;
 use NumberFormatter;
 use Symfony\Contracts\Translation\TranslatorInterface;
@@ -110,7 +111,9 @@ public function sizeFilter($number, $precision)
             $i = ($index - 1);
         }
         $formatter = new NumberFormatter($this->getLocale(), NumberFormatter::PATTERN_DECIMAL);
-        $formatter->setPattern($formatter->getPattern() . ' ' . $this->translator->trans($this->suffixes[$i]));
+        $formatter->setPattern(
+            $formatter->getPattern() . ' ' . $this->translator->trans(/** @Ignore */$this->suffixes[$i])
+        );
         $formatter->setAttribute(NumberFormatter::MAX_FRACTION_DIGITS, $precision);
 
         return $formatter->format($number);

diff --git a/src/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitor.php b/src/lib/MVC/Symfony/Translation/ExceptionMessageTemplateFileVisitor.php
@@ -7,6 +7,7 @@
 namespace Ibexa\Core\MVC\Symfony\Translation;
 
 use Doctrine\Common\Annotations\DocParser;
+use JMS\TranslationBundle\Annotation\Ignore;
 use JMS\TranslationBundle\Model\Message;
 use JMS\TranslationBundle\Model\MessageCatalogue;
 use JMS\TranslationBundle\Translation\Extractor\File\DefaultPhpFileExtractor;
@@ -34,10 +35,15 @@ class ExceptionMessageTemplateFileVisitor extends DefaultPhpFileExtractor
 
     private MessageCatalogue $catalogue;
 
+    private Node $previousNode;
+
+    private DocParser $docParser;
+
     public function __construct(DocParser $docParser, FileSourceFactory $fileSourceFactory)
     {
         parent::__construct($docParser, $fileSourceFactory);
         $this->fileSourceFactory = $fileSourceFactory;
+        $this->docParser = $docParser;
         $this->traverser = new NodeTraverser();
         $this->traverser->addVisitor($this);
     }
@@ -52,10 +58,18 @@ public function enterNode(Node $node): void
             !is_string($methodCallNodeName)
             || !array_key_exists($methodCallNodeName, $this->methodsToExtractFrom)
         ) {
+            $this->previousNode = $node;
+
             return;
         }
 
+        $ignore = $this->isIgnore($node);
+
         if (!$node->args[0]->value instanceof String_) {
+            if ($ignore) {
+                return;
+            }
+
             $message = sprintf('Can only extract the translation id from a scalar string, but got "%s". Please refactor your code to make it extractable, or add the doc comment /** @Ignore */ to this code element (in %s on line %d).', get_class($node->args[0]->value), $this->file, $node->args[0]->value->getLine());
 
             $this->logger->error($message);
@@ -76,6 +90,42 @@ public function visitPhpFile(SplFileInfo $file, MessageCatalogue $catalogue, arr
         $this->catalogue = $catalogue;
         $this->traverser->traverse($ast);
     }
+
+    private function getDocCommentForNode(Node $node): ?string
+    {
+        if (null !== $comment = $node->args[0]->getDocComment()) {
+            return $comment->getText();
+        }
+
+        if (null !== $comment = $node->getDocComment()) {
+            return $comment->getText();
+        }
+
+        if (null !== $this->previousNode && $this->previousNode->getDocComment() !== null) {
+            $comment = $this->previousNode->getDocComment();
+
+            return is_object($comment) ? $comment->getText() : $comment;
+        }
+
+        return null;
+    }
+
+    private function isIgnore($node): bool
+    {
+        if (null !== $docComment = $this->getDocCommentForNode($node)) {
+            $annotations = $this->docParser->parse(
+                $docComment,
+                'file ' . $this->file . ' near line ' . $node->getLine()
+            );
+            foreach ($annotations as $annot) {
+                if ($annot instanceof Ignore) {
+                    return true;
+                }
+            }
+        }
+
+        return false;
+    }
 }
 
 class_alias(ExceptionMessageTemplateFileVisitor::class, 'eZ\Publish\Core\MVC\Symfony\Translation\ExceptionMessageTemplateFileVisitor');