Merge pull request #30 from givebutter/develop

Add support for `archived_at` field

Author: liran-co

database/migrations/create_custom_fields_tables.php.stub

@@ -25,7 +25,8 @@ class CreateCustomFieldsTables extends Migration
             $table->string('default_value')->nullable();
             $table->string('order');
             $table->timestamps();
-            $table->timestamp('deleted_at')->nullable();
+            $table->timestamp('archived_at')->nullable();
+            $table->softDeletes();
         });
 
         Schema::create(config('custom-fields.tables.field_responses', 'custom_field_responses'), function (Blueprint $table) {

src/Models/CustomField.php

@@ -72,6 +72,7 @@ class CustomField extends Model
      */
     protected $casts = [
         'answers' => 'array',
+        'archived_at' => 'datetime',
     ];
 
     /**
@@ -96,7 +97,8 @@ protected static function boot()
         parent::boot();
 
         self::creating(function ($field) {
-            $lastFieldOnCurrentModel = $field->model->customFields()->orderBy('order', 'desc')->first();
+            $lastFieldOnCurrentModel = $field->model->customFields()->orderByDesc('order')->first();
+
             $field->order = ($lastFieldOnCurrentModel ? $lastFieldOnCurrentModel->order : 0) + 1;
         });
     }

src/Models/CustomFieldResponse.php

@@ -43,9 +43,13 @@ class CustomFieldResponse extends Model
      */
     public function __construct(array $attributes = [])
     {
-        // We have to do this because the `value` mutator depends on
-        // `field_id` being set. If `value` is declared earlie than `field_id`
-        // in a create() array, the mutator will blow up.
+        /*
+         * We have to do this because the `value` mutator
+         * depends on `field_id` being set. If `value`
+         * is declared earlier than `field_id` in a
+         * create() array, the mutator will fail.
+         */
+
         $this->attributes = $attributes;
 
         $this->bootIfNotBooted();