metaclassing / PHP7-Laravel5-EnterpriseAuth

publish/database/migrations/2018_02_19_152839_alter_users_table_for_azure_ad.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
      */
     public function up()
     {
-        Schema::table('users', function (Blueprint $table) {
+        Schema::table('users', function(Blueprint $table) {
             // Users must be able to support blank passwords for external identity
             $table->string('password')->nullable()->change();
             // We need a new string field to store the oauth provider unique id in

publish/routes/swagger.php

Spacing +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 
     // Redirect requests to /api to the swagger documentation
     //$api->any('', function (Illuminate\Http\Request $request) {
-    $api->any('', function () {
+    $api->any('', function() {
         return redirect('api/documentation/');
     });
 
@@ -17,6 +17,6 @@ 
      *     @SWG\Response(response="200", description="Hello world example")
      * )
      **/
-    $api->any('/api/hello', function () {
+    $api->any('/api/hello', function() {
         return 'hello world';
     });

publish/routes/api.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
  *     @SWG\Response(response="200", description="Hello world example")
  * )
  **/
-Route::middleware('api')->get('/hello', function (Request $request) {
+Route::middleware('api')->get('/hello', function(Request $request) {
     return 'hello world';
 });
 

src/AzureActiveDirectory.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
         if ($tenantName != 'common') {
             // Make sure the tenant is formatted like xyzcorp.onmicrosoft.com
             $regex = '/\.onmicrosoft\.com/';
-            if (! preg_match($regex, $tenantName, $hits)) {
+            if (!preg_match($regex, $tenantName, $hits)) {
                 // Append the suffix if it is missing
                 $tenantName .= '.onmicrosoft.com';
             }

src/Middleware/OauthTokenGuard.php

Spacing +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
      */
     public function check()
     {
-        return ! is_null($this->user());
+        return !is_null($this->user());
     }
 
     /**
@@ -67,7 +67,7 @@ 
      */
     public function guest()
     {
-        return ! $this->check();
+        return !$this->check();
     }
 
     /**
@@ -77,7 +77,7 @@ 
      */
     public function user()
     {
-        if (! is_null($this->user)) {
+        if (!is_null($this->user)) {
             return $this->user;
         }
     }

routes/api.microsoft.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
 // Authenticated user information routes
-Route::middleware([config('enterpriseauth.apiroutes.middleware'), config('enterpriseauth.apiroutes.authmiddleware')])->group(function () {
+Route::middleware([config('enterpriseauth.apiroutes.middleware'), config('enterpriseauth.apiroutes.authmiddleware')])->group(function() {
 
     /**
      * @SWG\Get(

src/Controllers/WebAuthController.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
         $destination = $request->session()
                                ->get('oauthIntendedUrl');
         // If there is no intended destination url, use the default
-        if (! $destination) {
+        if (!$destination) {
             $destination = config('enterpriseauth.redirect_on_login');
         }
         \Illuminate\Support\Facades\Log::info('AUTH success USER ID '.$user->id.' with redirect url '.$destination);

src/ServiceProvider.php

Spacing +6 added lines, -6 removed lines

@@ -47,7 +47,7 @@ 
             // Go through all the credential config and make sure they are set in the .env or config file
             foreach (config('enterpriseauth.credentials') as $config => $env) {
                 // If one isnt set, throw a red flat until the person fixes it
-                if (! config('enterpriseauth.credentials.'.$config)) {
+                if (!config('enterpriseauth.credentials.'.$config)) {
                     throw new \Exception('enterpriseauth setup error: missing mandatory config value for enterpriseauth.credentials.'.$config.' check your .env file!');
                 }
             }
@@ -59,7 +59,7 @@ 
         // Override the application configuration to use our oauth token guard driver at runtime
         config(['auth.guards.api.driver' => 'oauthtoken']);
         // Now I have a machine gun. ho ho ho!
-        \Illuminate\Support\Facades\Auth::extend('oauthtoken', function ($app, $name, array $config) {
+        \Illuminate\Support\Facades\Auth::extend('oauthtoken', function($app, $name, array $config) {
             $userProvider = \Illuminate\Support\Facades\Auth::createUserProvider($config['provider']);
 
             return new \Metaclassing\EnterpriseAuth\Middleware\OauthTokenGuard($userProvider, $app->make('request'));
@@ -69,13 +69,13 @@ 
     protected function configureSwaggerToScanEnterpriseAuthRouteFiles()
     {
         $swaggerScanPaths = config('l5-swagger.paths.annotations');
-        if (! is_array($swaggerScanPaths)) {
+        if (!is_array($swaggerScanPaths)) {
             $swaggerScanPaths = [$swaggerScanPaths];
         }
-        if (! in_array(base_path('routes'), $swaggerScanPaths)) {
+        if (!in_array(base_path('routes'), $swaggerScanPaths)) {
             $swaggerScanPaths[] = base_path('routes');
         }
-        if (! in_array(__DIR__.'/../routes/', $swaggerScanPaths)) {
+        if (!in_array(__DIR__.'/../routes/', $swaggerScanPaths)) {
             $swaggerScanPaths[] = __DIR__.'/../routes/';
         }
         config(['l5-swagger.paths.annotations' => $swaggerScanPaths]);
@@ -85,7 +85,7 @@ 
     {
         // If the routes files for the swagger oauth config is NOT present, and we have all the right info, then generate it really quick
         $swaggerAzureadFile = __DIR__.'/../routes/swagger.azuread.php';
-        if (! file_exists($swaggerAzureadFile)) {
+        if (!file_exists($swaggerAzureadFile)) {
             $aad = new AzureActiveDirectory(config('enterpriseauth.credentials.tenant'));
             //$authorizationUrl = $aad->authorizationEndpoint . '?resource=https://graph.microsoft.com';
             $authorizationUrl = $aad->authorizationEndpoint;

routes/web.microsoft.php

Spacing +4 added lines, -4 removed lines

@@ -1,19 +1,19 @@
 <?php
 
-Route::middleware([config('enterpriseauth.routes.middleware')])->group(function () {
+Route::middleware([config('enterpriseauth.routes.middleware')])->group(function() {
     Route::get(config('enterpriseauth.routes.login'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@redirectToOauthProvider');
     Route::get(config('enterpriseauth.routes.logout'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@logoutFromOauthProvider');
     Route::get(config('enterpriseauth.routes.callback'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@handleOauthResponse');
     Route::get(config('enterpriseauth.routes.adminconsent'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@redirectToOauthAdminConsent');
 
     // This handles a situation where a route with the NAME of login does not exist, we define it to keep from breaking framework redirects hard coded
-    if (! \Route::has('login')) {
+    if (!\Route::has('login')) {
         Route::get('login', 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@loginOrRegister')->name('login');
     }
-    if (! \Route::has('register')) {
+    if (!\Route::has('register')) {
         Route::get('register', 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@loginOrRegister')->name('register');
     }
-    if (! \Route::has('logout')) {
+    if (!\Route::has('logout')) {
         Route::get('logout', 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@logout')->name('logout');
     }
 });