Merge pull request #60 from boryn/return-exceptions-fix

fixed missing $notifiable in send() method in Pushover
laravel-notification-channels · Mar 11, 2024 · f0d6d9e · f0d6d9e
2 parents b334882 + f573ac6
commit f0d6d9e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/src/Pushover.php b/src/Pushover.php
@@ -55,11 +55,12 @@ public function __construct(HttpClient $http, $token)
      * @link  https://pushover.net/api
      *
      * @param  array  $params
+     * @param mixed $notifiable
      * @return \Psr\Http\Message\ResponseInterface
      *
      * @throws CouldNotSendNotification
      */
-    public function send($params)
+    public function send($params, $notifiable)
     {
         try {
             $multipart = [];

diff --git a/src/PushoverChannel.php b/src/PushoverChannel.php
@@ -47,7 +47,10 @@ public function send($notifiable, Notification $notification)
         $message = $notification->toPushover($notifiable);
 
         try {
-            $this->pushover->send(array_merge($message->toArray(), $pushoverReceiver->toArray()));
+            $this->pushover->send(
+                array_merge($message->toArray(), $pushoverReceiver->toArray()),
+                $notifiable
+            );
         } catch (ServiceCommunicationError $serviceCommunicationError) {
             $this->fireFailedEvent($notifiable, $notification, $serviceCommunicationError->getMessage());
         }