Don't force translations lazy-load on save

Prevent N+1 queries when creating/updating
non-translated model attributes.

Don't lazy-load the 'translations'
relationship when saving the base
translated model.

src/Translatable/Translatable.php

@@ -452,6 +452,11 @@ protected function getLocaleSeparator()
     protected function saveTranslations()
     {
         $saved = true;
+
+        if (! $this->relationLoaded('translations')) {
+            return $saved;
+        }
+
         foreach ($this->translations as $translation) {
             if ($saved && $this->isTranslationDirty($translation)) {
                 if (! empty($connectionName = $this->getConnectionName())) {

tests/TranslatableTest.php

@@ -127,6 +127,29 @@ public function test_it_saves_translations_with_mutator()
         $this->assertEquals('5678', $translation->name);
     }
 
+    public function test_it_does_not_lazy_load_translations_when_updating_non_translated_attributes()
+    {
+        DB::enableQueryLog();
+
+        $country = Country::create(['code' => 'be']);
+        $this->assertFalse($country->relationLoaded('translations'));
+        $this->assertCount(1, DB::getQueryLog());
+
+        DB::flushQueryLog();
+
+        $country->update(['code' => 'de']);
+        $this->assertFalse($country->relationLoaded('translations'));
+        $this->assertCount(1, DB::getQueryLog());
+
+        DB::flushQueryLog();
+
+        $country->update(['name' => 'Germany']);
+        $this->assertTrue($country->relationLoaded('translations'));
+        $this->assertCount(2, DB::getQueryLog());
+
+        DB::disableQueryLog();
+    }
+
     public function test_it_uses_default_locale_to_return_translations()
     {
         $country = Country::whereCode('gr')->first();