Automatically infer Behaviors and as much as possible and limit boilerplating

[Tofandel]

Dropping some ideas for v4

For instance we could automatically make fields translatable or not if the name is found in the translatable fields

Subject: [PATCH] Ensure fields are translatable
---
Index: src/Services/Forms/Fields/BaseFormField.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/Services/Forms/Fields/BaseFormField.php b/src/Services/Forms/Fields/BaseFormField.php
--- a/src/Services/Forms/Fields/BaseFormField.php	(revision 62278de675b6902c530ab5e82a8922710446fe1f)
+++ b/src/Services/Forms/Fields/BaseFormField.php	(date 1708969528664)
@@ -48,6 +48,11 @@
         return $this;
     }
 
+    public function getName(): ?string
+    {
+        return $this->name;
+    }
+
     public function default(mixed $default): static
     {
         $this->default = $default;
Index: src/Http/Controllers/Admin/ModuleController.php
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/src/Http/Controllers/Admin/ModuleController.php b/src/Http/Controllers/Admin/ModuleController.php
--- a/src/Http/Controllers/Admin/ModuleController.php	(revision 62278de675b6902c530ab5e82a8922710446fe1f)
+++ b/src/Http/Controllers/Admin/ModuleController.php	(date 1708969528714)
@@ -786,7 +786,25 @@
 
         $columns = $columns->merge($this->additionalBrowserTableColumns());
 
-        return $columns;
+        return $this->ensureFieldsAreTranslatable(app($this->modelName), $columns);
+    }
+
+    /**
+     * @template T of Collection
+     * @param T $form
+     * @return T
+     */
+    protected function ensureFieldsAreTranslatable(TwillModelContract $model, Collection $form)
+    {
+        $translatedAttributes = $model->getTranslatedAttributes();
+        foreach ($form as $field) {
+            /** @var BaseFormField $field */
+            if (method_exists($field, 'translatable') && in_array($field->getName(), $translatedAttributes)) {
+                $field->translatable();
+            }
+        }
+
+        return $form;
     }
 
     protected function getIndexTableColumns(): TableColumns
@@ -851,7 +869,7 @@
             );
         }
 
-        return $columns;
+        return $this->ensureFieldsAreTranslatable(app($this->modelName), $columns);
     }
 
     /**

I'm not a fan of the UseTrait pattern it's against a few best practices and resource intensive (we have to calculate recursively all the traits used by all the classes before knowing) I think an interface or method_exists based approach would be better

I also think if we do this right we can make the Repository class optionnal in favor of a basic one that infers all that is needed, we would only need the Controller like this (I don't like to have to keep 3 different files in wildly different places in sync like this it's really a lot of back and forth)

Can we maybe create a milestone pinned issue to reference what we should make happen in v4 and discuss this ?