Merge pull request #1 from perifer/master

Error support for Laravel 5.8 New minimum requirements are now Laravel 5.4
larsemil · Aug 8, 2019 · cc34c8e · cc34c8e
2 parents f96718b + 1a488a5
commit cc34c8e
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 12 deletions.
diff --git a/composer.json b/composer.json
@@ -15,8 +15,8 @@
         "php": ">=5.6.4",
         "guzzlehttp/guzzle": "^6.3",
         "illuminate/events": "^5.7",
-        "illuminate/notifications": "^5.3",
-        "illuminate/support": "^5.1|^5.2|^5.3"
+        "illuminate/notifications": "^5.4",
+        "illuminate/support": "^5.4"
     },
     "require-dev": {
         "mockery/mockery": "^0.9.5",

diff --git a/src/FortySixElksChannel.php b/src/FortySixElksChannel.php
@@ -37,7 +37,7 @@ public function send($notifiable, Notification $notification)
                 try {
                     $media->send();
                 } catch(\Exception $e){
-                    $this->events->fire(new NotificationFailed($notifiable, $notification, get_class($this), ['exception' => $e]));
+                    $this->events->dispatch(new NotificationFailed($notifiable, $notification, get_class($this), ['exception' => $e]));
                 }
             }
 		}

diff --git a/src/FortySixElksMMS.php b/src/FortySixElksMMS.php
@@ -32,7 +32,6 @@ public function image($url){
     public function send() {
 
         try {
-            echo 'doing request';
             $response = $this->client->request( 'POST', $this->endpoint, [
                 'form_params' => [
                     'from'     => $this->from,
@@ -42,18 +41,14 @@ public function send() {
                 ],
 
             ] );
-            echo 'did request';
-            var_dump($response);
-        } catch ( GuzzleHttp\Exception\BadResponseException $e ) {
+        } catch ( \GuzzleHttp\Exception\BadResponseException $e ) {
             $response = $e->getResponse();
-            var_dump('WHOOT');
             throw CouldNotSendNotification::serviceRespondedWithAnError($response->getBody()->getContents(), $response->getStatusCode());
 
         }
-        echo "hmm";
         return $this;
     }
 
 
 
-}
+}
diff --git a/src/FortySixElksSMS.php b/src/FortySixElksSMS.php
@@ -37,7 +37,7 @@ public function send() {
 				],
 
 			] );
-		} catch ( GuzzleHttp\Exception\BadResponseException $e ) {
+		} catch ( \GuzzleHttp\Exception\BadResponseException $e ) {
 			$response = $e->getResponse();
 			throw CouldNotSendNotification::serviceRespondedWithAnError($response->getBody()->getContents(), $response->getStatusCode());
 		}
@@ -49,4 +49,4 @@ public function flash() {
 		$this->payload['flash'] = true;
 		return $this;
 	}
-}
+}