Added allow-newlines option to config/laravel-translatable-string-exporter that indicates weather new lines are allowed in translations.

Including additional tests.

Author: Marijn Kampf

config/laravel-translatable-string-exporter.php

@@ -12,6 +12,9 @@
         '*.js',
     ],
 
+    // Indicates weather new lines are allowed in translations.
+    'allow-newlines' => false,
+
     // Translation function names.
     // If your function name contains $ escape it using \$ .
     'functions'=> [

src/Core/CodeParser.php

@@ -31,6 +31,10 @@ public function __construct()
                '@lang'
            ]);
         $this->pattern = str_replace('[FUNCTIONS]', implode('|', $this->functions), $this->pattern);
+
+        if (config('laravel-translatable-string-exporter.allow-newlines')) {
+            $this->pattern .= 's';
+        }
     }
 
     /**

tests/ExporterTest.php

@@ -93,6 +93,66 @@ public function testTranslationFunctionNames()
         $this->assertEquals($expected, $actual);
     }
 
+    public function testMultiLineDefaultSettingsTranslations()
+    {
+        $this->cleanLangsFolder();
+
+        $view = "{{ __('single line') }} " .
+            "{{ __('translation.keys') }} " .
+            // Verify legacy behaviours:
+            // 1) Strings including escaped newlines (\n) are processed
+            "{{ __('escaped\\nnewline') }}" .
+            // 2) Strings including un-escaped newlines are ignored.
+            "{{ __(\"ignored\nmultiple\nline\nstring\") }}";
+
+        $this->createTestView($view);
+
+        $this->artisan('translatable:export', ['lang' => 'es'])
+            ->expectsOutput('Translatable strings have been extracted and written to the es.json file.')
+            ->assertExitCode(0);
+
+        $actual = $this->getTranslationFileContent('es');
+        $expected = [
+            'single line' => 'single line',
+            'translation.keys' => 'translation.keys',
+            'escaped\\nnewline' => 'escaped\\nnewline',
+        ];
+
+        $this->assertEquals($expected, $actual);
+    }
+
+    public function testMultiLineSettingsTranslations()
+    {
+        $this->app['config']->set('laravel-translatable-string-exporter.allow-newlines', true);
+
+        $this->cleanLangsFolder();
+
+        $view = "{{ __('single line') }} " .
+                "{{ __('translation.keys') }} " .
+            // No change to 1st legacy behaviour:
+            // 1) Strings including escaped newlines (\n) are processed
+            "{{ __('escaped\\nnewline') }}" .
+            // Un-escaped newlines are now also processed.
+            // 2) Strings including un-escaped newlines are ignored.
+            "{{ __(\"detected\nmultiple\nline\nstring\") }}";
+
+        $this->createTestView($view);
+
+        $this->artisan('translatable:export', ['lang' => 'es'])
+            ->expectsOutput('Translatable strings have been extracted and written to the es.json file.')
+            ->assertExitCode(0);
+
+        $actual = $this->getTranslationFileContent('es');
+        $expected = [
+            'single line' => 'single line',
+            'translation.keys' => 'translation.keys',
+            'escaped\nnewline' => 'escaped\nnewline',
+            "detected\nmultiple\nline\nstring" => "detected\nmultiple\nline\nstring",
+        ];
+
+        $this->assertEquals($expected, $actual);
+    }
+
     public function testUpdatingTranslations()
     {
         $this->cleanLangsFolder();