rhertogh / yii2-oauth2-server

src/controllers/web/Oauth2ServerController.php

Indentation +2 added lines, -2 removed lines

@@ -33,8 +33,8 @@
     public function actions()
     {
         return [
-             static::ACTION_NAME_ACCESS_TOKEN => Oauth2AccessTokenAction::class,
-             static::ACTION_NAME_AUTHORIZE => Oauth2AuthorizeAction::class,
+                static::ACTION_NAME_ACCESS_TOKEN => Oauth2AccessTokenAction::class,
+                static::ACTION_NAME_AUTHORIZE => Oauth2AuthorizeAction::class,
         ];
     }
 }

src/controllers/web/Oauth2CertificatesController.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     public function actions()
     {
         return [
-             static::ACTION_NAME_JWKS => Oauth2JwksAction::class,
+                static::ACTION_NAME_JWKS => Oauth2JwksAction::class,
         ];
     }
 }

src/controllers/web/openidconnect/Oauth2OidcUserinfoAction.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             $response->data = $this->generateOpenIdConnectUserClaims($identity, $module, $nonce);
             return $response;
 
-        } elseif($userInfoAlg == 'RS256') {
+        } elseif ($userInfoAlg == 'RS256') {
 
             $response->format = Response::FORMAT_RAW;
             $response->headers->add('Content-Type', 'application/jwt');

src/controllers/web/Oauth2WellKnownController.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     public function actions()
     {
         return [
-             static::ACTION_NAME_OPENID_CONFIGURATION => Oauth2OpenidConfigurationAction::class,
+                static::ACTION_NAME_OPENID_CONFIGURATION => Oauth2OpenidConfigurationAction::class,
         ];
     }
 }

src/controllers/console/debug/Oauth2DebugConfigAction.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         $storageEncryptionKeys = $module->storageEncryptionKeys ? '[SET]' : '[NOT SET]';
 
         return [
-            'serverRole' => $module->serverRole . ' (' .implode(', ', $serverRoles) . ')',
+            'serverRole' => $module->serverRole . ' (' . implode(', ', $serverRoles) . ')',
 
             'privateKey' => $privateKey,
             'privateKeyPassphrase' => $privateKeyPassphrase,

src/controllers/console/client/Oauth2CreateClientAction.php

Spacing +2 added lines, -2 removed lines

@@ -49,11 +49,11 @@
 
         $this->controller->stdout("Creating new Oath2 Client\n");
 
-        if (empty($identifer)){
+        if (empty($identifer)) {
             $identifier = $this->controller->prompt('Client Identifier?', [
                 'required' => true,
                 'default' => $defaultIdentifier ?? null,
-                'validator' => function ($input, &$error) {
+                'validator' => function($input, &$error) {
                     /** @var string|Oauth2ClientInterface $clientClass */
                     $clientClass = DiHelper::getValidatedClassName(Oauth2ClientInterface::class);
                     if ($clientClass::findByIdentifier($input)) {

src/views/consent/authorize-client.php

Indentation -8 removed lines

@@ -163,11 +163,3 @@
 
 <?php
 $js = <<<'JS'
-    $('#oauth2_authorize-client-previously-approved-scopes-toggle').click(function(e) {
-        $('#oauth2_authorize-client-previously-approved-scopes-list').slideToggle();
-        $(this).toggleClass('open');
-        e.preventDefault();
-    });
-    JS;
-
-$this->registerJs($js);

Switch Indentation -8 removed lines

@@ -163,11 +163,3 @@
 
 <?php
 $js = <<<'JS'
-    $('#oauth2_authorize-client-previously-approved-scopes-toggle').click(function(e) {
-        $('#oauth2_authorize-client-previously-approved-scopes-list').slideToggle();
-        $(this).toggleClass('open');
-        e.preventDefault();
-    });
-    JS;
-
-$this->registerJs($js);

Spacing -8 removed lines

@@ -163,11 +163,3 @@
 
 <?php
 $js = <<<'JS'
-    $('#oauth2_authorize-client-previously-approved-scopes-toggle').click(function(e) {
-        $('#oauth2_authorize-client-previously-approved-scopes-list').slideToggle();
-        $(this).toggleClass('open');
-        e.preventDefault();
-    });
-    JS;
-
-$this->registerJs($js);

Braces +5 added lines, -10 removed lines

@@ -130,14 +130,17 @@ 
                     </div>
                 </div>
             <?php endif; ?>
-        <?php else: ?>
+        <?php else {
+    : ?>
             <h3 class="oauth2_authorize-client-modal-title oauth2_authorize-client-modal-title_no-scopes">
                 <?= Yii::t('oauth2', '{clientName} would like to access {appName} on your behalf.', [
                     'clientName' => Html::encode($clientAuthorizationRequest->getClient()->getName()),
                     'appName' => Yii::$app->name,
                 ]) ?>
             </h3>
-        <?php endif; ?>
+        <?php endif;
+}
+?>
         </div>
         <div class="oauth2_authorize-client-modal-footer">
             <button
@@ -163,11 +166,3 @@ 
 
 <?php
 $js = <<<'JS'
-    $('#oauth2_authorize-client-previously-approved-scopes-toggle').click(function(e) {
-        $('#oauth2_authorize-client-previously-approved-scopes-list').slideToggle();
-        $(this).toggleClass('open');
-        e.preventDefault();
-    });
-    JS;
-
-$this->registerJs($js);

Upper-Lower-Casing -8 removed lines

@@ -163,11 +163,3 @@
 
 <?php
 $js = <<<'JS'
-    $('#oauth2_authorize-client-previously-approved-scopes-toggle').click(function(e) {
-        $('#oauth2_authorize-client-previously-approved-scopes-list').slideToggle();
-        $(this).toggleClass('open');
-        e.preventDefault();
-    });
-    JS;
-
-$this->registerJs($js);

src/views/consent/_authorize-client-scope-list.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
         $scope = $scopeRequest->getScope();
         $inputId = Html::getInputId($clientAuthorizationRequest, 'selectedScopeIdentifiers')
                         . '-'
-                        . preg_replace('/[^a-z0-9_]/','_',  mb_strtolower($scope->getIdentifier()));
+                        . preg_replace('/[^a-z0-9_]/', '_', mb_strtolower($scope->getIdentifier()));
 
         $authorizationMessage = $scope->getAuthorizationMessage() ?? $scope->getDescription() ?? $scope->getIdentifier();
 

src/migrations/Oauth2_00001_CreateOauth2TablesMigration.php

Spacing +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
                     ->comment('Skip user authorization of client if there are no scopes that require authorization.'),
                 'client_credentials_grant_user_id' => ($userTable ? $userPkSchema->dbType : Schema::TYPE_STRING)
                     . ' COMMENT "Optional user id to use in case of grant type \'client_credentials\'.'
-                    . ' This user account should also be connected to the client via the `'.$userClientTable.'` table and, if applicable, the `'.$userClientScopeTable.'` table."',
+                    . ' This user account should also be connected to the client via the `' . $userClientTable . '` table and, if applicable, the `' . $userClientScopeTable . '` table."',
                 'oidc_allow_offline_access_without_consent' => $this->boolean()->notNull()->defaultValue(0)
                     ->comment('Allow the OpenID Connect "offline_access" scope for this client without the "prompt" parameter contains "consent".'),
                 'oidc_userinfo_encrypted_response_alg' => $this->string(),