Part-DB / Part-DB-symfony

src/Command/BackupCommand.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
         }
 
         //When all options are false, we abort and show an error message
-        if (! $backup_database && ! $backup_attachments && ! $backup_config) {
+        if (!$backup_database && !$backup_attachments && !$backup_config) {
             $io->error('You have to select at least one option what to backup! Use --full to backup everything.');
 
             return Command::FAILURE;

src/Form/Type/Helper/StructuralEntityChoiceLoader.php

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
 
         $results = [];
 
-        foreach($entities as $entity) {
+        foreach ($entities as $entity) {
             //If the entity is newly created (ID null), add it as result and persist it.
             if ($entity->getID() === null) {
                 $this->entityManager->persist($entity);

src/Form/Type/CurrencyEntityType.php

Spacing +5 added lines, -5 removed lines

@@ -58,7 +58,7 @@ 
         // This options allows you to override the currency shown for the null value
         $resolver->setDefault('base_currency', null);
 
-        $resolver->setDefault('empty_message', function (Options $options) {
+        $resolver->setDefault('empty_message', function(Options $options) {
             //By default we use the global base currency:
             $iso_code = $this->base_currency;
 
@@ -83,7 +83,7 @@ 
             throw new RuntimeException('The choice must be an instance of '.Currency::class);
         }
 
-        if(!empty($choice->getIsoCode())) {
+        if (!empty($choice->getIsoCode())) {
             $symbol = Currencies::getSymbol($choice->getIsoCode());
         } else {
             $symbol = null;
@@ -104,7 +104,7 @@ 
 
     protected function getChoiceContent(AbstractStructuralDBElement $choice, $key, $value, $options): string
     {
-        if(!$choice instanceof Currency) {
+        if (!$choice instanceof Currency) {
             throw new RuntimeException('$choice must be an instance of Currency!');
         }
 
@@ -121,10 +121,10 @@ 
         $tmp = str_repeat('<span class="picker-level"></span>', $level);
 
         //Show currency symbol or ISO code and the name of the currency
-        if(!empty($choice->getIsoCode())) {
+        if (!empty($choice->getIsoCode())) {
             $tmp .= Currencies::getSymbol($choice->getIsoCode());
             //Add currency name as badge
-            $tmp .= sprintf('<span class="badge bg-primary ms-2 %s">%s</span>', $options['short'] ? 'picker-hs' : '' , htmlspecialchars($choice->getName()));
+            $tmp .= sprintf('<span class="badge bg-primary ms-2 %s">%s</span>', $options['short'] ? 'picker-hs' : '', htmlspecialchars($choice->getName()));
         } else {
             $tmp .= htmlspecialchars($choice->getName());
         }

tests/Repository/StructuralDBElementRepositoryTest.php

Spacing +9 added lines, -9 removed lines

@@ -126,15 +126,15 @@ 
 
         $this->assertSame('Node 1', $nodes[0]->getName());
         //Node must be from DB
-        $this->assertNotNull( $nodes[0]->getID());
+        $this->assertNotNull($nodes[0]->getID());
 
         $this->assertSame('Node 1.1', $nodes[1]->getName());
         //Node must be from DB
-        $this->assertNotNull( $nodes[1]->getID());
+        $this->assertNotNull($nodes[1]->getID());
 
         $this->assertSame('Node 1.1.1', $nodes[2]->getName());
         //Node must be from DB
-        $this->assertNotNull( $nodes[2]->getID());
+        $this->assertNotNull($nodes[2]->getID());
     }
 
     public function testGetNewEntityNew(): void
@@ -145,15 +145,15 @@ 
 
         $this->assertSame('Element 1', $nodes[0]->getName());
         //Node must not be from DB
-        $this->assertNull( $nodes[0]->getID());
+        $this->assertNull($nodes[0]->getID());
 
         $this->assertSame('Element 1.1', $nodes[1]->getName());
         //Node must not be from DB
-        $this->assertNull( $nodes[1]->getID());
+        $this->assertNull($nodes[1]->getID());
 
         $this->assertSame('Element 1.1.1', $nodes[2]->getName());
         //Node must not be from DB
-        $this->assertNull( $nodes[2]->getID());
+        $this->assertNull($nodes[2]->getID());
     }
 
     public function testGetNewEntityMixed(): void
@@ -164,14 +164,14 @@ 
 
         $this->assertSame('Node 1', $nodes[0]->getName());
         //Node must be from DB
-        $this->assertNotNull( $nodes[0]->getID());
+        $this->assertNotNull($nodes[0]->getID());
 
         $this->assertSame('Node 1.1', $nodes[1]->getName());
         //Node must be from DB
-        $this->assertNotNull( $nodes[1]->getID());
+        $this->assertNotNull($nodes[1]->getID());
 
         $this->assertSame('New Node', $nodes[2]->getName());
         //Node must not be from DB
-        $this->assertNull( $nodes[2]->getID());
+        $this->assertNull($nodes[2]->getID());
     }
 }

src/Form/Type/StructuralEntityType.php

Spacing +14 added lines, -14 removed lines

@@ -71,7 +71,7 @@ 
 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
-        $builder->addEventListener(FormEvents::PRE_SUBMIT, function (PreSubmitEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SUBMIT, function(PreSubmitEvent $event) {
             //When the data contains non-digit characters, we assume that the user entered a new element.
             //In that case we add the new element to our choice_loader
 
@@ -89,9 +89,9 @@ 
         });
 
         $builder->addModelTransformer(new CallbackTransformer(
-            function ($value) use ($options) {
+            function($value) use ($options) {
                 return $this->modelTransform($value, $options);
-            }, function ($value) use ($options) {
+            }, function($value) use ($options) {
             return $this->modelReverseTransform($value, $options);
         }));
     }
@@ -102,9 +102,9 @@ 
         $resolver->setDefaults([
             'allow_add' => false,
             '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
-            'choice_value' => function (?AbstractStructuralDBElement $element) {
+            'subentities_of' => null, //Only show entities with the given parent class
+            'disable_not_selectable' => false, //Disable entries with not selectable property
+            'choice_value' => function(?AbstractStructuralDBElement $element) {
                 if ($element === null) {
                     return null;
                 }
@@ -116,20 +116,20 @@ 
 
                 return $element->getID();
             }, //Use the element id as option value and for comparing items
-            'choice_loader' => function (Options $options) {
+            'choice_loader' => function(Options $options) {
                 return new StructuralEntityChoiceLoader($options, $this->builder, $this->em);
             },
-            'choice_label' => function (Options $options) {
-                return function ($choice, $key, $value) use ($options) {
+            'choice_label' => function(Options $options) {
+                return function($choice, $key, $value) use ($options) {
                     return $this->generateChoiceLabels($choice, $key, $value, $options);
                 };
             },
-            'choice_attr' => function (Options $options) {
-                return function ($choice, $key, $value) use ($options) {
+            'choice_attr' => function(Options $options) {
+                return function($choice, $key, $value) use ($options) {
                     return $this->generateChoiceAttr($choice, $key, $value, $options);
                 };
             },
-            'group_by' => function (AbstractStructuralDBElement $element)
+            'group_by' => function(AbstractStructuralDBElement $element)
             {
                 //Show entities that are not added to DB yet separately from other entities
                 if ($element->getID() === null) {
@@ -142,7 +142,7 @@ 
         ]);
 
         //Set the constraints for the case that allow add is enabled (we then have to check that the new element is valid)
-        $resolver->setNormalizer('constraints', function (Options $options, $value) {
+        $resolver->setNormalizer('constraints', function(Options $options, $value) {
             if ($options['allow_add']) {
                 $value[] = new Valid();
             }
@@ -154,7 +154,7 @@ 
 
         $resolver->setDefault('controller', 'elements--structural-entity-select');
 
-        $resolver->setDefault('attr', function (Options $options) {
+        $resolver->setDefault('attr', function(Options $options) {
             $tmp = [
                 'data-controller' => $options['controller'],
                 'data-allow-add' => $options['allow_add'] ? 'true' : 'false',

src/Form/UserSettingsType.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
             ->add('reset', ResetType::class, ['label' => 'reset']);
 
         //Add the remove_avatar button if the user has an avatar (we have to add this dynamically)
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (PreSetDataEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(PreSetDataEvent $event) {
             $data = $event->getData();
             if (!$data instanceof User) {
                 return;

src/Form/Type/PartLotSelectType.php

Spacing +4 added lines, -4 removed lines

@@ -43,12 +43,12 @@
 
         $resolver->setDefaults([
             'class' => PartLot::class,
-            'choice_label' => ChoiceList::label($this, function (PartLot $part_lot) {
+            'choice_label' => ChoiceList::label($this, function(PartLot $part_lot) {
                 return ($part_lot->getStorageLocation() ? $part_lot->getStorageLocation()->getFullPath() : '')
-                    . ' (' . $part_lot->getName() . '): ' . $part_lot->getAmount();
+                    . ' ('.$part_lot->getName().'): '.$part_lot->getAmount();
             }),
-            'query_builder' => function (Options $options) {
-                return function (EntityRepository $er) use ($options) {
+            'query_builder' => function(Options $options) {
+                return function(EntityRepository $er) use ($options) {
                     return $er->createQueryBuilder('l')
                         ->where('l.part = :part')
                         ->setParameter('part', $options['part']);

src/Form/Type/ThemeChoiceType.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
     {
         $resolver->setDefaults([
             'choices' => $this->available_themes,
-            'choice_label' => static function ($entity, $key, $value) {
+            'choice_label' => static function($entity, $key, $value) {
                 return $value;
             },
             'choice_translation_domain' => false,

src/Form/Type/MasterPictureAttachmentType.php

Spacing +4 added lines, -4 removed lines

@@ -42,8 +42,8 @@ 
         $resolver->setDefaults([
             'filter' => 'picture',
             'choice_translation_domain' => false,
-            'choice_attr' => static function (Options $options) {
-                return  static function ($choice, $key, $value) use ($options) {
+            'choice_attr' => static function(Options $options) {
+                return  static function($choice, $key, $value) use ($options) {
                     /** @var Attachment $choice */
                     $tmp = ['data-subtext' => $choice->getFilename() ?? 'URL'];
 
@@ -57,9 +57,9 @@ 
                 };
             },
             'choice_label' => 'name',
-            'choice_loader' => static function (Options $options) {
+            'choice_loader' => static function(Options $options) {
                 return new CallbackChoiceLoader(
-                    static function () use ($options) {
+                    static function() use ($options) {
                         $entity = $options['entity'];
                         if (!$entity instanceof AttachmentContainingDBElement) {
                             throw new RuntimeException('$entity must have Attachments! (be of type AttachmentContainingDBElement)');