Merge pull request #55 from bookboon/hotfix/errors

Update error handling in BookboonProvider
bookboon · May 7, 2020 · 64c67ea · 64c67ea
2 parents 3b26ff3 + 30483ff
commit 64c67ea
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/Client/Oauth/BookboonProvider.php b/src/Client/Oauth/BookboonProvider.php
@@ -25,7 +25,7 @@ class BookboonProvider extends AbstractProvider
 
     public function __construct(array $options = [], array $collaborators = [])
     {
-        if (isset($options['baseUri']) && $options['baseUri'] != "") {
+        if (isset($options['baseUri']) && $options['baseUri'] !== "") {
             $parts = explode('://', $options['baseUri']);
             $this->protocol = $parts[0];
             $this->host = $parts[1];
@@ -50,7 +50,7 @@ public function __construct(array $options = [], array $collaborators = [])
      */
     public function getBaseAuthorizationUrl()
     {
-        return $this->protocol . "://" . $this->host . "/login/authorize";
+        return "$this->protocol://$this->host/login/authorize";
     }
 
     /**
@@ -63,7 +63,7 @@ public function getBaseAuthorizationUrl()
      */
     public function getBaseAccessTokenUrl(array $params)
     {
-        return $this->protocol . "://" . $this->host . "/login/access_token";
+        return "$this->protocol://$this->host/login/access_token";
     }
 
     /**
@@ -74,7 +74,7 @@ public function getBaseAccessTokenUrl(array $params)
      */
     public function getResourceOwnerDetailsUrl(AccessToken $token)
     {
-        return $this->protocol . "://" . $this->host . "/login/userinfo";
+        return "$this->protocol://$this->host/login/userinfo";
     }
 
 
@@ -96,7 +96,7 @@ public function getAccessToken($grant, array $options = [])
         }
 
         if (is_array($options['scope'])) {
-            $options['scope'] = join($this->getScopeSeparator(), $options['scope']);
+            $options['scope'] = implode($this->getScopeSeparator(), $options['scope']);
         }
 
         return parent::getAccessToken($grant, $options);
@@ -125,9 +125,9 @@ protected function getDefaultScopes()
      */
     protected function checkResponse(ResponseInterface $response, $data)
     {
-        if (is_array($data) && isset($data['error'])) {
+        if (isset($data['errors'])) {
             throw new IdentityProviderException(
-                $data['error'] ?? $response->getReasonPhrase(),
+                $data['errors'][0]['title'] ?? $response->getReasonPhrase(),
                 $response->getStatusCode(),
                 $response->getBody()
             );