Part-DB / Part-DB-symfony

tests/Controller/AdminPages/AbstractAdminControllerTest.php

Spacing +3 added lines, -3 removed lines

@@ -90,7 +90,7 @@ 
         $client->request('GET', static::$base_path.'/new');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     /**
@@ -115,7 +115,7 @@ 
         $client->request('GET', static::$base_path.'/1');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     public function deleteDataProvider(): array
@@ -152,6 +152,6 @@ 
 
         //Page is redirected to '/new', when delete was successful
         $this->assertSame($delete, $client->getResponse()->isRedirect(static::$base_path.'/new'));
-        $this->assertSame($delete, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($delete, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 }

src/EventSubscriber/SymfonyDebugToolbarSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -81,7 +81,7 @@
 
     public function onKernelResponse(ResponseEvent $event): void
     {
-        if (! $this->kernel_debug) {
+        if (!$this->kernel_debug) {
             return;
         }
 

src/Security/Voter/ExtendedVoter.php

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
         }
 
         // if the user is anonymous, we use the anonymous user.
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             /** @var UserRepository $repo */
             $repo = $this->entityManager->getRepository(User::class);
             $user = $repo->getAnonymousUser();

src/Services/GitVersionInfo.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
             $git = file($this->project_dir.'/.git/HEAD');
             $head = explode('/', $git[0], 3);
 
-            if (! isset($head[2])) {
+            if (!isset($head[2])) {
                 return null;
             }
 
@@ -89,7 +89,7 @@ 
         if (is_file($filename)) {
             $head = file($filename);
 
-            if (! isset($head[0])) {
+            if (!isset($head[0])) {
                 return null;
             }
 

src/Form/AdminPages/BaseEntityAdminForm.php

Spacing +9 added lines, -9 removed lines

@@ -93,14 +93,14 @@ 
                 'attr' => [
                     'placeholder' => 'part.name.placeholder',
                 ],
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             ])
 
             ->add('parent', StructuralEntityType::class, [
                 'class' => get_class($entity),
                 'required' => false,
                 'label' => 'parent.label',
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
             ])
 
             ->add('not_selectable', CheckboxType::class, [
@@ -110,7 +110,7 @@ 
                 'label_attr' => [
                     'class' => 'checkbox-custom',
                 ],
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             ])
 
             ->add('comment', CKEditorType::class, [
@@ -121,7 +121,7 @@ 
                     'rows' => 4,
                 ],
                 'help' => 'bbcode.hint',
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             ]);
 
         $this->additionalFormElements($builder, $options, $entity);
@@ -133,7 +133,7 @@ 
             'allow_delete' => true,
             'label' => false,
             'reindex_enable' => true,
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             'entry_options' => [
                 'data_class' => $options['attachment_class'],
             ],
@@ -142,7 +142,7 @@ 
 
         $builder->add('master_picture_attachment', MasterPictureAttachmentType::class, [
             'required' => false,
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             'label' => 'part.edit.master_attachment',
             'entity' => $entity,
         ]);
@@ -158,7 +158,7 @@ 
             'entry_type' => ParameterType::class,
             'allow_add' => $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             'allow_delete' => $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             'reindex_enable' => true,
             'label' => false,
             'by_reference' => false,
@@ -174,11 +174,11 @@ 
             'attr' => [
                 'class' => $is_new ? 'btn-success' : '',
             ],
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
         ])
             ->add('reset', ResetType::class, [
                 'label' => 'entity.edit.reset',
-                'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
             ]);
     }
 

src/Controller/PartController.php

Spacing +5 added lines, -5 removed lines

@@ -183,7 +183,7 @@ 
             $this->addFlash('info', 'part.edited_flash');
             //Reload form, so the SIUnitType entries use the new part unit
             $form = $this->createForm(PartBaseType::class, $part);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'part.edited_flash.invalid');
         }
 
@@ -227,7 +227,7 @@ 
      * @return Response
      */
     public function new(Request $request, EntityManagerInterface $em, TranslatorInterface $translator,
-        AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null): Response
+        AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null) : Response
     {
         if (null === $part) {
             $new_part = new Part();
@@ -256,7 +256,7 @@ 
         }
 
         $store_id = $request->get('storelocation', null);
-        $storelocation = $store_id ? $em->find(Storelocation::class, $store_id): null;
+        $storelocation = $store_id ? $em->find(Storelocation::class, $store_id) : null;
         if (null !== $storelocation && $new_part->getPartLots()->isEmpty()) {
             $partLot = new PartLot();
             $partLot->setStorageLocation($storelocation);
@@ -265,7 +265,7 @@ 
         }
 
         $supplier_id = $request->get('supplier', null);
-        $supplier = $supplier_id ? $em->find(Supplier::class, $supplier_id): null;
+        $supplier = $supplier_id ? $em->find(Supplier::class, $supplier_id) : null;
         if (null !== $supplier && $new_part->getOrderdetails()->isEmpty()) {
             $orderdetail = new Orderdetail();
             $orderdetail->setSupplier($supplier);
@@ -306,7 +306,7 @@ 
             return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'part.created_flash.invalid');
         }
 

src/Controller/SecurityController.php

Spacing +3 added lines, -3 removed lines

