metaclassing / PHP7-Laravel5-EnterpriseAuth

src/Middleware/Authenticate.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
      */
     public function guest()
     {
-        return ! $this->check();
+        return !$this->check();
     }
 
     /**

src/AuthController.php

Spacing +4 added lines, -4 removed lines

@@ -31,7 +31,7 @@ 
     public function certAuth()
     {
         // Make sure we got a client certificate from the web server
-        if (! $_SERVER['SSL_CLIENT_CERT']) {
+        if (!$_SERVER['SSL_CLIENT_CERT']) {
             throw new \Exception('TLS client certificate missing');
         }
         // try to parse the certificate we got
@@ -40,7 +40,7 @@ 
         $asciicert = str_replace("\t", '', $_SERVER['SSL_CLIENT_CERT']);
         $cert = $x509->loadX509($asciicert);
         $names = $x509->getExtension('id-ce-subjectAltName');
-        if (! $names) {
+        if (!$names) {
             throw new \Exception('TLS client cert missing subject alternative names');
         }
         // Search subject alt names for user principal name
@@ -54,12 +54,12 @@ 
                 }
             }
         }
-        if (! $upn) {
+        if (!$upn) {
             throw new \Exception('Could not find user principal name in TLS client cert');
         }
         $user_class = config('azure-oath.user_class');
         $user = $user_class::where('userPrincipalName', $upn)->first();
-        if (! $user) {
+        if (!$user) {
             throw new \Exception('No user found with user principal name '.$upn);
         }
         //dd($user);

src/AzureOauthProvider.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
     {
         //dd($user);
         // WELL this is extremely stupid. if email address isnt set, use their UPN...
-        if (! $user['mail']) {
+        if (!$user['mail']) {
             $user['mail'] = $user['userPrincipalName'];
         }
 

src/WebAuthController.php

Indentation +3 added lines, -3 removed lines

@@ -23,9 +23,9 @@
         if (count($user->groups)) {
             // remove the users existing database roles before assigning new ones
             \DB::table('assigned_roles')
-               ->where('entity_id', $authUser->id)
-               ->where('entity_type', get_class($authUser))
-               ->delete();
+                ->where('entity_id', $authUser->id)
+                ->where('entity_type', get_class($authUser))
+                ->delete();
             // add the user to each group they are assigned
             $authUser->assign($user->groups);
         }

src/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;
         }
     }

src/UserFactory.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
 
     public static function userCallback($callback)
     {
-        if (! is_callable($callback)) {
+        if (!is_callable($callback)) {
             throw new \Exception('Must provide a callable.');
         }
 

src/ServiceProvider.php

Spacing +7 added lines, -7 removed lines

@@ -26,19 +26,19 @@ 
 
         // Actually I have my own oauth token cache based authentication guard now lol
         config(['auth.guards.api.driver' => 'oauthtoken']);
-        Auth::extend('oauthtoken', function ($app, $name, array $config) {
+        Auth::extend('oauthtoken', function($app, $name, array $config) {
             return new OauthTokenGuard(Auth::createUserProvider($config['provider']), $app->make('request'));
         });
 
         // Make sure that this vendor dir and the routes dir are in any scanned paths for swagger documentation
         $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]);
@@ -54,7 +54,7 @@ 
             __DIR__.'/../publish/config/azure-oath.php', 'azure-oath'
         );
 
-        $this->app['Laravel\Socialite\Contracts\Factory']->extend('azure-oauth', function ($app) {
+        $this->app['Laravel\Socialite\Contracts\Factory']->extend('azure-oauth', function($app) {
             return $app['Laravel\Socialite\Contracts\Factory']->buildProvider(
                 'Metrogistics\AzureSocialite\AzureOauthProvider',
                 config('azure-oath.credentials')
@@ -66,10 +66,10 @@ 
 
         // 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) && env('AZURE_AD_CLIENT_ID') && env('AZURE_AD_OPENID_URL')) {
+        if (!file_exists($swaggerAzureadFile) && env('AZURE_AD_CLIENT_ID') && env('AZURE_AD_OPENID_URL')) {
             $openidConfig = $this->getOpenidConfiguration(env('AZURE_AD_OPENID_URL'));
             $authorizationUrl = $openidConfig['authorization_endpoint'];
-            if (! $authorizationUrl) {
+            if (!$authorizationUrl) {
                 throw new \Exception('Error building swagger oauth config, azure ad openid url didnt give me an authorization url!');
             }
             $client_id = env('AZURE_AD_CLIENT_ID');

src/ApiAuthController.php

Spacing +4 added lines, -4 removed lines

@@ -63,7 +63,7 @@ 
         $oauthAccessToken = $this->extractOauthAccessTokenFromRequest($request);
 
         // If we cant find ANY token to use, abort.
-        if (! $oauthAccessToken) {
+        if (!$oauthAccessToken) {
             throw new \Exception('error: token/access_token/authorization bearer token is missing');
         }
 
@@ -72,7 +72,7 @@ 
 
         try {
             // verify the credentials and create a token for the user
-            if (! $token = \JWTAuth::fromUser($authUser)) {
+            if (!$token = \JWTAuth::fromUser($authUser)) {
                 return response()->json(['error' => 'invalid_credentials'], 401);
             }
         } catch (JWTException $e) {
@@ -140,7 +140,7 @@ 
 
     protected function mapUserToObject(array $user)
     {
-        if (! $user['mail']) {
+        if (!$user['mail']) {
             $user['mail'] = $user['userPrincipalName'];
         }
 
@@ -184,7 +184,7 @@ 
         $roles = $user->roles()->get()->all();
         foreach ($roles as $key => $role) {
             $role->permissions = $role->abilities()->get()->all();
-            if (! count($role->permissions)) {
+            if (!count($role->permissions)) {
                 unset($roles[$key]);
             }
         }

Indentation +3 added lines, -3 removed lines

@@ -23,9 +23,9 @@
         if (count($user->groups)) {
             // remove the users existing database roles before assigning new ones
             \DB::table('assigned_roles')
-               ->where('entity_id', $authUser->id)
-               ->where('entity_type', get_class($authUser))
-               ->delete();
+                ->where('entity_id', $authUser->id)
+                ->where('entity_type', get_class($authUser))
+                ->delete();
             // add the user to each group they are assigned
             $authUser->assign($user->groups);
         }

routes/web.microsoft.php

Spacing +3 added lines, -3 removed lines

@@ -1,14 +1,14 @@
 <?php
 
-Route::middleware([config('azure-oath.routes.middleware')])->group(function () {
+Route::middleware([config('azure-oath.routes.middleware')])->group(function() {
     Route::get(config('azure-oath.routes.login'), 'Metrogistics\AzureSocialite\WebAuthController@redirectToOauthProvider');
     Route::get(config('azure-oath.routes.callback'), 'Metrogistics\AzureSocialite\WebAuthController@handleOauthResponse');
 
     // 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', 'Metrogistics\AzureSocialite\AuthController@loginOrRegister')->name('login');
     }
-    if (! \Route::has('register')) {
+    if (!\Route::has('register')) {
         Route::get('register', 'Metrogistics\AzureSocialite\AuthController@loginOrRegister')->name('register');
     }
 });