Added option to not add source files to xliff output

Author: artursvonda

Tests/Translation/Dumper/XliffDumperTest.php

@@ -18,6 +18,7 @@
 
 namespace JMS\TranslationBundle\Tests\Translation\Dumper;
 
+use JMS\TranslationBundle\Model\FileSource;
 use JMS\TranslationBundle\Model\Message;
 
 use JMS\TranslationBundle\Model\MessageCatalogue;
@@ -54,6 +55,23 @@ public function testCdataOutput()
         $this->assertEquals($expected, $dumper->dump($catalogue, 'messages'));
     }
 
+    public function testDumpWithoutSourcesAdded()
+    {
+        $catalogue = new MessageCatalogue();
+        $catalogue->setLocale('en');
+
+        $message = new Message('foo');
+        $message->addSource(new FileSource('/a/b/c/foo/bar', 1, 2));
+        $message->addSource(new FileSource('bar/baz', 1, 2));
+        $catalogue->add($message);
+
+        $dumper = $this->getDumper();
+        $dumper->setAddSources(false);
+        $dumped = $dumper->dump($catalogue, 'messages');
+
+        $this->assertEquals($this->getOutput('simple'), $dumped);
+    }
+
     protected function getDumper()
     {
         $dumper = new XliffDumper();
@@ -74,4 +92,4 @@ protected function getOutput($key)
 
         return file_get_contents($file);
     }
-}
+}

Translation/Dumper/XliffDumper.php

@@ -34,6 +34,7 @@ class XliffDumper implements DumperInterface
 {
     private $sourceLanguage = 'en';
     private $addDate = true;
+    private $addSources = true;
 
     /**
      * @param $bool
@@ -43,6 +44,14 @@ public function setAddDate($bool)
         $this->addDate = (Boolean) $bool;
     }
 
+    /**
+     * @param boolean $addSources
+     */
+    public function setAddSources($addSources)
+    {
+        $this->addSources = $addSources;
+    }
+
     /**
      * @param $lang
      */
@@ -115,24 +124,27 @@ public function dump(MessageCatalogue $catalogue, $domain = 'messages')
                 $target->setAttribute('state', 'new');
             }
 
-            // As per the OASIS XLIFF 1.2 non-XLIFF elements must be at the end of the <trans-unit>
-            if ($sources = $message->getSources()) {
-                foreach ($sources as $source) {
-                    if ($source instanceof FileSource) {
-                        $unit->appendChild($refFile = $doc->createElement('jms:reference-file', $source->getPath()));
+            if ($this->addSources) {
+                // As per the OASIS XLIFF 1.2 non-XLIFF elements must be at the end of the <trans-unit>
+                if ($sources = $message->getSources()) {
+                    foreach ($sources as $source) {
+                        if ($source instanceof FileSource) {
+                            $refFile = $doc->createElement('jms:reference-file', $source->getPath());
+                            $unit->appendChild($refFile);
 
-                        if ($source->getLine()) {
-                            $refFile->setAttribute('line', $source->getLine());
-                        }
+                            if ($source->getLine()) {
+                                $refFile->setAttribute('line', $source->getLine());
+                            }
 
-                        if ($source->getColumn()) {
-                            $refFile->setAttribute('column', $source->getColumn());
+                            if ($source->getColumn()) {
+                                $refFile->setAttribute('column', $source->getColumn());
+                            }
+
+                            continue;
                         }
 
-                        continue;
+                        $unit->appendChild($doc->createElementNS('jms:reference', (string) $source));
                     }
-
-                    $unit->appendChild($doc->createElementNS('jms:reference', (string) $source));
                 }
             }
 
@@ -144,4 +156,4 @@ public function dump(MessageCatalogue $catalogue, $domain = 'messages')
 
         return $doc->saveXML();
     }
-}
+}