refactor(price-lists): more refactoring

Author: KilianTrunk

src/Filament/Resources/PriceListResource/Pages/CreatePriceList.php

@@ -47,26 +47,4 @@ protected function getFormActions(): array
             $this->getCancelFormAction(),
         ];
     }
-
-    /**
-     * Store current tenant data before submitting
-     */
-    protected function storeCurrentTenantData(): void
-    {
-        $formData = $this->form->getState();
-        $selectedTenant = $formData['selected_tenant'] ?? null;
-
-        if ($selectedTenant && config('eclipse-catalogue.tenancy.foreign_key')) {
-            $currentData = [
-                'is_active' => $formData['tenant_data'][$selectedTenant]['is_active'] ?? true,
-                'is_default' => $formData['tenant_data'][$selectedTenant]['is_default'] ?? false,
-                'is_default_purchase' => $formData['tenant_data'][$selectedTenant]['is_default_purchase'] ?? false,
-            ];
-
-            $allTenantData = $formData['all_tenant_data'] ?? [];
-            $allTenantData[$selectedTenant] = $currentData;
-
-            $this->form->fill(['all_tenant_data' => $allTenantData]);
-        }
-    }
 }

src/Filament/Resources/PriceListResource/Pages/EditPriceList.php

@@ -100,28 +100,6 @@ protected function getFormActions(): array
         ];
     }
 
-    /**
-     * Store current tenant data before submitting
-     */
-    protected function storeCurrentTenantData(): void
-    {
-        $formData = $this->form->getState();
-        $selectedTenant = $formData['selected_tenant'] ?? null;
-
-        if ($selectedTenant && config('eclipse-catalogue.tenancy.foreign_key')) {
-            $currentData = [
-                'is_active' => $formData['tenant_data'][$selectedTenant]['is_active'] ?? true,
-                'is_default' => $formData['tenant_data'][$selectedTenant]['is_default'] ?? false,
-                'is_default_purchase' => $formData['tenant_data'][$selectedTenant]['is_default_purchase'] ?? false,
-            ];
-
-            $allTenantData = $formData['all_tenant_data'] ?? [];
-            $allTenantData[$selectedTenant] = $currentData;
-
-            $this->form->fill(['all_tenant_data' => $allTenantData]);
-        }
-    }
-
     protected function resolveRecord($key): Model
     {
         // Load record without joins to avoid ambiguous column issues

src/Traits/HandlesPriceListTenantData.php

@@ -6,6 +6,29 @@
 
 trait HandlesPriceListTenantData
 {
+    /**
+     * Persist the currently selected tenant's values into the hidden
+     * all_tenant_data state so switching tenants doesn't lose changes.
+     */
+    protected function storeCurrentTenantData(): void
+    {
+        $formData = $this->form->getState();
+        $selectedTenant = $formData['selected_tenant'] ?? null;
+
+        if ($selectedTenant && config('eclipse-catalogue.tenancy.foreign_key')) {
+            $currentData = [
+                'is_active' => $formData['tenant_data'][$selectedTenant]['is_active'] ?? true,
+                'is_default' => $formData['tenant_data'][$selectedTenant]['is_default'] ?? false,
+                'is_default_purchase' => $formData['tenant_data'][$selectedTenant]['is_default_purchase'] ?? false,
+            ];
+
+            $allTenantData = $formData['all_tenant_data'] ?? [];
+            $allTenantData[$selectedTenant] = $currentData;
+
+            $this->form->fill(['all_tenant_data' => $allTenantData]);
+        }
+    }
+
     /**
      * Validate default constraints before saving.
      */