Part-DB / Part-DB-symfony

src/Form/Type/SIUnitType.php

Spacing +6 added lines, -6 removed lines

@@ -58,7 +58,7 @@ 
     {
         $resolver->setDefaults([
             'measurement_unit' => null,
-            'show_prefix' => function (Options $options) {
+            'show_prefix' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -66,7 +66,7 @@ 
                 }
                 return false;
             },
-            'is_integer' => function (Options $options) {
+            'is_integer' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -74,7 +74,7 @@ 
                 }
                 return false;
             },
-            'unit' => function (Options $options) {
+            'unit' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -82,7 +82,7 @@ 
                 }
                 return null;
             },
-            'error_mapping' => [ '.' => 'value']
+            'error_mapping' => ['.' => 'value']
         ]);
 
         $resolver->setAllowedTypes('measurement_unit', [MeasurementUnit::class, 'null']);
@@ -92,7 +92,7 @@ 
         $resolver->setDefaults([
                 'min' => 0,
                 'max' => '',
-                'step' => function (Options $options) {
+                'step' => function(Options $options) {
                     if ($options['is_integer'] === true) {
                         return 1;
                     }
@@ -117,7 +117,7 @@ 
 
         if ($options['show_prefix']) {
             $builder->add('prefix', ChoiceType::class, [
-                'choices' => ['M' => 6, 'k' => 3, '' => 0, 'm' => -3, 'µ' => -6 ]
+                'choices' => ['M' => 6, 'k' => 3, '' => 0, 'm' => -3, 'µ' => -6]
             ]);
         }
 

src/Services/AttachmentHelper.php

Spacing +7 added lines, -7 removed lines

@@ -57,7 +57,7 @@ 
         if ($fs->isAbsolutePath($tmp_base_path)) {
             $this->base_path = $tmp_base_path;
         } else {
-            $this->base_path = realpath($kernel->getProjectDir() . DIRECTORY_SEPARATOR . $tmp_base_path);
+            $this->base_path = realpath($kernel->getProjectDir().DIRECTORY_SEPARATOR.$tmp_base_path);
         }
     }
 
@@ -159,7 +159,7 @@ 
 
         $sz = 'BKMGTP';
         $factor = (int) floor((strlen($bytes) - 1) / 3);
-        return sprintf("%.{$decimals}f", $bytes / 1024 ** $factor) . @$sz[$factor];
+        return sprintf("%.{$decimals}f", $bytes / 1024 ** $factor).@$sz[$factor];
     }
 
     /**
@@ -171,7 +171,7 @@ 
     {
         $mapping = [PartAttachment::class => 'part'];
 
-        $path = $this->base_path . DIRECTORY_SEPARATOR . $mapping[get_class($attachment)] . DIRECTORY_SEPARATOR . $attachment->getElement()->getID();
+        $path = $this->base_path.DIRECTORY_SEPARATOR.$mapping[get_class($attachment)].DIRECTORY_SEPARATOR.$attachment->getElement()->getID();
         return $path;
     }
 
@@ -193,17 +193,17 @@ 
         //Sanatize filename
         $originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
         $safeFilename = transliterator_transliterate('Any-Latin; Latin-ASCII; [^A-Za-z0-9_] remove; Lower()', $originalFilename);
-        $newFilename = $safeFilename . '.' . $file->getClientOriginalExtension();
+        $newFilename = $safeFilename.'.'.$file->getClientOriginalExtension();
 
         //If a file with this name is already existing add a number to the filename
-        if (file_exists($folder . DIRECTORY_SEPARATOR . $newFilename)) {
+        if (file_exists($folder.DIRECTORY_SEPARATOR.$newFilename)) {
             $bak = $newFilename;
 
             $number = 1;
-            $newFilename = $folder . DIRECTORY_SEPARATOR . $safeFilename . '-' . $number . '.' . $file->getClientOriginalExtension();
+            $newFilename = $folder.DIRECTORY_SEPARATOR.$safeFilename.'-'.$number.'.'.$file->getClientOriginalExtension();
             while (file_exists($newFilename)) {
                 $number++;
-                $newFilename = $folder . DIRECTORY_SEPARATOR . $safeFilename . '-' . $number . '.' . $file->getClientOriginalExtension();
+                $newFilename = $folder.DIRECTORY_SEPARATOR.$safeFilename.'-'.$number.'.'.$file->getClientOriginalExtension();
             }
         }
 

src/Controller/PartController.php

Spacing +4 added lines, -4 removed lines

@@ -85,7 +85,7 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
 
@@ -94,7 +94,7 @@ 
             $this->addFlash('info', $translator->trans('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', $translator->trans('part.edited_flash.invalid'));
         }
 
@@ -135,7 +135,7 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
             $em->persist($new_part);
@@ -143,7 +143,7 @@ 
             $this->addFlash('success', $translator->trans('part.created_flash'));
 
             return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $translator->trans('part.created_flash.invalid'));
         }
 

src/Command/CleanAttachmentsCommand.php

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
         $io = new SymfonyStyle($input, $output);
 
         $mediaPath = $this->attachment_helper->getMediaPath();
-        $io->note("The media path is " . $mediaPath);
+        $io->note("The media path is ".$mediaPath);
 
         $finder = new Finder();
         //We look for files in the media folder only
@@ -103,8 +103,8 @@ 
      */
     protected function removeEmptySubFolders($path)
     {
-        $empty=true;
-        foreach (glob($path . DIRECTORY_SEPARATOR . "*") as $file)
+        $empty = true;
+        foreach (glob($path.DIRECTORY_SEPARATOR."*") as $file)
         {
             $empty &= is_dir($file) && $this->removeEmptySubFolders($file);
         }

src/Form/AdminPages/CurrencyAdminForm.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     {
         $is_new = $entity->getID() === null;
 
-        $builder->add('iso_code', CurrencyType::class , ['required' => true,
+        $builder->add('iso_code', CurrencyType::class, ['required' => true,
             'required' => false,
             'label' => 'currency.iso_code.label',
             'preferred_choices' => ['EUR', 'USD', 'GBP', 'JPY', 'CNY'],

src/Form/Part/OrderdetailType.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
 
 
         //Add pricedetails after we know the data, so we can set the default currency
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) use ($options) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($options) {
             /** @var Orderdetail $orderdetail */
             $orderdetail = $event->getData();
 

src/Form/Type/CurrencyEntityType.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,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;
 

src/Form/Type/StructuralEntityType.php

Spacing +11 added lines, -11 removed lines

@@ -74,9 +74,9 @@ 
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         $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);
         }));
     }
@@ -86,24 +86,24 @@ 
         $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);
             }
         ]);
 
         $resolver->setDefault('empty_message', null);
 
