@@ -56,7 +56,7 @@ |
||
| 56 | 56 | $roles = $user->roles()->get()->all(); |
| 57 | 57 | foreach ($roles as $key => $role) { |
| 58 | 58 | $role->permissions = $role->abilities()->get()->all(); |
| 59 | - if (! count($role->permissions)) { |
|
| 59 | + if (!count($role->permissions)) { |
|
| 60 | 60 | unset($roles[$key]); |
| 61 | 61 | } |
| 62 | 62 | } |
@@ -15,27 +15,27 @@ discard block |
||
| 15 | 15 | { |
| 16 | 16 | // Make sure nobody is including or running this thing without all the required env settings |
| 17 | 17 | $requiredVariables = ['AZURE_AD_CLIENT_ID', 'AZURE_AD_CLIENT_SECRET', 'AZURE_AD_TENANT', 'AZURE_AD_CALLBACK_URL']; |
| 18 | - foreach($requiredVariables as $env) { |
|
| 19 | - if (! env($env)) { |
|
| 18 | + foreach ($requiredVariables as $env) { |
|
| 19 | + if (!env($env)) { |
|
| 20 | 20 | throw new \Exception('enterpriseauth setup error: missing mandatory .env value for '.$env); |
| 21 | 21 | } |
| 22 | 22 | } |
| 23 | 23 | |
| 24 | 24 | // Actually I have my own oauth token cache based authentication guard now lol |
| 25 | 25 | config(['auth.guards.api.driver' => 'oauthtoken']); |
| 26 | - Auth::extend('oauthtoken', function ($app, $name, array $config) { |
|
| 26 | + Auth::extend('oauthtoken', function($app, $name, array $config) { |
|
| 27 | 27 | return new OauthTokenGuard(Auth::createUserProvider($config['provider']), $app->make('request')); |
| 28 | 28 | }); |
| 29 | 29 | |
| 30 | 30 | // Make sure that this vendor dir and the routes dir are in any scanned paths for swagger documentation |
| 31 | 31 | $swaggerScanPaths = config('l5-swagger.paths.annotations'); |
| 32 | - if (! is_array($swaggerScanPaths)) { |
|
| 32 | + if (!is_array($swaggerScanPaths)) { |
|
| 33 | 33 | $swaggerScanPaths = [$swaggerScanPaths]; |
| 34 | 34 | } |
| 35 | - if (! in_array(base_path('routes'), $swaggerScanPaths)) { |
|
| 35 | + if (!in_array(base_path('routes'), $swaggerScanPaths)) { |
|
| 36 | 36 | $swaggerScanPaths[] = base_path('routes'); |
| 37 | 37 | } |
| 38 | - if (! in_array(__DIR__.'/../routes/', $swaggerScanPaths)) { |
|
| 38 | + if (!in_array(__DIR__.'/../routes/', $swaggerScanPaths)) { |
|
| 39 | 39 | $swaggerScanPaths[] = __DIR__.'/../routes/'; |
| 40 | 40 | } |
| 41 | 41 | config(['l5-swagger.paths.annotations' => $swaggerScanPaths]); |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | { |
| 66 | 66 | // If the routes files for the swagger oauth config is NOT present, and we have all the right info, then generate it really quick |
| 67 | 67 | $swaggerAzureadFile = __DIR__.'/../routes/swagger.azuread.php'; |
| 68 | - if (! file_exists($swaggerAzureadFile)) { |
|
| 68 | + if (!file_exists($swaggerAzureadFile)) { |
|
| 69 | 69 | $aad = new AzureActiveDirectory(env('AZURE_AD_TENANT')); |
| 70 | 70 | //$authorizationUrl = $aad->authorizationEndpoint . '?resource=https://graph.microsoft.com'; |
| 71 | 71 | $authorizationUrl = $aad->authorizationEndpoint; |
@@ -1,14 +1,14 @@ |
||
| 1 | 1 | <?php |
| 2 | 2 | |
| 3 | -Route::middleware([config('enterpriseauth.routes.middleware')])->group(function () { |
|
| 3 | +Route::middleware([config('enterpriseauth.routes.middleware')])->group(function() { |
|
| 4 | 4 | Route::get(config('enterpriseauth.routes.login'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@redirectToOauthProvider'); |
| 5 | 5 | Route::get(config('enterpriseauth.routes.callback'), 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@handleOauthResponse'); |
| 6 | 6 | |
| 7 | 7 | // 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 |
| 8 | - if (! \Route::has('login')) { |
|
| 8 | + if (!\Route::has('login')) { |
|
| 9 | 9 | Route::get('login', 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@loginOrRegister')->name('login'); |
| 10 | 10 | } |
| 11 | - if (! \Route::has('register')) { |
|
| 11 | + if (!\Route::has('register')) { |
|
| 12 | 12 | Route::get('register', 'Metaclassing\EnterpriseAuth\Controllers\WebAuthController@loginOrRegister')->name('register'); |
| 13 | 13 | } |
| 14 | 14 | }); |
@@ -1,7 +1,7 @@ |
||
| 1 | 1 | <?php |
| 2 | 2 | |
| 3 | 3 | // Authenticated user information routes |
| 4 | -Route::middleware([config('enterpriseauth.apiroutes.middleware'), config('enterpriseauth.apiroutes.authmiddleware')])->group(function () { |
|
| 4 | +Route::middleware([config('enterpriseauth.apiroutes.middleware'), config('enterpriseauth.apiroutes.authmiddleware')])->group(function() { |
|
| 5 | 5 | |
| 6 | 6 | /** |
| 7 | 7 | * @SWG\Get( |
@@ -46,7 +46,7 @@ |
||
| 46 | 46 | */ |
| 47 | 47 | public function guest() |
| 48 | 48 | { |
| 49 | - return ! $this->check(); |
|
| 49 | + return !$this->check(); |
|
| 50 | 50 | } |
| 51 | 51 | |
| 52 | 52 | /** |
@@ -98,7 +98,7 @@ |
||
| 98 | 98 | { |
| 99 | 99 | //dd($user); |
| 100 | 100 | // WELL this is extremely stupid. if email address isnt set, use their UPN... |
| 101 | - if (! $user['mail']) { |
|
| 101 | + if (!$user['mail']) { |
|
| 102 | 102 | $user['mail'] = $user['userPrincipalName']; |
| 103 | 103 | } |
| 104 | 104 | |