Merge pull request #77 from hazaveh/master

christophrumpel · web-flow · commit 9c850a20eb73 · 2022-01-31T09:43:55.000+01:00
use per user settings only if notifiable is a type of notifiable
diff --git a/src/TwitterChannel.php b/src/TwitterChannel.php
@@ -49,7 +49,7 @@ public function send($notifiable, Notification $notification)
      */
     private function changeTwitterSettingsIfNeeded($notifiable)
     {
-        if ($twitterSettings = $notifiable->routeNotificationFor('twitter')) {
+        if (method_exists($notifiable, 'routeNotificationFor') && $twitterSettings = $notifiable->routeNotificationFor('twitter')) {
             $this->twitter = new TwitterOAuth($twitterSettings[0], $twitterSettings[1], $twitterSettings[2],
                 $twitterSettings[3]);
         }

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public function send($notifiable, Notification $notification)`
`49`	`49`	`*/`
`50`	`50`	`private function changeTwitterSettingsIfNeeded($notifiable)`
`51`	`51`	`{`
`52`		`- if ($twitterSettings = $notifiable->routeNotificationFor('twitter')) {`
	`52`	`+ if (method_exists($notifiable, 'routeNotificationFor') && $twitterSettings = $notifiable->routeNotificationFor('twitter')) {`
`53`	`53`	`$this->twitter = new TwitterOAuth($twitterSettings[0], $twitterSettings[1], $twitterSettings[2],`
`54`	`54`	`$twitterSettings[3]);`
`55`	`55`	`}`