Part-DB / Part-DB-symfony

tests/Entity/UserSystem/UserTest.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
     {
         $user = new User();
         $user->setGoogleAuthenticatorSecret($secret);
-        $this->assertSame($expected ,$user->isGoogleAuthenticatorEnabled());
+        $this->assertSame($expected, $user->isGoogleAuthenticatorEnabled());
     }
 
     /**

src/Controller/UserController.php

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     {
         //Handle 2FA disabling
 
-        if($request->request->has('reset_2fa')) {
+        if ($request->request->has('reset_2fa')) {
             //Check if the admin has the needed permissions
             $this->denyAccessUnlessGranted('set_password', $entity);
             if ($this->isCsrfTokenValid('reset_2fa'.$entity->getId(), $request->request->get('_token'))) {
@@ -77,7 +77,7 @@ 
                 $entity->setGoogleAuthenticatorSecret(null);
                 $entity->setBackupCodes([]);
                 //Remove all U2F keys
-                foreach($entity->getU2FKeys() as $key) {
+                foreach ($entity->getU2FKeys() as $key) {
                     $em->remove($key);
                 }
                 //Invalidate trusted devices

src/Controller/UserSettingsController.php

Spacing +9 added lines, -9 removed lines

@@ -85,7 +85,7 @@ 
      */
     public function removeU2FToken(Request $request, EntityManagerInterface $entityManager, BackupCodeManager $backupCodeManager)
     {
-        if($this->demo_mode) {
+        if ($this->demo_mode) {
             throw new \RuntimeException('You can not do 2FA things in demo mode');
         }
 
@@ -100,13 +100,13 @@ 
 
 
         if ($this->isCsrfTokenValid('delete'.$user->getId(), $request->request->get('_token'))) {
-            if($request->request->has('key_id')) {
+            if ($request->request->has('key_id')) {
                 $key_id = $request->request->get('key_id');
                 $key_repo = $entityManager->getRepository(U2FKey::class);
                 /** @var U2FKey|null $u2f */
                 $u2f = $key_repo->find($key_id);
-                if($u2f === null) {
-                    $this->addFlash('danger','tfa_u2f.u2f_delete.not_existing');
+                if ($u2f === null) {
+                    $this->addFlash('danger', 'tfa_u2f.u2f_delete.not_existing');
                     throw new \RuntimeException('Key not existing!');
                 }
 
@@ -122,7 +122,7 @@ 
                 $this->addFlash('success', 'tfa.u2f.u2f_delete.success');
             }
         } else {
-            $this->addFlash('error','csfr_invalid');
+            $this->addFlash('error', 'csfr_invalid');
         }
 
         return $this->redirectToRoute('user_settings');
@@ -133,7 +133,7 @@ 
      */
     public function resetTrustedDevices(Request $request, EntityManagerInterface $entityManager)
     {
-        if($this->demo_mode) {
+        if ($this->demo_mode) {
             throw new \RuntimeException('You can not do 2FA things in demo mode');
         }
 
@@ -152,7 +152,7 @@ 
             $entityManager->flush();
             $this->addFlash('success', 'tfa_trustedDevice.invalidate.success');
         } else {
-            $this->addFlash('error','csfr_invalid');
+            $this->addFlash('error', 'csfr_invalid');
         }
 
         return $this->redirectToRoute('user_settings');
@@ -253,7 +253,7 @@ 
         }
         $google_form->handleRequest($request);
 
-        if($google_form->isSubmitted() && $google_form->isValid() && !$this->demo_mode) {
+        if ($google_form->isSubmitted() && $google_form->isValid() && !$this->demo_mode) {
             if (!$google_enabled) {
                 //Save 2FA settings (save secrets)
                 $user->setGoogleAuthenticatorSecret($google_form->get('googleAuthenticatorSecret')->getData());
@@ -271,7 +271,7 @@ 
             }
         }
 
-        $backup_form = $this->get('form.factory')->createNamedBuilder('backup_codes')->add('reset_codes', SubmitType::class,[
+        $backup_form = $this->get('form.factory')->createNamedBuilder('backup_codes')->add('reset_codes', SubmitType::class, [
             'label' => 'tfa_backup.regenerate_codes',
             'attr' => ['class' => 'btn-danger'],
             'disabled' => empty($user->getBackupCodes())

src/Form/TFAGoogleSettingsType.php

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
             $user = $event->getData();
 
             //Only show setup fields, when google authenticator is not enabled
-            if(!$user->isGoogleAuthenticatorEnabled()) {
+            if (!$user->isGoogleAuthenticatorEnabled()) {
                 $form->add(
                     'google_confirmation',
                     TextType::class,

src/Validator/Constraints/ValidGoogleAuthCodeValidator.php

Spacing +4 added lines, -4 removed lines

@@ -58,23 +58,23 @@
             throw new UnexpectedValueException($value, 'string');
         }
 
-        if(!ctype_digit($value)) {
+        if (!ctype_digit($value)) {
             $this->context->addViolation('validator.google_code.only_digits_allowed');
         }
 
         //Number must have 6 digits
-        if(strlen($value) !== 6) {
+        if (strlen($value) !== 6) {
             $this->context->addViolation('validator.google_code.wrong_digit_count');
         }
 
         //Try to retrieve the user we want to check
-        if($this->context->getObject() instanceof FormInterface &&
+        if ($this->context->getObject() instanceof FormInterface &&
             $this->context->getObject()->getParent() instanceof FormInterface
         && $this->context->getObject()->getParent()->getData() instanceof User) {
             $user = $this->context->getObject()->getParent()->getData();
 
             //Check if the given code is valid
-            if(!$this->googleAuthenticator->checkCode($user, $value)) {
+            if (!$this->googleAuthenticator->checkCode($user, $value)) {
                 $this->context->addViolation('validator.google_code.wrong_code');
             }
 

src/EventSubscriber/PasswordChangeNeededSubscriber.php

Spacing +5 added lines, -5 removed lines

@@ -76,10 +76,10 @@ 
         $user = $this->security->getUser();
         $request = $event->getRequest();
 
-        if(!$event->isMasterRequest()) {
+        if (!$event->isMasterRequest()) {
             return;
         }
-        if(!$user instanceof User) {
+        if (!$user instanceof User) {
             return;
         }
 
@@ -98,16 +98,16 @@ 
 
         /* Dont redirect tree endpoints, as this would cause trouble and creates multiple flash
         warnigs for one page reload */
-        if(strpos($request->getUri(), '/tree/') !== false) {
+        if (strpos($request->getUri(), '/tree/') !== false) {
             return;
         }
 
         //Show appropriate message to user about the reason he was redirected
-        if($user->isNeedPwChange()) {
+        if ($user->isNeedPwChange()) {
             $this->flashBag->add('warning', 'user.pw_change_needed.flash');
         }
 
-        if(static::TFARedirectNeeded($user)) {
+        if (static::TFARedirectNeeded($user)) {
             $this->flashBag->add('warning', 'user.2fa_needed.flash');
         }
 

src/Services/TFA/BackupCodeManager.php

Spacing +2 added lines, -2 removed lines

@@ -44,7 +44,7 @@ 
      */
     public function enableBackupCodes(User $user)
     {
-        if(empty($user->getBackupCodes())) {
+        if (empty($user->getBackupCodes())) {
            $this->regenerateBackupCodes($user);
         }
     }
@@ -55,7 +55,7 @@ 
      */
     public function disableBackupCodesIfUnused(User $user)
     {
-        if($user->isGoogleAuthenticatorEnabled()) {
+        if ($user->isGoogleAuthenticatorEnabled()) {
             return;
         }
 

src/Services/TFA/BackupCodeGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
     public function generateCodeSet() : array
     {
         $array = [];
-        for($n=0; $n<$this->code_count; $n++) {
+        for ($n = 0; $n < $this->code_count; $n++) {
             $array[] = $this->generateSingleCode();
         }
 

src/Entity/UserSystem/User.php

Indentation +2 added lines, -2 removed lines

@@ -205,8 +205,8 @@
     protected $trustedDeviceCookieVersion = 0;
 
     /** @var Collection<TwoFactorKeyInterface>
-      * @ORM\OneToMany(targetEntity="App\Entity\UserSystem\U2FKey", mappedBy="user", cascade={"REMOVE"}, orphanRemoval=true)
-      */
+     * @ORM\OneToMany(targetEntity="App\Entity\UserSystem\U2FKey", mappedBy="user", cascade={"REMOVE"}, orphanRemoval=true)
+     */
     protected $u2fKeys;
 
     /**

Spacing +3 added lines, -3 removed lines

@@ -754,7 +754,7 @@ 
     public function invalidateBackupCode(string $code): void
     {
         $key = array_search($code, $this->backupCodes);
-        if ($key !== false){
+        if ($key !== false) {
             unset($this->backupCodes[$key]);
         }
     }
@@ -776,7 +776,7 @@ 
     public function setBackupCodes(array $codes) : self
     {
         $this->backupCodes = $codes;
-        if(empty($codes)) {
+        if (empty($codes)) {
             $this->backupCodesGenerationDate = null;
         } else {
             $this->backupCodesGenerationDate = new \DateTime();
@@ -853,7 +853,7 @@ 
     public function getPreferredTwoFactorProvider(): ?string
     {
         //If U2F is available then prefer it
-        if($this->isU2FAuthEnabled()) {
+        if ($this->isU2FAuthEnabled()) {
             return 'u2f_two_factor';
         }