Part-DB / Part-DB-symfony

src/Controller/AdminPages/BaseAdminController.php

Spacing +7 added lines, -7 removed lines

@@ -80,7 +80,7 @@ 
             //Rebuild form, so it is based on the updated data. Important for the parent field!
             //We can not use dynamic form events here, because the parent entity list is build from database!
             $form = $this->createForm($this->form_class, $entity);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $this->translator->trans('entity.edit_flash.invalid'));
         }
 
@@ -107,8 +107,8 @@ 
             $em->flush();
             $this->addFlash('success', $this->translator->trans('entity.created_flash'));
 
-            return $this->redirectToRoute($this->route_base . '_edit', ['id' => $new_entity->getID()]);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+            return $this->redirectToRoute($this->route_base.'_edit', ['id' => $new_entity->getID()]);
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $this->translator->trans('entity.created_flash.invalid'));
         }
 
@@ -128,7 +128,7 @@ 
 
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -147,7 +147,7 @@ 
             //Show errors to user:
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -188,7 +188,7 @@ 
             $this->addFlash('success', 'attachment_type.deleted');
         }
 
-        return $this->redirectToRoute($this->route_base .  '_new');
+        return $this->redirectToRoute($this->route_base.'_new');
     }
 
     protected function _exportAll(EntityManagerInterface $em, EntityExporter $exporter, Request $request)
@@ -199,7 +199,7 @@ 
 
         $entities = $em->getRepository($this->entity_class)->findAll();
 
-        return $exporter->exportEntityFromRequest($entities,$request);
+        return $exporter->exportEntityFromRequest($entities, $request);
     }
 
     protected function _exportEntity(NamedDBElement $entity, EntityExporter $exporter, Request $request)

src/Form/Part/PartBaseType.php

Spacing +2 added lines, -2 removed lines

@@ -77,7 +77,7 @@ 
             ->add('description', CKEditorType::class, ['required' => false, 'empty_data' => '',
                 'label' => 'description.label', 'help' => 'bbcode.hint', 'config_name' => 'description_config',
                 'attr' => ['placeholder' => 'part.description.placeholder', 'rows' => 2],
-                'disabled' => !$this->security->isGranted('description.edit', $part) ])
+                'disabled' => !$this->security->isGranted('description.edit', $part)])
             ->add('minAmount', SIUnitType::class,
                 ['attr' => ['min' => 0, 'placeholder' => 'part.mininstock.placeholder'], 'label' => 'mininstock.label',
                     'measurement_unit' => $part->getPartUnit(),
@@ -90,7 +90,7 @@ 
                 'disabled' => !$this->security->isGranted('move', $part), ])
             ->add('tags', TextType::class, ['required' => false, 'label' => 'part.tags', 'empty_data' => "",
                 'attr' => ['data-role' => 'tagsinput'],
-                'disabled' => !$this->security->isGranted('edit', $part) ]);
+                'disabled' => !$this->security->isGranted('edit', $part)]);
 
         //Manufacturer section
         $builder->add('manufacturer', StructuralEntityType::class, ['class' => Manufacturer::class,

src/Services/AmountFormatter.php

Spacing +6 added lines, -6 removed lines

@@ -52,7 +52,7 @@ 
     protected function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'show_prefix' => function (Options $options) {
+            'show_prefix' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -60,7 +60,7 @@ 
                 }
                 return true;
             },
-            'is_integer' => function (Options $options) {
+            'is_integer' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -68,7 +68,7 @@ 
                 }
                 return true;
             },
-            'unit' => function (Options $options) {
+            'unit' => function(Options $options) {
                 if ($options['measurement_unit'] !== null) {
                     /** @var MeasurementUnit $unit */
                     $unit = $options['measurement_unit'];
@@ -77,12 +77,12 @@ 
                 return '';
             },
             'decimals' => 2,
-            'error_mapping' => [ '.' => 'value']
+            'error_mapping' => ['.' => 'value']
         ]);
 
         $resolver->setAllowedTypes('decimals', 'int');
 
-        $resolver->setNormalizer('decimals', function (Options $options, $value) {
+        $resolver->setNormalizer('decimals', function(Options $options, $value) {
             // If the unit is integer based, then dont show any decimals
             if ($options['is_integer']) {
                 return 0;
@@ -125,7 +125,7 @@ 
         }
 
         //Otherwise just output it
-        $format_string = '%.' . $options['decimals'] . 'f ' . $options['unit'];
+        $format_string = '%.'.$options['decimals'].'f '.$options['unit'];
         return sprintf($format_string, $value);
     }
 }
\ No newline at end of file

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