Merge pull request #14 from nextapps-be/respect-excluded-files-config…

…-when-downloading

Do not delete excluded translation files when downloading poeditor translations

.php-cs-fixer.php

@@ -32,7 +32,7 @@
     // Converts implicit variables into explicit ones in double-quoted strings or heredoc syntax.
     'explicit_string_variable' => true,
     // Transforms imported FQCN parameters and return types in function arguments to short version.
-    'fully_qualified_strict_types' => true,
+    'fully_qualified_strict_types' => false,
     // Add missing space between function's argument and its typehint.
     'function_typehint_space' => true,
     // Pre- or post-increment and decrement operators should be used if possible.

src/Translations/TranslationManager.php

@@ -258,7 +258,13 @@ protected function createEmptyLocaleFolder(string $locale)
         $path = $this->getLangPath($locale);
 
         if (file_exists($path)) {
-            $this->filesystem->deleteDirectory($path);
+            foreach ($this->filesystem->allFiles($path) as $file) {
+                if (! in_array($file->getFilename(), config('poeditor-sync.excluded_files', []))) {
+                    $this->filesystem->delete($file->getRealPath());
+                }
+            }
+
+            return;
         }
 
         $this->filesystem->makeDirectory($path);

tests/DownloadCommandTest.php

@@ -207,6 +207,27 @@ public function it_removes_old_php_translations_of_locale()
         $this->assertTrue(file_exists($this->getLangPath('en/new-php-file.php')));
     }
 
+    /** @test */
+    public function it_does_not_remove_php_translations_of_locale_if_php_translation_is_excluded_in_config()
+    {
+        file_put_contents($this->getLangPath('en/excluded-php-file.php'), ['foo' => 'bar']);
+        file_put_contents($this->getLangPath('en/not-excluded-php-file.php'), ['foo' => 'bar']);
+
+        config()->set('poeditor-sync.excluded_files', ['excluded-php-file.php']);
+
+        $this->mockPoeditorDownload('en', [
+            'some-php-file' => [
+                'foo' => 'bar',
+            ],
+        ]);
+
+        $this->artisan('poeditor:download')->assertExitCode(0);
+
+        $this->assertTrue(file_exists($this->getLangPath('en/excluded-php-file.php')));
+        $this->assertFalse(file_exists($this->getLangPath('en/not-excluded-php-file.php')));
+        $this->assertTrue(file_exists($this->getLangPath('en/some-php-file.php')));
+    }
+
     /** @test */
     public function it_overrides_old_json_translation_of_locale()
     {