moves is_public in file_types table

Author: aocneanu

database/migrations/2017_01_01_112100_create_file_types_table.php

@@ -4,8 +4,7 @@
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class extends Migration
-{
+return new class extends Migration {
     public function up()
     {
         Schema::create('file_types', function (Blueprint $table) {
@@ -19,6 +18,7 @@ public function up()
 
             $table->text('description')->nullable();
 
+            $table->boolean('is_public');
             $table->boolean('is_browsable');
             $table->boolean('is_system');
 

src/Forms/Templates/type.json

@@ -39,9 +39,23 @@
                         "type": "input",
                         "content": "text"
                     }
+                }
+            ]
+        },
+        {
+            "columns": 3,
+            "fields": [
+                {
+                    "label": "Public",
+                    "name": "is_public",
+                    "value": false,
+                    "meta": {
+                        "type": "input",
+                        "content": "checkbox"
+                    }
                 },
                 {
-                    "label": "Is Browsable",
+                    "label": "Browsable",
                     "name": "is_browsable",
                     "value": false,
                     "meta": {
@@ -50,7 +64,7 @@
                     }
                 },
                 {
-                    "label": "Is System",
+                    "label": "System",
                     "name": "is_system",
                     "value": false,
                     "meta": {

src/Http/Requests/ValidateType.php

@@ -22,6 +22,7 @@ public function rules()
             'icon' => 'nullable|required_if:is_browsable,true|string',
             'folder' => 'required_with:model|string',
             'description' => 'nullable|string',
+            'is_public' => 'required|boolean',
             'is_browsable' => 'required|boolean',
             'is_system' => 'required|boolean',
         ];

src/Models/Type.php

@@ -11,7 +11,6 @@
 use Illuminate\Support\Facades\Storage;
 use Illuminate\Support\Str;
 use LaravelEnso\Files\Contracts\Attachable;
-use LaravelEnso\Files\Contracts\PublicFile;
 use LaravelEnso\Rememberable\Traits\Rememberable;
 use LaravelEnso\Tables\Traits\TableCache;
 
@@ -23,7 +22,10 @@ class Type extends Model
 
     protected $guarded = [];
 
-    protected $casts = ['is_browsable' => 'boolean', 'is_system' => 'boolean'];
+    protected $casts = [
+        'is_browsable' => 'boolean', 'is_system' => 'boolean',
+        'is_public' => 'boolean',
+    ];
 
     protected array $rememberableKeys = ['id', 'model'];
 
@@ -56,7 +58,7 @@ public function model(): Attachable
 
     public function isPublic(): bool
     {
-        return $this->model() instanceof PublicFile;
+        return $this->is_public;
     }
 
     public static function for(string $model): self

src/Tables/Builders/Type.php

@@ -14,7 +14,7 @@ public function query(): Builder
     {
         return Model::select([
             'id', 'name', 'icon', 'folder', 'model',
-            'is_browsable', 'is_system',
+            'is_public',  'is_browsable', 'is_system',
         ]);
     }
 

src/Tables/Templates/types.json

@@ -46,7 +46,16 @@
             ]
         },
         {
-            "label": "Is Browsable",
+            "label": "Public",
+            "name": "is_public",
+            "data": "file_types.is_public",
+            "meta": [
+                "boolean",
+                "sortable"
+            ]
+        },
+        {
+            "label": "Browsable",
             "name": "is_browsable",
             "data": "file_types.is_browsable",
             "meta": [
@@ -55,7 +64,7 @@
             ]
         },
         {
-            "label": "Is System",
+            "label": "System",
             "name": "is_system",
             "data": "file_types.is_system",
             "meta": [

src/Upgrades/TypeIsPublic.php

@@ -0,0 +1,37 @@
+<?php
+
+namespace LaravelEnso\Files\Upgrades;
+
+use Illuminate\Support\Facades\Schema;
+use LaravelEnso\Files\Contracts\PublicFile;
+use LaravelEnso\Files\Models\Type;
+use LaravelEnso\Upgrade\Contracts\MigratesData;
+use LaravelEnso\Upgrade\Contracts\MigratesTable;
+use LaravelEnso\Upgrade\Helpers\Table;
+
+class TypeIsPublic implements MigratesTable, MigratesData
+{
+    public function isMigrated(): bool
+    {
+        return ! Table::hasColumn('file_types', 'is_public');
+    }
+
+    public function migrateTable(): void
+    {
+        Schema::table('file_types', fn ($table) => $table->boolean('is_public')
+            ->default(false)
+            ->after('description'));
+
+        Schema::table('file_types', fn ($table) => $table->boolean('is_public')
+            ->default(null)
+            ->change());
+    }
+
+    public function migrateData(): void
+    {
+        Type::whereNotNull('model')->get()
+            ->each(fn ($type) => $type->update([
+                'is_public' => $type->model() instanceof PublicFile,
+            ]));
+    }
+}