Merge branch 'release/1.1.0'

svennis94 · May 12, 2018 · cb86856 · cb86856
2 parents a5887c0 + 86ad4f7
commit cb86856
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 29 deletions.
diff --git a/src/Commands/SynchroniseTranslations.php b/src/Commands/SynchroniseTranslations.php
@@ -3,6 +3,7 @@
 namespace SeBuDesign\PoEditor\Commands;
 
 use Illuminate\Console\Command;
+use Illuminate\Support\Collection;
 use SeBuDesign\PoEditor\PoEditor;
 
 class SynchroniseTranslations extends Command
@@ -16,9 +17,8 @@ public function handle()
     {
         $poeditor = new PoEditor($this->argument('project'));
 
+        $locales = collect();
         foreach ($poeditor->languages() as $language) {
-            $languageFile = resource_path('lang/' . $language['code'] . '.json');
-
             $terms = collect($poeditor->terms($language['code']))
                 // Pluck the translation content and set the term as the key
                 ->pluck('translation.content', 'term')
@@ -29,18 +29,40 @@ public function handle()
                     }
 
                     return $content;
-                })
-                ->toJson();
+                });
 
-            // If an old file exists remove it
-            if (\File::exists($languageFile)) {
-                \File::delete($languageFile);
-            }
+            $this->updateFile(
+                resource_path('lang/' . $language['code'] . '.json'),
+                $terms
+            );
 
-            // Write the new file
-            \File::put($languageFile, $terms);
+            $locales->push([
+                'name' => $language['name'],
+                'code' => $language['code'],
+            ]);
         }
 
+        $this->updateFile(
+            resource_path('lang/locales.json'),
+            $locales
+        );
+
         $this->info("Translations synchronised");
     }
+
+    protected function updateFile($file, $content)
+    {
+        // If an old file exists remove it
+        if (\File::exists($file)) {
+            \File::delete($file);
+        }
+
+        // If it is a collection create a json string of it
+        if ($content instanceof Collection) {
+            $content = $content->toJson();
+        }
+
+        // Write the new file
+        \File::put($file, $content);
+    }
 }
diff --git a/tests/LanguageTest.php b/tests/LanguageTest.php
diff --git a/tests/LocalesTest.php b/tests/LocalesTest.php
@@ -0,0 +1,29 @@
+<?php
+
+namespace SeBuDesign\PoEditor\Test;
+
+use SeBuDesign\PoEditor\PoEditor;
+
+class LocalesTest extends TestCase
+{
+    /** @test */
+    public function it_has_all_locales_synchronised()
+    {
+        $this->artisan('synchronise:translations');
+
+        $poeditor = new PoEditor();
+
+        // Get the locales file
+        $locales = [];
+        foreach ($poeditor->languages() as $language) {
+            $this->assertFileExists( resource_path('lang/' . $language['code'] . '.json') );
+
+            $locales[] = [
+                'name' => $language['name'],
+                'code' => $language['code'],
+            ];
+        }
+
+        $this->assertJsonStringEqualsJsonFile(resource_path('lang/locales.json'), json_encode($locales));
+    }
+}