Rename locale to site on redirects table (#156)

* Fix validation for eloquent redirects

* Revert "Fix validation for eloquent redirects"

This reverts commit dbff7f0.

* Fix error on site creation

* Use site column in `findByUrl` query

database/migrations/create_eloquent_redirect_table.php.stub

@@ -15,7 +15,7 @@ class CreateEloquentRedirectTable extends Migration
                 $table->string('destination');
                 $table->char('match_type', 10);
                 $table->char('type', 10);
-                $table->string('locale');
+                $table->string('site')->index();
                 $table->integer('order')->nullable();
                 $table->boolean('enabled');
                 $table->timestamps();

src/Eloquent/Redirects/RedirectRepository.php

@@ -33,7 +33,7 @@ public function findByUrl(string $siteHandle, string $url): ?Redirect
     {
         return $this->query()
             ->where('enabled', true)
-            ->where('locale', $siteHandle)
+            ->where('site', $siteHandle)
             ->orderBy('order')
             ->get()
             ->map(function (Redirect $redirect) use ($url) {
@@ -87,7 +87,7 @@ public function make(): Redirect
     {
         return new Redirect();
     }
-    
+
     public static function fromModel(Model $model)
     {
         return (new Redirect)
@@ -98,9 +98,9 @@ public static function fromModel(Model $model)
             ->matchType($model->match_type)
             ->enabled($model->enabled)
             ->order($model->order)
-            ->locale($model->locale);
+            ->locale($model->site);
     }
-    
+
     private function toModel(Redirect $redirect)
     {
         return RedirectModel::firstOrNew([
@@ -112,7 +112,7 @@ private function toModel(Redirect $redirect)
             'type' => $redirect->type(),
             'enabled' => $redirect->enabled(),
             'order' => $redirect->order(),
-            'locale' => $redirect->locale(),
+            'site' => $redirect->locale(),
         ]);
     }
 }

src/RedirectServiceProvider.php

@@ -20,6 +20,7 @@
 use Rias\StatamicRedirect\UpdateScripts\AddHitsCount;
 use Rias\StatamicRedirect\UpdateScripts\ClearErrors;
 use Rias\StatamicRedirect\UpdateScripts\MoveRedirectsToDefaultSite;
+use Rias\StatamicRedirect\UpdateScripts\RenameLocaleToSiteOnRedirectsTable;
 use Rias\StatamicRedirect\Widgets\ErrorsLastDayWidget;
 use Rias\StatamicRedirect\Widgets\ErrorsLastMonthWidget;
 use Rias\StatamicRedirect\Widgets\ErrorsLastWeekWidget;
@@ -39,6 +40,7 @@ class RedirectServiceProvider extends AddonServiceProvider
         AddHitsCount::class,
         ClearErrors::class,
         MoveRedirectsToDefaultSite::class,
+        RenameLocaleToSiteOnRedirectsTable::class,
     ];
 
     protected $scripts = [
@@ -90,7 +92,7 @@ public function boot()
                     __DIR__ . '/../database/migrations/create_redirect_tables.php.stub' => database_path('migrations/' . date('Y_m_d_His', time()) . '_create_redirect_tables.php'),
                 ], 'migrations');
             }
-            
+
             if (! class_exists('CreateEloquentRedirectTable')) {
                 $this->publishes([
                     __DIR__ . '/../database/migrations/create_eloquent_redirect_table.php.stub' => database_path('migrations/' . date('Y_m_d_His', time()) . '_create_eloquent_redirect_table.php'),

src/UpdateScripts/RenameLocaleToSiteOnRedirectsTable.php

@@ -0,0 +1,24 @@
+<?php
+
+namespace Rias\StatamicRedirect\UpdateScripts;
+
+use Illuminate\Support\Facades\Schema;
+use Statamic\UpdateScripts\UpdateScript;
+
+class RenameLocaleToSiteOnRedirectsTable extends UpdateScript
+{
+    public function shouldUpdate($newVersion, $oldVersion)
+    {
+        return Schema::hasColumn('redirects', 'locale');
+    }
+
+    public function update()
+    {
+        Schema::table('redirects', function ($table) {
+            $table->renameColumn('locale', 'site');
+            $table->index(['site']);
+        });
+
+        $this->console()->info('Column renamed succesfully!');
+    }
+}