Merge pull request #4 from crynobone/patch-1

[5.8] Use Facade::resolved() before extending channel
laravel · Jan 6, 2019 · 4590d38 · 4590d38
2 parents 9b3ac5e + 3c8e8fb
commit 4590d38
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/src/NexmoChannelServiceProvider.php b/src/NexmoChannelServiceProvider.php
@@ -4,6 +4,7 @@
 
 use Nexmo\Client as NexmoClient;
 use Illuminate\Support\ServiceProvider;
+use Illuminate\Notifications\ChannelManager;
 use Illuminate\Support\Facades\Notification;
 use Nexmo\Client\Credentials\Basic as NexmoCredentials;
 
@@ -16,14 +17,16 @@ class NexmoChannelServiceProvider extends ServiceProvider
      */
     public function register()
     {
-        Notification::extend('nexmo', function ($app) {
-            return new Channels\NexmoSmsChannel(
-                new NexmoClient(new NexmoCredentials(
-                    $this->app['config']['services.nexmo.key'],
-                    $this->app['config']['services.nexmo.secret']
-                )),
-                $this->app['config']['services.nexmo.sms_from']
-            );
+        Notification::resolved(function (ChannelManager $service) {
+            $service->extend('nexmo', function ($app) {
+                return new Channels\NexmoSmsChannel(
+                    new NexmoClient(new NexmoCredentials(
+                        $this->app['config']['services.nexmo.key'],
+                        $this->app['config']['services.nexmo.secret']
+                    )),
+                    $this->app['config']['services.nexmo.sms_from']
+                );
+            });
         });
     }
 }