Part-DB / Part-DB-symfony

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

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

src/Migrations/Version20190812154222.php

Spacing +5 added lines, -5 removed lines

@@ -23,7 +23,7 @@ 
         $old_db = true;
         try {
             //Check if we can use this migration method:
-            $version = (int)$this->connection->fetchColumn("SELECT keyValue AS version FROM `internal` WHERE `keyName` = 'dbVersion'");
+            $version = (int) $this->connection->fetchColumn("SELECT keyValue AS version FROM `internal` WHERE `keyName` = 'dbVersion'");
         } catch (DBALException $ex) {
             //when the table was not found, then we have an new database
             $old_db = false;
@@ -40,15 +40,15 @@ 
 
         /** Migrate the part locations for parts with known instock */
         $this->addSql(
-            'INSERT INTO part_lots (id_part, id_store_location, amount, instock_unknown, last_modified, datetime_added) ' .
-                'SELECT parts.id, parts.id_storelocation,  parts.instock, 0, NOW(), NOW() FROM parts ' .
+            'INSERT INTO part_lots (id_part, id_store_location, amount, instock_unknown, last_modified, datetime_added) '.
+                'SELECT parts.id, parts.id_storelocation,  parts.instock, 0, NOW(), NOW() FROM parts '.
                 'WHERE parts.instock >= 0'
         );
 
         //Migrate part locations for parts with unknown instock
         $this->addSql(
-            'INSERT INTO part_lots (id_part, id_store_location, amount, instock_unknown, last_modified, datetime_added) ' .
-            'SELECT parts.id, parts.id_storelocation, 0, 1, NOW(), NOW() FROM parts ' .
+            'INSERT INTO part_lots (id_part, id_store_location, amount, instock_unknown, last_modified, datetime_added) '.
+            'SELECT parts.id, parts.id_storelocation, 0, 1, NOW(), NOW() FROM parts '.
             'WHERE parts.instock = -2'
         );