Comments

Author: tonysm

config/turbo-laravel.php

@@ -19,10 +19,10 @@
      | Root Model Namespaces
      |--------------------------------------------------------------------------
      |
-     | When generating the DOM ID for models, we need to strip out parts of the namespace, so we
-     | can more accurate DOM IDs. If you happen to use a different convention for where you place
-     | your models, please, add the model namespace prefixes here. We'll strip out the first one that
-     | matches a "Str:startsWith()" check, so sort them from more specific to less specific.
+     | When generating DOM IDs for models, we need to strip out the root namespaces from the model's FQCN. Please,
+     | if you use non-conventional folder structures, make sure you add your custom namespaces to this list. The
+     | first one that matches a "starts with" check will be used and removed from the model's FQCN for DOM IDs.
+     |
      */
     'models_namespace' => [
         'App\\Models\\',

src/NamesResolver.php

@@ -80,6 +80,10 @@ public static function resourceIdFor(Model $model, string $prefix = ""): string
 
     private static function getModelWithoutRootNamespaces(Model $model): string
     {
+        // We will attempt to strip out only the root namespace from the model's FQCN. For that, we will use
+        // the configured namespaces, stripping out the first one that matches on a Str::startsWith check.
+        // Namespaces are configurable. We'll default back to class_basename when no namespace matches.
+
         $className = get_class($model);
 
         foreach (config('turbo-laravel.models_namespace') as $rootNs) {
@@ -88,7 +92,6 @@ private static function getModelWithoutRootNamespaces(Model $model): string
             }
         }
 
-        // If none of the previous matchers work, we use the class_basename helper instead.
         return class_basename($model);
     }
 }