Anonymous edit feature (#9)

* Remove deprecated middleware
* Add password to Paste entity
* Add new custom rule to determine password match
* Improve paste menu
* Add new FormRequest for update paste
* Add fixed Paste test on PasteTableSeeder
* Add the ability to edit a paste
* Improve tests suite
* Remove dead code
* Add new alert style for callouts
* Add the ability to set a password on paste creation and fork
* Fix a bug that caused crashes when a password was null
* Fix a stupid bug on LanguageFactory causing tests to fail

Author: ludo237

app/Entities/Paste.php

@@ -17,11 +17,14 @@ final class Paste extends Model
     protected $table = self::TABLE_NAME;
 
     /** {@inheritdoc} */
-    protected $fillable = ["paste_id", "language_id", "slug", "name", "extension", "code", "description"];
+    protected $fillable = ["paste_id", "language_id", "slug", "name", "extension", "code", "description", "password"];
 
     /** {@inheritdoc} */
     protected $guarded = ["id"];
 
+    /** {@inheritdoc} */
+    protected $hidden = ["password"];
+    
     /** {@inheritdoc} */
     protected $casts = [
         "paste_id" => "integer",
@@ -45,6 +48,18 @@ public function getFileNameAttribute() : string
         return "{$this->name}.{$this->extension}";
     }
 
+    /**
+     * Use built-in function to store the password for
+     * a paste
+     *
+     * @param string|null $value
+     * @return void
+     */
+    public function setPasswordAttribute($value)
+    {
+        $this->attributes["password"] = !is_null($value) ? bcrypt($value) : $value;
+    }
+    
     /**
      * Determine if a paste is a fork
      * of another one

app/Exceptions/Handler.php

@@ -43,14 +43,4 @@ public function render($request, Exception $exception)
     {
         return parent::render($request, $exception);
     }
-    
-    /** {@inheritdoc} */
-    protected function unauthenticated($request, AuthenticationException $exception)
-    {
-        if ($request->expectsJson()) {
-            return response()->json(["error" => "Unauthenticated."], 401);
-        }
-        
-        return redirect()->guest(route("login"));
-    }
 }

app/Http/Handlers/Paste/EditPasteHandler.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Wdi\Http\Handlers\Paste;
+
+use Illuminate\Http\Request;
+use Illuminate\View\View;
+use Wdi\Http\Handlers\Handler;
+
+/**
+ * Class EditPasteHandler
+ * @package Wdi\Http\Handlers\Paste
+ */
+final class EditPasteHandler extends Handler
+{
+    /**
+     * @param \Illuminate\Http\Request $request
+     * @return \Illuminate\View\View
+     */
+    public function __invoke(Request $request) : View
+    {
+        return view("paste.edit")->with("paste", $request->paste);
+    }
+}

app/Http/Handlers/Paste/UpdatePasteHandler.php

@@ -0,0 +1,26 @@
+<?php
+
+namespace Wdi\Http\Handlers\Paste;
+
+use Illuminate\Http\RedirectResponse;
+use Wdi\Http\Handlers\Handler;
+use Wdi\Http\Requests\UpdatePasteRequest;
+
+/**
+ * Class UpdatePasteHandler
+ * @package Wdi\Http\Handlers\Paste
+ */
+final class UpdatePasteHandler extends Handler
+{
+    /**
+     * @param \Wdi\Http\Requests\UpdatePasteRequest $request
+     * @return \Illuminate\Http\RedirectResponse
+     */
+    public function __invoke(UpdatePasteRequest $request) : RedirectResponse
+    {
+        $paste = $request->paste;
+        $paste->update($request->only(["name", "description", "language_id", "extension", "code"]));
+        
+        return redirect()->route("paste.show", $paste);
+    }
+}

app/Http/Middleware/RedirectIfAuthenticated.php

@@ -36,12 +36,19 @@ public function rules() : array
                 "min:3",
             ],
             "code" => [
-                "required"
+                "required",
+            ],
+            "password" => [
+                "sometimes",
+                "confirmed",
             ],
             "language_id" => [
                 "required",
-                Rule::exists(Language::TABLE_NAME, "id")
-            ]
+                Rule::exists(Language::TABLE_NAME, "id"),
+            ],
+            "extension" => [
+                "required",
+            ],
         ];
     }
 }

app/Http/Requests/AddPasteRequest.php

@@ -0,0 +1,54 @@
+<?php
+
+namespace Wdi\Http\Requests;
+
+use Illuminate\Foundation\Http\FormRequest;
+use Illuminate\Validation\Rule;
+use Wdi\Entities\Language;
+use Wdi\Rules\PasswordMatch;
+
+/**
+ * Class UpdatePasteRequest
+ * @package Wdi\Http\Requests
+ */
+final class UpdatePasteRequest extends FormRequest
+{
+    /**
+     * Determine if the user is authorized to make this request.
+     *
+     * @return bool
+     */
+    public function authorize() : bool
+    {
+        return true;
+    }
+    
+    /**
+     * Get the validation rules that apply to the request.
+     *
+     * @return array
+     */
+    public function rules() : array
+    {
+        return [
+            "name" => [
+                "required",
+                "min:3",
+            ],
+            "code" => [
+                "required",
+            ],
+            "password" => [
+                "required",
+                new PasswordMatch($this->paste->password),
+            ],
+            "language_id" => [
+                "required",
+                Rule::exists(Language::TABLE_NAME, "id"),
+            ],
+            "extension" => [
+                "required",
+            ],
+        ];
+    }
+}

app/Http/Requests/UpdatePasteRequest.php

@@ -23,7 +23,7 @@ public function boot()
         Route::model("language", Language::class);
 
         Route::bind("paste", function ($slug) {
-            return Paste::select(["id", "language_id", "paste_id", "user_id", "slug", "name", "extension", "code", "description", "created_at", "updated_at"])
+            return Paste::select(["id", "language_id", "paste_id", "user_id", "slug", "name", "extension", "code", "description", "password", "created_at", "updated_at"])
                 ->where("slug", $slug)->firstOrFail();
         });
     }

app/Providers/RouteServiceProvider.php

@@ -0,0 +1,47 @@
+<?php
+
+namespace Wdi\Rules;
+
+use Hash;
+use Illuminate\Contracts\Validation\Rule;
+
+/**
+ * Class PasswordMatch
+ * @package Wdi\Rules
+ */
+final class PasswordMatch implements Rule
+{
+    /** @var string */
+    private $password;
+    
+    /**
+     * Create a new rule instance.
+     * @param string $password
+     */
+    public function __construct(string $password)
+    {
+        $this->password = $password;
+    }
+    
+    /**
+     * Determine if the validation rule passes.
+     *
+     * @param  string $attribute
+     * @param  mixed $value
+     * @return bool
+     */
+    public function passes($attribute, $value)
+    {
+        return Hash::check($value, $this->password);
+    }
+    
+    /**
+     * Get the validation error message.
+     *
+     * @return string
+     */
+    public function message()
+    {
+        return "La password non corrisponde.";
+    }
+}

app/Rules/PasswordMatch.php

@@ -3,7 +3,7 @@
 /** @var \Illuminate\Database\Eloquent\Factory $factory */
 $factory->define(\Wdi\Entities\Language::class, function (Faker\Generator $faker) {
     return [
-        "name" => $faker->unique()->word,
+        "name" => $faker->lexify("???????"),
         "extensions" => $faker->randomElements(config("procedural.extensions"), 3),
     ];
 });