Part-DB / Part-DB-symfony

src/Controller/AdminPages/BaseAdminController.php

Spacing +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
         $form->handleRequest($request);
         if ($form->isSubmitted() && $form->isValid()) {
             //Check if we editing a user and if we need to change the password of it
-            if ($entity instanceof User && ! empty($form['new_password']->getData())) {
+            if ($entity instanceof User && !empty($form['new_password']->getData())) {
                 $password = $this->passwordEncoder->encodePassword($entity, $form['new_password']->getData());
                 $entity->setPassword($password);
                 //By default the user must change the password afterwards
@@ -199,7 +199,7 @@ 
                 'attachment_class' => $this->attachment_class,
                 'parameter_class' => $this->parameter_class,
             ]);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'entity.edit_flash.invalid');
         }
 
@@ -229,7 +229,7 @@ 
         $form->handleRequest($request);
 
         if ($form->isSubmitted() && $form->isValid()) {
-            if ($new_entity instanceof User && ! empty($form['new_password']->getData())) {
+            if ($new_entity instanceof User && !empty($form['new_password']->getData())) {
                 $password = $this->passwordEncoder->encodePassword($new_entity, $form['new_password']->getData());
                 $new_entity->setPassword($password);
                 //By default the user must change the password afterwards
@@ -264,7 +264,7 @@ 
             return $this->redirectToRoute($this->route_base.'_edit', ['id' => $new_entity->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'entity.created_flash.invalid');
         }
 

src/Services/EntityURLGenerator.php

Spacing +1 added lines, -1 removed lines

@@ -381,7 +381,7 @@
         $class = get_class($entity);
 
         //Check if we have an direct mapping for the given class
-        if (! array_key_exists($class, $map)) {
+        if (!array_key_exists($class, $map)) {
             //Check if we need to check inheritance by looping through our map
             foreach (array_keys($map) as $key) {
                 if (is_a($entity, $key)) {

src/Services/Parameters/ParameterExtractor.php

Spacing +2 added lines, -2 removed lines

@@ -40,7 +40,7 @@ 
      */
     public function extractParameters(string $input, string $class = PartParameter::class): array
     {
-        if (! is_a($class, AbstractParameter::class, true)) {
+        if (!is_a($class, AbstractParameter::class, true)) {
             throw new \InvalidArgumentException('$class must be a child class of AbstractParameter!');
         }
 
@@ -68,7 +68,7 @@ 
 
         $matches = [];
         \preg_match($regex, $input, $matches);
-        if (! empty($matches)) {
+        if (!empty($matches)) {
             [, $name, $value] = $matches;
             $value = trim($value);
 

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/Form/CollectionTypeExtension.php

Spacing +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
 
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) use ($options) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($options) {
             $data = $event->getData();
             $config = $event->getForm()->getConfig();
             //If enabled do a reindexing of the collection
@@ -84,8 +84,8 @@ 
                 $error_mapping = [];
 
                 foreach ($data->toArray() as $key => $item) {
-                    $index = $options['reindex_prefix'] . $this->propertyAccess->getValue($item, $options['reindex_path']);
-                    $error_mapping['[' . $key . ']'] = $index;
+                    $index = $options['reindex_prefix'].$this->propertyAccess->getValue($item, $options['reindex_path']);
+                    $error_mapping['['.$key.']'] = $index;
                     $reindexed_data->set($index, $item);
                 }
                 $event->setData($reindexed_data);