Add a false timestamp property to the model if timestamps are disabled (#189)

Author: PWParsons

src/Generators/ModelGenerator.php

@@ -92,6 +92,10 @@ private function buildProperties(Model $model)
     {
         $properties = '';
 
+        if (! $model->usesTimestamps()) {
+            $properties .= $this->files->stub('model/timestamps.stub');
+        }
+
         if (config('blueprint.use_guarded')) {
             $properties .= $this->files->stub('model/guarded.stub');
         } else {

stubs/model/timestamps.stub

@@ -0,0 +1,6 @@
+    /**
+     * Indicates if the model should be timestamped.
+     *
+     * @var bool
+     */
+    public $timestamps = false;

tests/Feature/Generator/ModelGeneratorTest.php

@@ -79,6 +79,7 @@ public function output_generates_models($definition, $path, $model)
             ->with('model/method-comment.stub')
             ->andReturn(file_get_contents('stubs/model/method-comment.stub'));
 
+
         $this->files->expects('exists')
             ->with(dirname($path))
             ->andReturnTrue();
@@ -163,6 +164,39 @@ public function output_generates_relationships()
         $this->assertEquals(['created' => ['app/Subscription.php']], $this->subject->output($tree));
     }
 
+    /**
+     * @test
+     */
+    public function output_generates_disabled_auto_columns()
+    {
+        $this->files->expects('stub')
+            ->with('model/class.stub')
+            ->andReturn(file_get_contents('stubs/model/class.stub'));
+        $this->files->expects('stub')
+            ->with('model/timestamps.stub')
+            ->andReturn(file_get_contents('stubs/model/timestamps.stub'));
+        $this->files->expects('stub')
+            ->with('model/fillable.stub')
+            ->andReturn(file_get_contents('stubs/model/fillable.stub'));
+        $this->files->expects('stub')
+            ->with('model/casts.stub')
+            ->andReturn(file_get_contents('stubs/model/casts.stub'));
+        $this->files->expects('stub')
+            ->with('model/method.stub')
+            ->andReturn(file_get_contents('stubs/model/method.stub'));
+
+        $this->files->expects('exists')
+            ->with('app')
+            ->andReturnTrue();
+        $this->files->expects('put')
+            ->with('app/State.php', $this->fixture('models/disable-auto-columns.php'));
+
+        $tokens = $this->blueprint->parse($this->fixture('definitions/disable-auto-columns.bp'));
+        $tree = $this->blueprint->analyze($tokens);
+
+        $this->assertEquals(['created' => ['app/State.php']], $this->subject->output($tree));
+    }
+
     /**
      * @test
      */
@@ -214,6 +248,12 @@ public function output_generates_phpdoc_for_model($definition, $path, $model)
             ->with('model/class.stub')
             ->andReturn(file_get_contents('stubs/model/class.stub'));
 
+        if ($definition === 'definitions/disable-auto-columns.bp') {
+            $this->files->expects('stub')
+                ->with('model/timestamps.stub')
+                ->andReturn(file_get_contents('stubs/model/timestamps.stub'));
+        }
+
         $this->files->expects('stub')
             ->with('model/fillable.stub')
             ->andReturn(file_get_contents('stubs/model/fillable.stub'));

tests/fixtures/models/disable-auto-columns-phpdoc.php

@@ -11,6 +11,13 @@
  */
 class State extends Model
 {
+    /**
+     * Indicates if the model should be timestamped.
+     *
+     * @var bool
+     */
+    public $timestamps = false;
+
     /**
      * The attributes that are mass assignable.
      *

tests/fixtures/models/disable-auto-columns.php

@@ -0,0 +1,35 @@
+<?php
+
+namespace App;
+
+use Illuminate\Database\Eloquent\Model;
+
+class State extends Model
+{
+    /**
+     * Indicates if the model should be timestamped.
+     *
+     * @var bool
+     */
+    public $timestamps = false;
+
+    /**
+     * The attributes that are mass assignable.
+     *
+     * @var array
+     */
+    protected $fillable = [
+        'name',
+        'code',
+        'country_id',
+    ];
+
+    /**
+     * The attributes that should be cast to native types.
+     *
+     * @var array
+     */
+    protected $casts = [
+        'country_id' => 'integer',
+    ];
+}