metaclassing / PHP7-Laravel5-EnterpriseAuth

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::get(config('azure-oath.routes.login'   ), 'Metrogistics\AzureSocialite\WebAuthController@redirectToOauthProvider');
+    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');
     }
 });

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/');
     });
 
@@ -18,7 +18,7 @@ 
      * )
      **/
 
-    $api->any('/api/hello', function () {
+    $api->any('/api/hello', function() {
         return 'hello world';
     });
 

publish/routes/api.php

Spacing +1 added lines, -1 removed lines

@@ -25,7 +25,7 @@
  * )
  **/
 
-Route::middleware('api')->get('/hello', function (Request $request) {
+Route::middleware('api')->get('/hello', function(Request $request) {
     return 'hello world';
 });
 

src/UserFactory.php

Spacing +3 added lines, -3 removed lines

@@ -22,13 +22,13 @@ 
         $new_user->$id_field = $azure_user->id;
         //$new_user->password = bcrypt('');
 
-        foreach($user_map as $azure_field => $user_field){
+        foreach ($user_map as $azure_field => $user_field) {
             $new_user->$user_field = $azure_user->$azure_field;
         }
 
         $callback = static::$user_callback;
 
-        if($callback && is_callable($callback)){
+        if ($callback && is_callable($callback)) {
             $callback($new_user);
         }
 
@@ -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/Middleware/Authenticate.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      */
     public function user()
     {
-        if($this->user){
+        if ($this->user) {
             return $this->user;
         }
 

src/OauthTokenGuard.php

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function check()
     {
-        return ! is_null($this->user());
+        return !is_null($this->user());
     }
 
     /**
@@ -66,7 +66,7 @@ 
      */
     public function guest()
     {
-        return ! $this->check();
+        return !$this->check();
     }
 
     /**
@@ -76,7 +76,7 @@ 
      */
     public function user()
     {
-        if (! is_null($this->user)) {
+        if (!is_null($this->user)) {
             return $this->user;
         }
     }
@@ -111,7 +111,7 @@ 
      *
      * @return bool
      */
-    public function validate(Array $credentials=[])
+    public function validate(Array $credentials = [])
     {
         return is_null($this->user);
     }

src/AzureOauthProvider.php

Spacing +1 added lines, -1 removed lines

@@ -60,7 +60,7 @@
             $azureadgroups = $this->getUserGroupsByToken($token);
             // only proceed if we got a good response with group info
             if (isset($azureadgroups['value']) && count($azureadgroups['value'])) {
-                foreach($azureadgroups['value'] as $group) {
+                foreach ($azureadgroups['value'] as $group) {
                     $groups[] = $group['displayName'];
                 }
             }

src/ServiceProvider.php

Spacing +6 added lines, -6 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,7 +66,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) && 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) {