wip

Author: driesvints

app/Http/Controllers/SubscriptionController.php

@@ -0,0 +1,20 @@
+<?php
+
+namespace App\Http\Controllers;
+
+use App\Jobs\UnsubscribeFromSubscriptionAble;
+use App\Models\Subscription;
+
+class SubscriptionController extends Controller
+{
+    public function unsubscribe(Subscription $subscription)
+    {
+        $thread = $subscription->subscriptionAble();
+
+        $this->dispatch(new UnsubscribeFromSubscriptionAble($subscription->user(), $thread));
+
+        $this->success("You're now unsubscribed from this thread.");
+
+        return redirect()->route('thread', $thread->slug());
+    }
+}

app/Listeners/SendNewReplyNotification.php

@@ -12,7 +12,7 @@ public function handle(ReplyWasCreated $event): void
     {
         foreach ($event->reply->replyAble()->subscriptions() as $subscription) {
             if ($this->replyAuthorDoesNotMatchSubscriber($event->reply->author(), $subscription)) {
-                $subscription->user()->notify(new NewReplyNotification($event->reply));
+                $subscription->user()->notify(new NewReplyNotification($event->reply, $subscription));
             }
         }
     }

app/Mail/NewReplyEmail.php

@@ -3,6 +3,7 @@
 namespace App\Mail;
 
 use App\Models\Reply;
+use App\Models\Subscription;
 use Illuminate\Bus\Queueable;
 use Illuminate\Contracts\Queue\ShouldQueue;
 use Illuminate\Mail\Mailable;
@@ -17,9 +18,15 @@ class NewReplyEmail extends Mailable implements ShouldQueue
      */
     public $reply;
 
-    public function __construct(Reply $reply)
+    /**
+     * @var \App\Models\Subscription
+     */
+    public $subscription;
+
+    public function __construct(Reply $reply, Subscription $subscription)
     {
         $this->reply = $reply;
+        $this->subscription = $subscription;
     }
 
     public function build()

app/Models/Subscription.php

@@ -6,14 +6,25 @@
 use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Eloquent\Relations\BelongsTo;
 use Illuminate\Database\Eloquent\Relations\MorphTo;
+use Ramsey\Uuid\Uuid;
+use Ramsey\Uuid\UuidInterface;
+use Spatie\BinaryUuid\HasBinaryUuid;
+use Spatie\BinaryUuid\HasUuidPrimaryKey;
 
 class Subscription extends Model
 {
+    use HasBinaryUuid, HasUuidPrimaryKey;
+
     /**
      * {@inheritdoc}
      */
     protected $table = 'subscriptions';
 
+    public function uuid(): UuidInterface
+    {
+        return Uuid::fromString($this->uuid_text);
+    }
+
     public function user(): User
     {
         return $this->userRelation;

app/Notifications/NewReplyNotification.php

@@ -4,6 +4,7 @@
 
 use App\Mail\NewReplyEmail;
 use App\Models\Reply;
+use App\Models\Subscription;
 use App\User;
 use Illuminate\Bus\Queueable;
 use Illuminate\Notifications\Notification;
@@ -18,9 +19,15 @@ class NewReplyNotification extends Notification implements ShouldQueue
      */
     public $reply;
 
-    public function __construct(Reply $reply)
+    /**
+     * @var \App\Models\Subscription
+     */
+    public $subscription;
+
+    public function __construct(Reply $reply, Subscription $subscription)
     {
         $this->reply = $reply;
+        $this->subscription = $subscription;
     }
 
     public function via(User $user)
@@ -30,7 +37,7 @@ public function via(User $user)
 
     public function toMail(User $user)
     {
-        return (new NewReplyEmail($this->reply))
+        return (new NewReplyEmail($this->reply, $this->subscription))
             ->to($user->emailAddress(), $user->name());
     }
 

composer.json

@@ -15,6 +15,7 @@
         "lasserafn/php-initial-avatar-generator": "^2.0",
         "league/commonmark": "^0.15.2",
         "roave/security-advisories": "dev-master",
+        "spatie/laravel-binary-uuid": "^1.1",
         "spatie/laravel-robots-middleware": "^1.0",
         "tijsverkoyen/akismet": "^1.1"
     },

composer.lock

@@ -9,7 +9,8 @@ class CreateSubscriptionsTable extends Migration
     public function up()
     {
         Schema::create('subscriptions', function (Blueprint $table) {
-            $table->increments('id');
+            $table->uuid('uuid');
+            $table->primary('uuid');
             $table->integer('user_id')->unsigned();
             $table->integer('subscriptionable_id');
             $table->string('subscriptionable_type')->default('');

database/migrations/2017_10_18_193001_create_subscriptions_table.php

@@ -1,13 +1,13 @@
 @component('mail::message')
 
-**{{ $reply->replyAble()->author()->name() }}** has replied to this thread.
+**{{ $reply->author()->name() }}** has replied to this thread.
 
 @component('mail::panel')
 @md($reply->body())
 @endcomponent
 
 You are receiving this because you are subscribed to this thread.<br>
 [View it on Laravel.io]({{ route('thread', $reply->replyAble()->slug()) }}),
-or [unsubscribe]({{ route('threads.unsubscribe', $reply->replyAble()->slug()) }}).
+or [unsubscribe]({{ route('subscriptions.unsubscribe', $subscription->uuid()->toString()) }}).
 
 @endcomponent

resources/views/emails/new_reply.blade.php

@@ -6,6 +6,9 @@
 Route::bind('reply', function ($id) {
     return App\Models\Reply::findOrFail($id);
 });
+Route::bind('subscription', function ($uuid) {
+    return App\Models\Subscription::withUuid($uuid)->firstOrFail();
+});
 Route::bind('tag', function ($slug) {
     return App\Models\Tag::findBySlug($slug);
 });