Part-DB / Part-DB-symfony

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);

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');
         }