-        $resolver->setDefault('attr', function (Options $options) {
-            $tmp =  ['class' => 'selectpicker', 'data-live-search' => true];
+        $resolver->setDefault('attr', function(Options $options) {
+            $tmp = ['class' => 'selectpicker', 'data-live-search' => true];
             if ($options['empty_message']) {
                 $tmp['data-none-Selected-Text'] = $options['empty_message'];
             }
@@ -140,7 +140,7 @@ 
 
 
         $tmp = str_repeat('   ', $choice->getLevel()); //Use 3 spaces for intendation
-        $tmp .=  htmlspecialchars($choice->getName());
+        $tmp .= htmlspecialchars($choice->getName());
         return $tmp;
     }
 

src/Entity/PriceInformations/Pricedetail.php

Indentation -1 removed lines

@@ -188,7 +188,6 @@
      *          in the database, you have to pass the "price_related_quantity" count as $multiplier.
      *
      * @return string  the price as a bcmath string
-
      */
     public function getPricePerUnit($multiplier = 1.0) : string
     {

Spacing +1 added lines, -1 removed lines

@@ -356,6 +356,6 @@
      */
     public function getIDString(): string
     {
-        return 'PD' . sprintf('%06d', $this->getID());
+        return 'PD'.sprintf('%06d', $this->getID());
     }
 }