Add index to the date column (#429)

* Add index to the date column

* Add migration for existing installs

* Only run migration if we have entries table

* Correct version number

---------

Co-authored-by: Ryan Mitchell <[email protected]>

Author: afonic

database/migrations/entries/2024_03_07_100000_create_entries_table.php

@@ -15,7 +15,7 @@ public function up()
             $table->boolean('published')->default(true);
             $table->string('slug')->nullable();
             $table->string('uri')->nullable()->index();
-            $table->string('date')->nullable();
+            $table->string('date')->nullable()->index();
             $table->integer('order')->nullable()->index();
             $table->string('collection')->index();
             $table->string('blueprint', 30)->nullable()->index();

database/migrations/entries/2024_03_07_100000_create_entries_table_with_string_ids.php

@@ -15,7 +15,7 @@ public function up()
             $table->boolean('published')->default(true);
             $table->string('slug')->nullable();
             $table->string('uri')->nullable()->index();
-            $table->string('date')->nullable();
+            $table->string('date')->nullable()->index();
             $table->integer('order')->nullable()->index();
             $table->string('collection')->index();
             $table->string('blueprint', 30)->nullable()->index();

database/migrations/updates/add_index_to_date_on_entries_table.php.stub

@@ -0,0 +1,29 @@
+<?php
+
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+use Statamic\Eloquent\Database\BaseMigration as Migration;
+
+return new class extends Migration {
+    public function up()
+    {
+        if (Schema::hasIndex($this->prefix('entries'), 'date')) {
+            return;
+        }
+
+        Schema::table($this->prefix('entries'), function (Blueprint $table) {
+            $table->index(['date']);
+        });
+    }
+
+    public function down()
+    {
+        if (! Schema::hasIndex($this->prefix('entries'), 'date')) {
+            return;
+        }
+
+        Schema::table($this->prefix('entries'), function (Blueprint $table) {
+            $table->dropIndex(['date']);
+        });
+    }
+};

src/ServiceProvider.php

@@ -57,6 +57,7 @@ class ServiceProvider extends AddonServiceProvider
         \Statamic\Eloquent\Updates\RelateFormSubmissionsByHandle::class,
         \Statamic\Eloquent\Updates\DropStatusOnEntries::class,
         \Statamic\Eloquent\Updates\ChangeFormSubmissionsIdType::class,
+        \Statamic\Eloquent\Updates\AddIndexToDateOnEntriesTable::class,
     ];
 
     public function boot()

src/Updates/AddIndexToDateOnEntriesTable.php

@@ -0,0 +1,27 @@
+<?php
+
+namespace Statamic\Eloquent\Updates;
+
+use Illuminate\Support\Facades\Schema;
+use Statamic\UpdateScripts\UpdateScript;
+
+class AddIndexToDateOnEntriesTable extends UpdateScript
+{
+    public function shouldUpdate($newVersion, $oldVersion)
+    {
+        return $this->isUpdatingTo('4.22.0')
+            && Schema::hasTable(config('statamic.eloquent-driver.table_prefix', '').'entries')
+            && ! Schema::hasIndex(config('statamic.eloquent-driver.table_prefix', '').'entries', 'date');
+    }
+
+    public function update()
+    {
+        $source = __DIR__.'/../../database/migrations/updates/add_index_to_date_on_entries_table.php.stub';
+        $dest = database_path('migrations/'.date('Y_m_d_His').'_add_index_to_date_on_entries_table.php');
+
+        $this->files->copy($source, $dest);
+
+        $this->console()->info('Migration created');
+        $this->console()->comment('Remember to run `php artisan migrate` to apply it to your database.');
+    }
+}