Part-DB / Part-DB-symfony

src/Form/Type/StructuralEntityType.php

Spacing +9 added lines, -9 removed lines

@@ -82,9 +82,9 @@ 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->addModelTransformer(new CallbackTransformer(
-            function ($value) use ($options) {
+            function($value) use ($options) {
                 return $this->transform($value, $options);
-            }, function ($value) use ($options) {
+            }, function($value) use ($options) {
                 return $this->reverseTransform($value, $options);
             }));
     }
@@ -94,18 +94,18 @@ 
         $resolver->setRequired(['class']);
         $resolver->setDefaults([
             'show_fullpath_in_subtext' => true, //When this is enabled, the full path will be shown in subtext
-            'subentities_of' => null,   //Only show entities with the given parent class
-            'disable_not_selectable' => false,  //Disable entries with not selectable property
+            'subentities_of' => null, //Only show entities with the given parent class
+            'disable_not_selectable' => false, //Disable entries with not selectable property
             'choice_value' => 'id', //Use the element id as option value and for comparing items
-            'choice_loader' => function (Options $options) {
-                return new CallbackChoiceLoader(function () use ($options) {
+            'choice_loader' => function(Options $options) {
+                return new CallbackChoiceLoader(function() use ($options) {
                     return $this->getEntries($options);
                 });
             },
-            'choice_label' => function ($choice, $key, $value) {
+            'choice_label' => function($choice, $key, $value) {
                 return $this->generateChoiceLabels($choice, $key, $value);
             },
-            'choice_attr' => function ($choice, $key, $value) {
+            'choice_attr' => function($choice, $key, $value) {
                 return $this->generateChoiceAttr($choice, $key, $value);
             },
             'choice_translation_domain' => false, //Don't translate the entity names
@@ -113,7 +113,7 @@ 
 
         $resolver->setDefault('empty_message', null);
 
-        $resolver->setDefault('attr', function (Options $options) {
+        $resolver->setDefault('attr', function(Options $options) {
             $tmp = [
                 'class' => 'selectpicker',
                 'data-live-search' => true,

src/Form/Part/PartBaseType.php

Spacing +20 added lines, -20 removed lines

@@ -105,7 +105,7 @@ 
                 'attr' => [
                     'placeholder' => 'part.edit.name.placeholder',
                 ],
-                'disabled' => ! $this->security->isGranted('name.edit', $part),
+                'disabled' => !$this->security->isGranted('name.edit', $part),
             ])
             ->add('description', CKEditorType::class, [
                 'required' => false,
@@ -116,7 +116,7 @@ 
                     'placeholder' => 'part.edit.description.placeholder',
                     'rows' => 2,
                 ],
-                'disabled' => ! $this->security->isGranted('description.edit', $part),
+                'disabled' => !$this->security->isGranted('description.edit', $part),
             ])
             ->add('minAmount', SIUnitType::class, [
                 'attr' => [
@@ -125,20 +125,20 @@ 
                 ],
                 'label' => 'part.edit.mininstock',
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('minamount.edit', $part),
+                'disabled' => !$this->security->isGranted('minamount.edit', $part),
             ])
             ->add('category', StructuralEntityType::class, [
                 'class' => Category::class,
                 'label' => 'part.edit.category',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('category.edit', $part),
+                'disabled' => !$this->security->isGranted('category.edit', $part),
             ])
             ->add('footprint', StructuralEntityType::class, [
                 'class' => Footprint::class,
                 'required' => false,
                 'label' => 'part.edit.footprint',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('footprint.edit', $part),
+                'disabled' => !$this->security->isGranted('footprint.edit', $part),
             ])
             ->add('tags', TextType::class, [
                 'required' => false,
@@ -148,7 +148,7 @@ 
                     'class' => 'tagsinput',
                     'data-autocomplete' => $this->urlGenerator->generate('typeahead_tags', ['query' => 'QUERY']),
                 ],
-                'disabled' => ! $this->security->isGranted('tags.edit', $part),
+                'disabled' => !$this->security->isGranted('tags.edit', $part),
             ]);
 
         //Manufacturer section
@@ -157,25 +157,25 @@ 
             'required' => false,
             'label' => 'part.edit.manufacturer.label',
             'disable_not_selectable' => true,
-            'disabled' => ! $this->security->isGranted('manufacturer.edit', $part),
+            'disabled' => !$this->security->isGranted('manufacturer.edit', $part),
         ])
             ->add('manufacturer_product_url', UrlType::class, [
                 'required' => false,
                 'empty_data' => '',
                 'label' => 'part.edit.manufacturer_url.label',
-                'disabled' => ! $this->security->isGranted('mpn.edit', $part),
+                'disabled' => !$this->security->isGranted('mpn.edit', $part),
             ])
             ->add('manufacturer_product_number', TextType::class, [
                 'required' => false,
                 'empty_data' => '',
                 'label' => 'part.edit.mpn',
-                'disabled' => ! $this->security->isGranted('mpn.edit', $part),
+                'disabled' => !$this->security->isGranted('mpn.edit', $part),
             ])
             ->add('manufacturing_status', ChoiceType::class, [
                 'label' => 'part.edit.manufacturing_status',
                 'choices' => $status_choices,
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('status.edit', $part),
+                'disabled' => !$this->security->isGranted('status.edit', $part),
             ]);
 
         //Advanced section
@@ -185,7 +185,7 @@ 
             ],
             'required' => false,
             'label' => 'part.edit.needs_review',
-            'disabled' => ! $this->security->isGranted('edit', $part),
+            'disabled' => !$this->security->isGranted('edit', $part),
         ])
             ->add('favorite', CheckboxType::class, [
                 'label_attr' => [
@@ -193,20 +193,20 @@ 
                 ],
                 'required' => false,
                 'label' => 'part.edit.is_favorite',
-                'disabled' => ! $this->security->isGranted('change_favorite', $part),
+                'disabled' => !$this->security->isGranted('change_favorite', $part),
             ])
             ->add('mass', SIUnitType::class, [
                 'unit' => 'g',
                 'label' => 'part.edit.mass',
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('mass.edit', $part),
+                'disabled' => !$this->security->isGranted('mass.edit', $part),
             ])
             ->add('partUnit', StructuralEntityType::class, [
                 'class' => MeasurementUnit::class,
                 'required' => false,
                 'disable_not_selectable' => true,
                 'label' => 'part.edit.partUnit',
-                'disabled' => ! $this->security->isGranted('unit.edit', $part),
+                'disabled' => !$this->security->isGranted('unit.edit', $part),
             ]);
 
         //Comment section
@@ -216,7 +216,7 @@ 
             'attr' => [
                 'rows' => 4,
             ],
-            'disabled' => ! $this->security->isGranted('comment.edit', $part),
+            'disabled' => !$this->security->isGranted('comment.edit', $part),
             'empty_data' => '',
         ]);
 
@@ -228,7 +228,7 @@ 
             'label' => false,
             'entry_options' => [
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('lots.edit', $part),
+                'disabled' => !$this->security->isGranted('lots.edit', $part),
             ],
             'by_reference' => false,
         ]);
