Edit series

Author: joedixon

app/Http/Controllers/Articles/SeriesController.php

@@ -7,7 +7,10 @@
 use App\Http\Middleware\RedirectIfUnconfirmed;
 use App\Http\Requests\SeriesRequest;
 use App\Jobs\CreateSeries;
+use App\Jobs\UpdateSeries;
+use App\Models\Series;
 use App\Models\Tag;
+use App\Policies\SeriesPolicy;
 
 class SeriesController extends Controller
 {
@@ -16,6 +19,11 @@ public function __construct()
         $this->middleware([Authenticate::class, RedirectIfUnconfirmed::class], ['except' => ['index', 'show']]);
     }
 
+    public function show(Series $series)
+    {
+    
+    }
+
     public function create()
     {
         $tags = Tag::all();
@@ -32,4 +40,23 @@ public function store(SeriesRequest $request)
 
         return redirect()->route('series.show', $series->id());
     }
+
+    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]);
+    }
+
+    public function update(SeriesRequest $request, Series $series)
+    {
+        $this->authorize(SeriesPolicy::UPDATE, $series);
+
+        $series = $this->dispatchNow(UpdateSeries::fromRequest($series, $request));
+
+        $this->success('series.updated');
+
+        return redirect()->route('series.show', $series->id());
+    }
 }

app/Jobs/UpdateSeries.php

@@ -0,0 +1,41 @@
+<?php
+
+namespace App\Jobs;
+
+use App\Http\Requests\SeriesRequest;
+use App\Models\Series;
+use Illuminate\Support\Arr;
+
+final class UpdateSeries
+{
+    private $series;
+
+    private $attributes;
+
+    public function __construct(Series $series, array $attributes = [])
+    {
+        $this->series = $series;
+        $this->attributes = Arr::only($attributes, ['title', 'tags']);
+    }
+
+    public static function fromRequest(Series $series, SeriesRequest $request): self
+    {
+        return new static($series, [
+            'title' => $request->title(),
+            'tags' => $request->tags(),
+        ]);
+    }
+
+    public function handle(): Series
+    {
+        $this->series->update($this->attributes);
+
+        if (Arr::has($this->attributes, 'tags')) {
+            $this->series->syncTags($this->attributes['tags']);
+        }
+
+        $this->series->save();
+
+        return $this->series;
+    }
+}

app/Policies/SeriesPolicy.php

@@ -0,0 +1,22 @@
+<?php
+
+namespace App\Policies;
+
+use App\Models\Series;
+use App\User;
+
+final class SeriesPolicy
+{
+    const UPDATE = 'update';
+    const DELETE = 'delete';
+
+    public function update(User $user, Series $series): bool
+    {
+        return $series->isAuthoredBy($user) || $user->isModerator() || $user->isAdmin();
+    }
+
+    public function delete(User $user, Series $series): bool
+    {
+        return $series->isAuthoredBy($user) || $user->isModerator() || $user->isAdmin();
+    }
+}

app/Providers/AuthServiceProvider.php

@@ -4,10 +4,12 @@
 
 use App\Models\Article;
 use App\Models\Reply;
+use App\Models\Series;
 use App\Models\Thread;
 use App\Policies\ArticlePolicy;
 use App\Policies\NotificationPolicy;
 use App\Policies\ReplyPolicy;
+use App\Policies\SeriesPolicy;
 use App\Policies\ThreadPolicy;
 use App\Policies\UserPolicy;
 use App\User;
@@ -22,6 +24,7 @@ class AuthServiceProvider extends ServiceProvider
         Thread::class => ThreadPolicy::class,
         User::class => UserPolicy::class,
         Article::class => ArticlePolicy::class,
+        Series::class => SeriesPolicy::class,
     ];
 
     /**

database/factories/SeriesFactory.php

@@ -1,6 +1,6 @@
 <?php
 
-use App\Series;
+use App\Models\Series;
 use Faker\Generator as Faker;
 
 $factory->define(Series::class, function (Faker $faker) {

resources/lang/en/series.php

@@ -3,5 +3,6 @@
 return [
 
     'created' => 'Series successfully created!',
+    'updated' => 'Series successfully updated!',
 
 ];

resources/views/articles/series/edit.blade.php

@@ -0,0 +1,16 @@
+@title('Update series')
+
+@extends('layouts.default')
+
+@section('content')
+    <div class="container mx-auto p-4 flex justify-center">
+        <div class="w-full md:w-2/3 xl:w-1/2">
+            <div class="md:p-4 md:border-2 md:rounded md:bg-gray-100">
+                @include('articles.series._form', [
+                    'route' => ['series.update', $series->id()],
+                    'method' => 'PUT',
+                ])
+            </div>
+        </div>
+    </div>
+@endsection

routes/web.php

@@ -83,6 +83,9 @@
     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');

tests/Feature/SeriesTest.php

@@ -3,6 +3,7 @@
 namespace Tests\Feature;
 
 use App\Models\Article;
+use App\Models\Series;
 use App\Models\Tag;
 use Illuminate\Foundation\Testing\DatabaseMigrations;
 
@@ -44,4 +45,41 @@ public function users_cannot_create_a_series_with_a_title_that_is_too_long()
         $response->assertSessionHas('error', 'Something went wrong. Please review the fields below.');
         $response->assertSessionHasErrors(['title' => 'The title may not be greater than 100 characters.']);
     }
+
+    /** @test */
+    public function users_can_edit_a_series()
+    {
+        $user = $this->createUser();
+        $tag = factory(Tag::class)->create(['name' => 'Test Tag']);
+        factory(Series::class)->create([
+            'author_id' => $user->id(),
+        ]);
+
+        $this->loginAs($user);
+
+        $this->put('/articles/series/1', [
+            'title' => 'A developers guide to SVG',
+            'tags' => [$tag->id()],
+        ])
+            ->assertRedirectedTo('/articles/series/1')
+            ->assertSessionHas('success', 'Series successfully updated!');
+    }
+
+    /** @test */
+    public function users_cannot_edit_a_series_with_a_title_that_is_too_long()
+    {
+        $user = $this->createUser();
+        factory(Series::class)->create([
+            'author_id' => $user->id()
+        ]);
+
+        $this->loginAs($user);
+
+        $response = $this->put('/articles/series/1', [
+            'title' => 'A developers guide to SVG which is an image format written in XML which is highly customisable and flexible',
+        ]);
+
+        $response->assertSessionHas('error', 'Something went wrong. Please review the fields below.');
+        $response->assertSessionHasErrors(['title' => 'The title may not be greater than 100 characters.']);
+    }
 }

tests/Integration/Jobs/UpdateArticleTest.php

@@ -12,7 +12,7 @@ class UpdateArticleTest extends TestCase
     use DatabaseMigrations;
 
     /** @test */
-    public function we_can_create_an_article()
+    public function we_can_update_an_article()
     {
         $user = $this->createUser();
         $article = factory(Article::class)->create(['author_id' => $user->id()]);