@@ -100,7 +100,7 @@ 
      */
     public function requestPwReset(PasswordResetManager $passwordReset, Request $request)
     {
-        if (! $this->allow_email_pw_reset) {
+        if (!$this->allow_email_pw_reset) {
             throw new AccessDeniedHttpException('The password reset via email is disabled!');
         }
 
@@ -144,7 +144,7 @@ 
      */
     public function pwResetNewPw(PasswordResetManager $passwordReset, Request $request, EntityManagerInterface $em, EventDispatcherInterface $eventDispatcher, ?string $user = null, ?string $token = null)
     {
-        if (! $this->allow_email_pw_reset) {
+        if (!$this->allow_email_pw_reset) {
             throw new AccessDeniedHttpException('The password reset via email is disabled!');
         }
 
@@ -189,7 +189,7 @@ 
             $data = $form->getData();
             //Try to set the new password
             $success = $passwordReset->setNewPassword($data['username'], $data['token'], $data['new_password']);
-            if (! $success) {
+            if (!$success) {
                 $this->addFlash('error', 'pw_reset.new_pw.error');
             } else {
                 $this->addFlash('success', 'pw_reset.new_pw.success');

src/Form/UserSettingsType.php

Spacing +6 added lines, -6 removed lines

@@ -72,27 +72,27 @@ 
         $builder
             ->add('name', TextType::class, [
                 'label' => 'user.username.label',
-                'disabled' => ! $this->security->isGranted('edit_username', $options['data']) || $this->demo_mode,
+                'disabled' => !$this->security->isGranted('edit_username', $options['data']) || $this->demo_mode,
             ])
             ->add('first_name', TextType::class, [
                 'required' => false,
                 'label' => 'user.firstName.label',
-                'disabled' => ! $this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
+                'disabled' => !$this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
             ])
             ->add('last_name', TextType::class, [
                 'required' => false,
                 'label' => 'user.lastName.label',
-                'disabled' => ! $this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
+                'disabled' => !$this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
             ])
             ->add('department', TextType::class, [
                 'required' => false,
                 'label' => 'user.department.label',
-                'disabled' => ! $this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
+                'disabled' => !$this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
             ])
             ->add('email', EmailType::class, [
                 'required' => false,
                 'label' => 'user.email.label',
-                'disabled' => ! $this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
+                'disabled' => !$this->security->isGranted('edit_infos', $options['data']) || $this->demo_mode,
             ])
             ->add('language', LanguageType::class, [
                 'disabled' => $this->demo_mode,
@@ -124,7 +124,7 @@ 
                 ],
                 'translation_domain' => false,
                 'choices' => User::AVAILABLE_THEMES,
-                'choice_label' => function ($entity, $key, $value) {
+                'choice_label' => function($entity, $key, $value) {
                     return $value;
                 },
                 'placeholder' => 'user_settings.theme.placeholder',

src/DataTables/LogDataTable.php

Spacing +12 added lines, -12 removed lines

@@ -102,7 +102,7 @@ 
         $optionsResolver->setAllowedTypes('filter_elements', ['array', 'object']);
         $optionsResolver->setAllowedTypes('mode', 'string');
 
-        $optionsResolver->setNormalizer('filter_elements', function (Options $options, $value) {
+        $optionsResolver->setNormalizer('filter_elements', function(Options $options, $value) {
             if (!is_array($value)) {
                 return [$value];
             }
@@ -120,7 +120,7 @@ 
 
         $dataTable->add('symbol', TextColumn::class, [
             'label' => '',
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 switch ($context->getLevelString()) {
                     case LogLevel::DEBUG:
                         $symbol = 'fa-bug';
@@ -181,7 +181,7 @@ 
         $dataTable->add('type', TextColumn::class, [
             'label' => $this->translator->trans('log.type'),
             'propertyPath' => 'type',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $this->translator->trans('log.type.'.$value);
             },
         ]);
@@ -190,14 +190,14 @@ 
             'label' => $this->translator->trans('log.level'),
             'visible' => 'system_log' === $options['mode'],
             'propertyPath' => 'levelString',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $value;
             },
         ]);
 
         $dataTable->add('user', TextColumn::class, [
             'label' => $this->translator->trans('log.user'),
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $user = $context->getUser();
 
                 return sprintf(
@@ -211,7 +211,7 @@ 
         $dataTable->add('target_type', TextColumn::class, [
             'label' => $this->translator->trans('log.target_type'),
             'visible' => false,
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $class = $context->getTargetClass();
                 if (null !== $class) {
                     return $this->elementTypeNameGenerator->getLocalizedTypeLabel($class);
@@ -233,7 +233,7 @@ 
         $dataTable->add('timeTravel', IconLinkColumn::class, [
             'label' => '',
             'icon' => 'fas fa-fw fa-eye',
-            'href' => function ($value, AbstractLogEntry $context) {
+            'href' => function($value, AbstractLogEntry $context) {
                 if (
                     ($context instanceof TimeTravelInterface
                         && $context->hasOldDataInformations())
@@ -251,10 +251,10 @@ 
 
                 return null;
             },
-            'disabled' => function ($value, AbstractLogEntry $context) {
+            'disabled' => function($value, AbstractLogEntry $context) {
                 return
-                    ! $this->security->isGranted('@tools.timetravel')
-                    || ! $this->security->isGranted('show_history', $context->getTargetClass());
+                    !$this->security->isGranted('@tools.timetravel')
+                    || !$this->security->isGranted('show_history', $context->getTargetClass());
             },
         ]);
 
@@ -266,7 +266,7 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => AbstractLogEntry::class,
-            'query' => function (QueryBuilder $builder) use ($options): void {
+            'query' => function(QueryBuilder $builder) use ($options): void {
                 $this->getQuery($builder, $options);
             },
         ]);
@@ -292,7 +292,7 @@ 
             ]);
         }
 
-        if (! empty($options['filter_elements'])) {
+        if (!empty($options['filter_elements'])) {
             foreach ($options['filter_elements'] as $element) {
                 /** @var AbstractDBElement $element */