don't resolve in the constructor

FriendsOfFlarum · Oct 21, 2023 · 517e680 · 517e680
1 parent 54612d6
commit 517e680
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/Api/Controller/UpdateReactionController.php b/src/Api/Controller/UpdateReactionController.php
@@ -22,10 +22,8 @@
 
 class UpdateReactionController extends AbstractShowController
 {
-    /**
-     * {@inheritdoc}
-     */
     public $serializer = ReactionSerializer::class;
+
     /**
      * @var Dispatcher
      */

diff --git a/src/Listener/SaveReactionsToDatabase.php b/src/Listener/SaveReactionsToDatabase.php
@@ -62,7 +62,6 @@ public function __construct(SettingsRepositoryInterface $settings, TranslatorInt
         $this->translator = $translator;
         $this->extensions = $extensions;
         $this->events = $events;
-        $this->request = resolve('fof-reactions.request');
     }
 
     /**
@@ -76,6 +75,8 @@ public function handle(Saving $event)
         $post = $event->post;
         $data = $event->data;
 
+        $this->request = resolve('fof-reactions.request');
+
         if ($post->exists && Arr::has($data, 'attributes.reaction')) {
             $actor = $event->actor;