@@ -241,14 +241,14 @@ 
             'label' => false,
             'entry_options' => [
                 'data_class' => PartAttachment::class,
-                'disabled' => ! $this->security->isGranted('attachments.edit', $part),
+                'disabled' => !$this->security->isGranted('attachments.edit', $part),
             ],
             'by_reference' => false,
         ]);
 
         $builder->add('master_picture_attachment', MasterPictureAttachmentType::class, [
             'required' => false,
-            'disabled' => ! $this->security->isGranted('attachments.edit', $part),
+            'disabled' => !$this->security->isGranted('attachments.edit', $part),
             'label' => 'part.edit.master_attachment',
             'entity' => $part,
         ]);
@@ -263,7 +263,7 @@ 
             'prototype_data' => new Orderdetail(),
             'entry_options' => [
                 'measurement_unit' => $part->getPartUnit(),
-                'disabled' => ! $this->security->isGranted('orderdetails.edit', $part),
+                'disabled' => !$this->security->isGranted('orderdetails.edit', $part),
             ],
         ]);
 
@@ -275,7 +275,7 @@ 
             'by_reference' => false,
             'prototype_data' => new PartParameter(),
             'entry_options' => [
-                'disabled' => ! $this->security->isGranted('parameters.edit', $part),
+                'disabled' => !$this->security->isGranted('parameters.edit', $part),
                 'data_class' => PartParameter::class,
             ],
         ]);

src/Repository/LogEntryRepository.php

Spacing +1 added lines, -1 removed lines

@@ -171,7 +171,7 @@
         $query = $qb->getQuery();
         $count = $query->getSingleScalarResult();
 
-        return ! ($count > 0);
+        return !($count > 0);
     }
 
     /**

src/DataTables/Column/LogEntryTargetColumn.php

Spacing +1 added lines, -1 removed lines

@@ -98,7 +98,7 @@
         $tmp = '';
 
         //The element is existing
-        if ($target instanceof NamedElementInterface && ! empty($target->getName())) {
+        if ($target instanceof NamedElementInterface && !empty($target->getName())) {
             try {
                 $tmp = sprintf(
                     '<a href="%s">%s</a>',

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