Value not nullable on extract (#31)

* Do not use updateOrCreate, so we can set value when creating translatable string

* Fix tests

* Fix styling

---------

Co-authored-by: jyrkidn <2447042+jyrkidn@users.noreply.github.com>

Author: jyrkidn

src/ExtractTranslatableStrings.php

@@ -125,17 +125,21 @@ public function missingKey(string $scope, string $name, string $key, bool $isHtm
     {
         TranslatableString::flushEventListeners();
 
-        $translatableString = TranslatableString::updateOrCreate([
+        $translatableString = TranslatableString::firstOrNew([
             'scope' => $scope,
             'name' => $name,
-        ], [
-            'key' => $key,
-            'is_html' => $isHtml,
         ]);
 
-        if ($translatableString->wasRecentlyCreated) {
+        // Set/update fields that should always be updated
+        $translatableString->key = $key;
+        $translatableString->is_html = $isHtml;
+
+        // Only set value on creation (not on update)
+        if (! $translatableString->exists) {
             $translatableString->value = $this->getValue($key);
         }
+
+        $translatableString->save();
     }
 
     public function getAllFunctions(): Collection

tests/Feature/Filament/TranslatableStringResource/Pages/ListTranslatableStringsTest.php

@@ -107,7 +107,7 @@
     Notification::assertNotified(
         Notification::make()
             ->success()
-            ->title('Import was successful')
+            ->title(__('filament-translatable-strings::admin.import completed'))
     );
 
     $this->assertDatabaseCount(TranslatableString::class, 2);