Add a way to avoid throwing Exceptions

Author: Benoît Burnichon

Diff for: Translation/Extractor/File/TwigFileExtractor.php

@@ -19,20 +19,24 @@
 namespace JMS\TranslationBundle\Translation\Extractor\File;
 
 use JMS\TranslationBundle\Exception\RuntimeException;
-use Symfony\Bridge\Twig\Node\TransNode;
-
+use JMS\TranslationBundle\Logger\LoggerAwareInterface;
 use JMS\TranslationBundle\Model\FileSource;
 use JMS\TranslationBundle\Model\Message;
 use JMS\TranslationBundle\Model\MessageCatalogue;
 use JMS\TranslationBundle\Translation\Extractor\FileVisitorInterface;
+use Symfony\Bridge\Twig\Node\TransNode;
+use Symfony\Component\HttpKernel\Log\LoggerInterface;
 
-class TwigFileExtractor implements FileVisitorInterface, \Twig_NodeVisitorInterface
+class TwigFileExtractor implements FileVisitorInterface, \Twig_NodeVisitorInterface, LoggerAwareInterface
 {
+    /** @var \SplFileInfo */
     private $file;
     private $catalogue;
     private $traverser;
     private $stack = array();
     private $stackCount = 0;
+    /** @var LoggerInterface */
+    private $logger;
 
     public function __construct(\Twig_Environment $env)
     {
@@ -47,6 +51,9 @@ public function enterNode(\Twig_NodeInterface $node, \Twig_Environment $env)
             $id = $node->getNode('body')->getAttribute('data');
             $domain = 'messages';
             if (null !== $domainNode = $node->getNode('domain')) {
+                if (!$this->checkNodeIsConstant($domainNode)) {
+                    return $node;
+                }
                 $domain = $domainNode->getAttribute('value');
             }
 
@@ -58,10 +65,8 @@ public function enterNode(\Twig_NodeInterface $node, \Twig_Environment $env)
 
             if ('trans' === $name || 'transchoice' === $name) {
                 $idNode = $node->getNode('node');
-                if (!$idNode instanceof \Twig_Node_Expression_Constant) {
+                if (!$this->checkNodeIsConstant($idNode)) {
                     return $node;
-                    // FIXME: see below
-//                     throw new \RuntimeException(sprintf('Cannot infer translation id from node "%s". Please refactor to only translate constants.', get_class($idNode)));
                 }
                 $id = $idNode->getAttribute('value');
 
@@ -70,10 +75,8 @@ public function enterNode(\Twig_NodeInterface $node, \Twig_Environment $env)
                 $arguments = $node->getNode('arguments');
                 if ($arguments->hasNode($index)) {
                     $argument = $arguments->getNode($index);
-                    if (!$argument instanceof \Twig_Node_Expression_Constant) {
+                    if (!$this->checkNodeIsConstant($argument)) {
                         return $node;
-                        // FIXME: Throw exception if there is some way for the user to turn this off
-                        //        on a case-by-case basis, similar to @Ignore in PHP
                     }
 
                     $domain = $argument->getAttribute('value');
@@ -153,4 +156,35 @@ public function leaveNode(\Twig_NodeInterface $node, \Twig_Environment $env)
 
     public function visitFile(\SplFileInfo $file, MessageCatalogue $catalogue) { }
     public function visitPhpFile(\SplFileInfo $file, MessageCatalogue $catalogue, array $ast) { }
-}
+
+    public function setLogger(LoggerInterface $logger)
+    {
+        $this->logger = $logger;
+    }
+
+    /**
+     * @param \Twig_Node $node
+     * @return bool
+     */
+    private function checkNodeIsConstant(\Twig_Node $node)
+    {
+        if ($node instanceof \Twig_Node_Expression_Constant) {
+            return true;
+        }
+
+        $message = sprintf(
+            'Cannot infer translation id from node "%s". Please refactor to only translate constants in file "%s" on line %d',
+            get_class($node),
+            $this->file,
+            $node->getLine()
+        );
+
+        if (!$this->logger) {
+            throw new RuntimeException($message);
+        }
+
+        $this->logger->err($message);
+
+        return false;
+    }
+}