fix: laravel 9, make default name static (#70)

Author: atymic

composer.json

@@ -32,16 +32,16 @@
         }
     ],
     "require": {
-        "php": ">=7.0",
-        "illuminate/events": "~5.5 || ~6.0 || ~7.0 || ~8.0",
-        "illuminate/notifications": "~5.5 || ~6.0 || ~7.0 || ~8.0",
-        "illuminate/queue": "~5.5 || ~6.0 || ~7.0 || ~8.0",
-        "illuminate/support": "~5.5 || ~6.0 || ~7.0 || ~8.0",
+        "php": ">=7.4",
+        "illuminate/events": "~7.0 || ~8.0 || ~9.0",
+        "illuminate/notifications": "~7.0 || ~8.0 || ~9.0",
+        "illuminate/queue": "~7.0 || ~8.0 || ~9.0",
+        "illuminate/support": "~7.0 || ~8.0 || ~9.0",
         "pusher/pusher-push-notifications": "^1.1"
     },
     "require-dev": {
         "mockery/mockery": "^1.3",
-        "phpunit/phpunit": "^8.5"
+        "phpunit/phpunit": "^9.5"
     },
     "autoload": {
         "psr-4": {

src/PusherChannel.php

@@ -1,5 +1,7 @@
 <?php
 
+declare(strict_types=1);
+
 namespace NotificationChannels\PusherPushNotifications;
 
 use Illuminate\Contracts\Events\Dispatcher;
@@ -12,25 +14,12 @@
 
 class PusherChannel
 {
-    /**
-     * @var string
-     */
-    const INTERESTS = 'interests';
+    public const INTERESTS = 'interests';
 
-    /**
-     * @var PushNotifications
-     */
-    protected $beamsClient;
+    protected PushNotifications $beamsClient;
 
-    /**
-     * @var \Illuminate\Contracts\Events\Dispatcher
-     */
-    private $events;
+    private Dispatcher $events;
 
-    /**
-     * @param PushNotifications $beamsClient
-     * @param Dispatcher $events
-     */
     public function __construct(PushNotifications $beamsClient, Dispatcher $events)
     {
         $this->beamsClient = $beamsClient;
@@ -40,17 +29,16 @@ public function __construct(PushNotifications $beamsClient, Dispatcher $events)
     /**
      * Send the given notification.
      *
-     * @param mixed $notifiable
-     * @param \Illuminate\Notifications\Notification $notification
-     *
+     * @param  mixed  $notifiable
+     * @param  Notification  $notification
      * @return void
      */
-    public function send($notifiable, Notification $notification)
+    public function send($notifiable, Notification $notification): void
     {
         $type = $notifiable->pushNotificationType ?? self::INTERESTS;
 
         $data = $notifiable->routeNotificationFor('PusherPushNotifications')
-            ?: $this->defaultName($notifiable);
+            ?: self::defaultName($notifiable);
 
         try {
             $notificationType = sprintf('publishTo%s', Str::ucfirst($type));
@@ -69,11 +57,10 @@ public function send($notifiable, Notification $notification)
     /**
      * Get the default name for the notifiable.
      *
-     * @param $notifiable
-     *
+     * @param  mixed  $notifiable
      * @return string
      */
-    protected function defaultName($notifiable)
+    public static function defaultName($notifiable): string
     {
         $class = str_replace('\\', '.', get_class($notifiable));