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

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/Requests/AddPasteRequest.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/UpdatePasteRequest.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/Providers/RouteServiceProvider.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/Rules/PasswordMatch.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.";
+    }
+}

database/factories/LanguageFactory.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),
     ];
 });

database/factories/PasteFactory.php

@@ -28,3 +28,9 @@
         "code" => $faker->paragraph,
     ];
 });
+
+$factory->state(\Wdi\Entities\Paste::class, "with-password", function (Faker\Generator $faker) {
+    return [
+        "password" => $faker->password,
+    ];
+});

database/migrations/2017_11_07_065216_add_password_column_to_pastes_table.php

@@ -0,0 +1,34 @@
+<?php
+
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+
+/**
+ * Class AddPasswordColumnToPastesTable
+ */
+final class AddPasswordColumnToPastesTable extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        Schema::table("pastes", function (Blueprint $table) {
+            $table->string("password")->nullable()->after("description");
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::table("pastes", function (Blueprint $table) {
+            $table->dropColumn("password");
+        });
+    }
+}

database/seeds/PastesTableSeeder.php

@@ -14,13 +14,20 @@ final class PastesTableSeeder extends Seeder
      */
     public function run()
     {
+        // Create a password fixed test
+        factory(\Wdi\Entities\Paste::class)->states("with-password")->create([
+            "language_id" => \Wdi\Entities\Language::first()->id,
+            "name" => "DatabaseFactory",
+            "password" => "foobar",
+        ]);
+
         // Create a bunch of anon pastes
-        factory(\Wdi\Entities\Paste::class, 100)->create([
+        factory(\Wdi\Entities\Paste::class, 100)->states("with-password")->create([
             "language_id" => \Wdi\Entities\Language::first()->id,
         ]);
-
+        
         // Create forks of a single paste
-        factory(\Wdi\Entities\Paste::class, 10)->create([
+        factory(\Wdi\Entities\Paste::class, 10)->states("with-password")->create([
             "paste_id" => \Wdi\Entities\Paste::first()->id,
         ]);
     }

package.json

@@ -17,11 +17,11 @@
     "moment": "^2.19.1",
     "popper.js": "^1.12.5",
     "tether": "^1.4.0",
-    "vue": "^2.5.2",
+    "vue": "^2.5.3",
     "vuex": "^2.5.0"
   },
   "devDependencies": {
     "cross-env": "^5.1.1",
-    "laravel-mix": "^1.5.0"
+    "laravel-mix": "^1.6.1"
   }
 }