Final

Author: RhysLees

README.md

@@ -53,10 +53,9 @@ This is the contents of the published config file:
 
 return [
     'auth' => [
-        'client_id' => env('MICROSOFT_PLANNER_CLIENT_ID'),
-        'client_secret' => env('MICROSOFT_PLANNER_CLIENT_SECRET'),
-        'tenant_id' => env('MICROSOFT_PLANNER_TENANT_ID'),
-        'redirect_uri' => env('MICROSOFT_PLANNER_REDIRECT_URI'),
+        'client_id' => env('LARAVEL_MICROSOFT_PLANNER_CLIENT_ID'),
+        'client_secret' => env('LARAVEL_MICROSOFT_PLANNER_CLIENT_SECRET'),
+        'tenant_id' => env('LARAVEL_MICROSOFT_PLANNER_TENANT_ID'),
     ]
 ];
 ```

composer.json

@@ -76,10 +76,7 @@
         "laravel": {
             "providers": [
                 "CodebarAg\\LaravelMicrosoftPlanner\\LaravelMicrosoftPlannerServiceProvider"
-            ],
-            "aliases": {
-                "LaravelMicrosoftPlanner": "CodebarAg\\LaravelMicrosoftPlanner\\Facades\\LaravelMicrosoftPlanner"
-            }
+            ]
         }
     },
     "minimum-stability": "dev",

config/laravel-microsoft-planner.php

@@ -0,0 +1,11 @@
+<?php
+
+// config for CodebarAg/LaravelMicrosoftPlanner
+
+return [
+    'auth' => [
+        'client_id' => env('LARAVEL_MICROSOFT_PLANNER_CLIENT_ID'),
+        'client_secret' => env('LARAVEL_MICROSOFT_PLANNER_CLIENT_SECRET'),
+        'tenant_id' => env('LARAVEL_MICROSOFT_PLANNER_TENANT_ID'),
+    ],
+];

config/microsoft-planner.php

@@ -10,4 +10,7 @@ parameters:
     checkOctaneCompatibility: true
     checkModelProperties: true
     checkMissingIterableValueType: false
+    ignoreErrors:
+        # new static() is a best practice in Drupal, so we cannot fix that.
+        - "#^Unsafe usage of new static#"
 

phpstan.neon.dist

@@ -17,12 +17,29 @@ public function resolveBaseUrl(): string
 
     protected function defaultOauthConfig(): OAuthConfig
     {
+        $this->setAuth();
+
         return OAuthConfig::make()
-            ->setClientId(config('microsoft-planner.auth.client_id'))
-            ->setClientSecret(config('microsoft-planner.auth.client_secret'))
-            ->setTokenEndpoint('https://login.windows.net/'.config('microsoft-planner.auth.tenant_id').'/oauth2/v2.0/token')
+            ->setClientId(config('laravel-microsoft-planner.auth.client_id'))
+            ->setClientSecret(config('laravel-microsoft-planner.auth.client_secret'))
+            ->setTokenEndpoint('https://login.windows.net/'.config('laravel-microsoft-planner.auth.tenant_id').'/oauth2/v2.0/token')
             ->setDefaultScopes([
                 'https://graph.microsoft.com/.default',
             ]);
     }
+
+    public function setAuth(): void
+    {
+        if (! config('laravel-microsoft-planner.auth.client_id')) {
+            throw new \Exception('No client_id provided.', 500);
+        }
+
+        if (! config('laravel-microsoft-planner.auth.client_secret')) {
+            throw new \Exception('No client_secret provided.', 500);
+        }
+
+        if (! config('laravel-microsoft-planner.auth.tenant_id')) {
+            throw new \Exception('No tenant_id provided.', 500);
+        }
+    }
 }

src/Facades/LaravelMicrosoftPlanner.php

@@ -16,6 +16,6 @@ public function configurePackage(Package $package): void
          */
         $package
             ->name('laravel-microsoft-planner')
-            ->hasConfigFile();
+            ->hasConfigFile('laravel-microsoft-planner');
     }
 }

src/Http/Connectors/MicrosoftPlannerConnector.php

@@ -27,19 +27,3 @@
 expect()->extend('toBeOne', function () {
     return $this->toBe(1);
 });
-
-/*
-|--------------------------------------------------------------------------
-| Functions
-|--------------------------------------------------------------------------
-|
-| While Pest is very powerful out-of-the-box, you may have some testing code specific to your
-| project that you don't want to repeat in every file. Here you can also expose helpers as
-| global functions to help you to reduce the number of lines of code in your test files.
-|
-*/
-
-function something()
-{
-    // ..
-}