Call bindDynamically in __constructor

Author: Sairahcaz

README.md

@@ -113,7 +113,8 @@ you can just create your own Eloquent model and **extend** it by the **DynamicMo
 or if the model is already extended by another model, you can just use the **DynamicModelBinding** trait.
 
 If you use the trait, make sure your model implements the **DynamicModelInterface**.
-Also make sure to resolve these models through the **DynamicModelFactory**!
+Also make sure to resolve these models through the **DynamicModelFactory** and
+call the `bindDynamically` function in the `__constructor`!
 
 ``` php
 namespace App\Models;
@@ -133,11 +134,21 @@ class MyDynamicModel extends DynamicModel
     }
 }
 
-# option 2: use the trait
+# option 2: use the trait when class already extends a class
 class MyDynamicModel extends SomeBaseModel implements DynamicModelInterface
 {
     use DynamicModelBinding;
 
+    protected $guarded = [];
+
+    // !!important!! call the parent constructor and bindDynamically function in the constructor!
+    public function __construct(array $attributes = [])
+    {
+        parent::__construct($attributes);
+
+        $this->bindDynamically();
+    }
+
     public function doSomething()
     {
         // do something

src/DynamicModel.php

@@ -14,4 +14,11 @@ class DynamicModel extends Model implements DynamicModelInterface
      * and implementing the DynamicModelInterface!
      */
     protected $guarded = [];
+
+    public function __construct(array $attributes = [])
+    {
+        parent::__construct($attributes);
+
+        $this->bindDynamically();
+    }
 }

src/DynamicModelBinding.php

@@ -6,31 +6,45 @@
 
 trait DynamicModelBinding
 {
-    public function bindDynamically(string $tableName, string $dbConnection = null): void
+    private static string $dynamicTableName;
+
+    private static ?string $dynamicDBConnection = null;
+
+    public static function setDynamicTableName(string $tableName): void
     {
-        if (! Schema::hasTable($tableName)) {
-            throw DynamicModelException::tableDoesNotExist($tableName);
-        }
+        self::$dynamicTableName = $tableName;
+    }
+
+    public static function setDynamicDBConnection(string $dbConnection): void
+    {
+        self::$dynamicDBConnection = $dbConnection;
+    }
 
+    public function bindDynamically(): void
+    {
         // change connection, if desired
-        if ($dbConnection) {
-            $this->setConnection($dbConnection);
+        if (self::$dynamicDBConnection) {
+            $this->setConnection(self::$dynamicDBConnection);
+        }
+
+        if (! Schema::hasTable(self::$dynamicTableName)) {
+            throw DynamicModelException::tableDoesNotExist(self::$dynamicTableName);
         }
 
         // set the table for the dynamic model
-        $this->setTable($tableName);
+        $this->setTable(self::$dynamicTableName);
 
         // apply primary key, incrementing and key type
         $connection = Schema::getConnection();
 
-        $table = $connection->getDoctrineSchemaManager()->listTableDetails($tableName);
+        $table = $connection->getDoctrineSchemaManager()->listTableDetails(self::$dynamicTableName);
 
         if (! $primaryKey = $table->getPrimaryKey()) {
             throw DynamicModelException::primaryKeyDoesNotExist();
         }
 
         $primaryKeyName = $primaryKey->getColumns()[0];
-        $primaryColumn = $connection->getDoctrineColumn($tableName, $primaryKeyName);
+        $primaryColumn = $connection->getDoctrineColumn(self::$dynamicTableName, $primaryKeyName);
 
         $this->primaryKey = $primaryColumn->getName();
         $this->incrementing = $primaryColumn->getAutoincrement();

src/DynamicModelFactory.php

@@ -12,16 +12,16 @@ class DynamicModelFactory
      */
     public function create(string $concreteClassName, string $tableName, string $dbConnection = null): Model&DynamicModelInterface
     {
-        $dynamicModel = new $concreteClassName();
-
-        // tell the IDE which type $dynamicModel should be...
-        /** @var $dynamicModel Model&DynamicModelInterface */
-        if (! method_exists($dynamicModel, 'bindDynamically')) {
+        /** @var $concreteClassName Model&DynamicModelInterface */
+        if (! method_exists($concreteClassName, 'bindDynamically')) {
             throw DynamicModelException::bindFuncDoesNotExist($concreteClassName);
         }
 
-        $dynamicModel->bindDynamically($tableName, $dbConnection);
+        $concreteClassName::setDynamicTableName($tableName);
+        if ($dbConnection) {
+            $concreteClassName::setDynamicDBConnection($dbConnection);
+        }
 
-        return $dynamicModel;
+        return new $concreteClassName();
     }
 }

src/DynamicModelInterface.php

@@ -4,8 +4,12 @@
 
 interface DynamicModelInterface
 {
+    public static function setDynamicTableName(string $tableName): void;
+
+    public static function setDynamicDBConnection(string $dbConnection): void;
+
     /**
      * Make sure the DynamicModel has a bindDynamic function.
      */
-    public function bindDynamically(string $tableName, string $dbConnection = null): void;
+    public function bindDynamically(): void;
 }

src/LaravelDynamicModelServiceProvider.php

@@ -31,7 +31,11 @@ public function register()
             }
 
             return app(DynamicModelFactory::class)
-                ->create(DynamicModel::class, $parameters['table_name']);
+                ->create(
+                    DynamicModel::class,
+                    $parameters['table_name'],
+                    $parameters['db_connection'] ?? null
+                );
         });
     }
 }

tests/MyTraitedDynamicModel.php

@@ -10,4 +10,11 @@ class MyTraitedDynamicModel extends MyBaseModel implements DynamicModelInterface
     use DynamicModelBinding;
 
     protected $guarded = [];
+
+    public function __construct(array $attributes = [])
+    {
+        parent::__construct($attributes);
+
+        $this->bindDynamically();
+    }
 }