OpenConext / Stepup-SelfService

src/Surfnet/StepupSelfService/SelfServiceBundle/Form/Type/VerifyEmailType.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
         ]);
         $builder->add('verifyEmail', SubmitType::class, [
             'label' => 'ss.form.ss_verify_email.button.verify_email',
-            'attr' => [ 'class' => 'btn btn-primary' ],
+            'attr' => ['class' => 'btn btn-primary'],
         ]);
     }
 

src/Surfnet/StepupSelfService/SelfServiceBundle/Form/Type/RevokeSecondFactorType.php

Spacing +2 added lines, -2 removed lines

@@ -29,11 +29,11 @@
     {
         $builder->add('revoke', SubmitType::class, [
             'label' => 'ss.form.ss_revoke_second_factor.revoke',
-            'attr' => [ 'class' => 'btn btn-danger pull-right' ],
+            'attr' => ['class' => 'btn btn-danger pull-right'],
         ]);
         $builder->add('cancel', AnchorType::class, [
             'label' => 'ss.form.ss_revoke_second_factor.cancel',
-            'attr' => [ 'class' => 'btn pull-right' ],
+            'attr' => ['class' => 'btn pull-right'],
             'route' => 'ss_second_factor_list',
         ]);
     }

src/Surfnet/StepupSelfService/SelfServiceBundle/Controller/RecoveryTokenController.php

Indentation -1 removed lines

@@ -47,7 +47,6 @@
 
 /**
  * @SuppressWarnings(PHPMD.CouplingBetweenObjects)
-
  */
 class RecoveryTokenController extends Controller
 {

src/Surfnet/StepupSelfService/SelfServiceBundle/Security/Authentication/Session/SessionStorage.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
 
     public function hasRequestId()
     {
-        return $this->session->has(self::SAML_SESSION_KEY. 'request_id');
+        return $this->session->has(self::SAML_SESSION_KEY . 'request_id');
     }
 
     public function clearRequestId()

src/Surfnet/StepupSelfService/SamlStepupProviderBundle/DependencyInjection/Configuration.php

Spacing +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
                     ->scalarNode('consume_assertion')
                         ->isRequired()
                         ->validate()
-                            ->ifTrue(function ($v) {
+                            ->ifTrue(function($v) {
                                 return !is_string($v) || strlen($v) === 0;
                             })
                             ->thenInvalid('Consume assertion route must be a non-empty string')
@@ -56,7 +56,7 @@ 
                     ->scalarNode('metadata')
                         ->isRequired()
                         ->validate()
-                            ->ifTrue(function ($v) {
+                            ->ifTrue(function($v) {
                                 return !is_string($v) || strlen($v) === 0;
                             })
                             ->thenInvalid('Metadata route must be a non-empty string')

src/Surfnet/StepupSelfService/SamlStepupProviderBundle/DependencyInjection/SurfnetStepupSelfServiceSamlStepupProviderExtension.php

Spacing +5 added lines, -5 removed lines

@@ -40,7 +40,7 @@ 
     {
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
-        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
+        $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__ . '/../Resources/config'));
         $loader->load('services.yml');
 
         foreach ($config['providers'] as $provider => $providerConfiguration) {
@@ -145,7 +145,7 @@ 
         $hostedDefinition = $this->buildHostedEntityDefinition($provider, $configuration, $routes);
         $container->setDefinition('gssp.provider.' . $provider . '.hosted_entities', $hostedDefinition);
 
-        $hostedSpDefinition  = (new Definition())
+        $hostedSpDefinition = (new Definition())
             ->setClass('Surfnet\SamlBundle\Entity\ServiceProvider')
             ->setFactory([
                 new Reference('gssp.provider.' . $provider . '.hosted_entities'),
@@ -194,7 +194,7 @@ 
      */
     private function createRemoteDefinition($provider, array $configuration, ContainerBuilder $container)
     {
-        $definition    = new Definition('Surfnet\SamlBundle\Entity\IdentityProvider', [
+        $definition = new Definition('Surfnet\SamlBundle\Entity\IdentityProvider', [
             [
                 'entityId'        => $configuration['entity_id'],
                 'ssoUrl'          => $configuration['sso_url'],
@@ -255,14 +255,14 @@ 
 
     private function validateDescriptions($descriptions, $appUrl, $provider, $type)
     {
-        $regex ="/%%{$type}_link_start%%[a-zA-Z0-9 ]+%%{$type}_link_end%%/";
+        $regex = "/%%{$type}_link_start%%[a-zA-Z0-9 ]+%%{$type}_link_end%%/";
         foreach ($descriptions as $lang => $description) {
             if ($appUrl !== false && preg_match($regex, $description) === 0) {
                 throw new InvalidConfigurationException(
                     sprintf(
                         'You have configured a GSSP provider with app URL\'s but the description is not ' .
                         'configured correctly yet. Missing "%%%1$s_link_start%%" or "%%%1$s_link_end%%" in ' .
-                        'GSSP description for language "%2$s" in "providers.%3$s.view_config.description" of '.
+                        'GSSP description for language "%2$s" in "providers.%3$s.view_config.description" of ' .
                         'samlstepupproviders.yml',
                         $type,
                         $lang,

src/Kernel.php

Spacing +11 added lines, -11 removed lines

@@ -32,7 +32,7 @@ 
 
     public function registerBundles(): iterable
     {
-        $contents = require $this->getProjectDir().'/config/bundles.php';
+        $contents = require $this->getProjectDir() . '/config/bundles.php';
         foreach ($contents as $class => $envs) {
             if ($envs[$this->environment] ?? $envs['all'] ?? false) {
                 yield new $class();
@@ -47,23 +47,23 @@ 
 
     protected function configureContainer(ContainerBuilder $container, LoaderInterface $loader): void
     {
-        $container->addResource(new FileResource($this->getProjectDir().'/config/bundles.php'));
+        $container->addResource(new FileResource($this->getProjectDir() . '/config/bundles.php'));
         $container->setParameter('container.dumper.inline_class_loader', \PHP_VERSION_ID < 70400 || $this->debug);
         $container->setParameter('container.dumper.inline_factories', true);
-        $confDir = $this->getProjectDir().'/config';
+        $confDir = $this->getProjectDir() . '/config';
 
-        $loader->load($confDir.'/{packages}/*'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{packages}/'.$this->environment.'/*'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{services}'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/{services}_'.$this->environment.self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{packages}/*' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{packages}/' . $this->environment . '/*' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{services}' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/{services}_' . $this->environment . self::CONFIG_EXTS, 'glob');
     }
 
     protected function configureRoutes(RouteCollectionBuilder $routes): void
     {
-        $confDir = $this->getProjectDir().'/config';
+        $confDir = $this->getProjectDir() . '/config';
 
-        $routes->import($confDir.'/{routes}/'.$this->environment.'/*'.self::CONFIG_EXTS, '/', 'glob');
-        $routes->import($confDir.'/{routes}/*'.self::CONFIG_EXTS, '/', 'glob');
-        $routes->import($confDir.'/{routes}'.self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}/' . $this->environment . '/*' . self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}/*' . self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/{routes}' . self::CONFIG_EXTS, '/', 'glob');
     }
 }