Use slugs

Author: joedixon

app/Http/Controllers/Articles/SeriesController.php

@@ -29,7 +29,7 @@ public function create()
         $tags = Tag::all();
         $selectedTags = old('tags') ?: [];
 
-        return view('articles.series.create', ['tags' => $tags, 'selectedTags' => $selectedTags]);
+        return view('series.create', ['tags' => $tags, 'selectedTags' => $selectedTags]);
     }
 
     public function store(SeriesRequest $request)
@@ -38,15 +38,15 @@ public function store(SeriesRequest $request)
 
         $this->success('series.created');
 
-        return redirect()->route('series.show', $series->id());
+        return redirect()->route('series.show', $series->slug());
     }
 
     public function edit(Series $series)
     {
         $this->authorize(SeriesPolicy::UPDATE, $series);
         $selectedTags = $series->tags()->pluck('id')->toArray();
 
-        return view('articles.series.edit', ['series' => $series, 'tags' => Tag::all(), 'selectedTags' => $selectedTags]);
+        return view('series.edit', ['series' => $series, 'tags' => Tag::all(), 'selectedTags' => $selectedTags]);
     }
 
     public function update(SeriesRequest $request, Series $series)
@@ -57,6 +57,6 @@ public function update(SeriesRequest $request, Series $series)
 
         $this->success('series.updated');
 
-        return redirect()->route('series.show', $series->id());
+        return redirect()->route('series.show', $series->slug());
     }
 }

app/Jobs/CreateSeries.php

@@ -43,6 +43,7 @@ public function handle(): Series
     {
         $series = new Series([
             'title' => $this->title,
+            'slug' => $this->title,
         ]);
         $series->authoredBy($this->author);
         $series->syncTags($this->tags);

app/Jobs/UpdateSeries.php

@@ -15,13 +15,14 @@ final class UpdateSeries
     public function __construct(Series $series, array $attributes = [])
     {
         $this->series = $series;
-        $this->attributes = Arr::only($attributes, ['title', 'tags']);
+        $this->attributes = Arr::only($attributes, ['title', 'slug', 'tags']);
     }
 
     public static function fromRequest(Series $series, SeriesRequest $request): self
     {
         return new static($series, [
             'title' => $request->title(),
+            'slug' => $request->title(),
             'tags' => $request->tags(),
         ]);
     }

app/Models/Series.php

@@ -3,18 +3,20 @@
 namespace App\Models;
 
 use App\Helpers\HasAuthor;
+use App\Helpers\HasSlug;
 use App\Helpers\HasTags;
 use Illuminate\Database\Eloquent\Model;
 
 class Series extends Model
 {
-    use HasAuthor, HasTags;
+    use HasAuthor, HasTags, HasSlug;
 
     /**
      * {@inheritdoc}
      */
     protected $fillable = [
         'title',
+        'slug',
     ];
 
     public function id(): int

database/migrations/2020_04_07_181731_create_series_table.php

@@ -17,6 +17,7 @@ public function up()
             $table->increments('id');
             $table->unsignedInteger('author_id');
             $table->string('title');
+            $table->string('slug')->unique();
             $table->timestamps();
 
             $table->foreign('author_id')

resources/views/articles/series/_form.blade.php renamed to resources/views/series/_form.blade.php

@@ -24,6 +24,6 @@
 Route::bind('article', function (string $slug) {
     return App\Models\Article::findBySlug($slug);
 });
-Route::bind('series', function (string $id) {
-    return App\Models\Series::findOrFail($id);
+Route::bind('series', function (string $slug) {
+    return App\Models\Series::findBySlug($slug);
 });

resources/views/articles/series/create.blade.php renamed to resources/views/series/create.blade.php

@@ -79,15 +79,6 @@
 
 // Articles
 Route::prefix('articles')->namespace('Articles')->group(function () {
-    // Series
-    Route::prefix('series')->group(function () {
-        Route::get('/create', 'SeriesController@create')->name('series.create');
-        Route::post('/', 'SeriesController@store')->name('series.store');
-        Route::get('{series}', 'SeriesController@show')->name('series.show');
-        Route::get('{series}/edit', 'SeriesController@edit')->name('series.edit');
-        Route::put('{series}', 'SeriesController@update')->name('series.update');
-    });
-
     Route::get('/', 'ArticlesController@index')->name('articles');
     Route::get('/create', 'ArticlesController@create')->name('articles.create');
     Route::post('/', 'ArticlesController@store')->name('articles.store');
@@ -97,6 +88,15 @@
     Route::delete('{article}', 'ArticlesController@delete')->name('articles.delete');
 });
 
+// Series
+Route::prefix('series')->namespace('Articles')->group(function () {
+    Route::get('/create', 'SeriesController@create')->name('series.create');
+    Route::post('/', 'SeriesController@store')->name('series.store');
+    Route::get('{series}', 'SeriesController@show')->name('series.show');
+    Route::get('{series}/edit', 'SeriesController@edit')->name('series.edit');
+    Route::put('{series}', 'SeriesController@update')->name('series.update');
+});
+
 // Admin
 Route::prefix('admin')->name('admin')->namespace('Admin')->group(function () {
     Route::get('/', 'AdminController@index');