Part-DB / Part-DB-symfony

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

src/Migrations/Version20190902140506.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,7 @@ 
 
         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'");
             $this->abortIf($version !== 26, "This database migration can only be used if the database version is 26! Install Part-DB 0.5.6 and update database there!");
         } catch (DBALException $ex) {
             //when the table was not found, then you can not use this migration
@@ -46,15 +46,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'
         );
 

src/Migrations/Version1.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     {
         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'");
             $this->skipIf(true, "Old Part-DB Database detected! Continue with upgrade...");
         } catch (DBALException $ex) {
             //when the table was not found, we can proceed, because we have an empty DB!
@@ -80,7 +80,7 @@ 
         $this->addSql('ALTER TABLE `groups` ADD CONSTRAINT FK_F06D3970727ACA70 FOREIGN KEY (parent_id) REFERENCES `groups` (id)');
 
         //Create table for user logs:
-        $sql =  $updateSteps[] = "CREATE TABLE `log` ".
+        $sql = $updateSteps[] = "CREATE TABLE `log` ".
             "( `id` INT NOT NULL AUTO_INCREMENT , `datetime` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP() ,".
             " `id_user` INT NOT NULL ,".
             " `level` TINYINT NOT NULL ,".

src/Controller/PartController.php

Spacing +5 added lines, -5 removed lines

@@ -94,7 +94,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());
             }
 
 
@@ -103,7 +103,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'));
         }
 
@@ -125,7 +125,7 @@ 
     {
         $this->denyAccessUnlessGranted('delete', $part);
 
-        if ($this->isCsrfTokenValid('delete' . $part->getId(), $request->request->get('_token'))) {
+        if ($this->isCsrfTokenValid('delete'.$part->getId(), $request->request->get('_token'))) {
             $entityManager = $this->getDoctrine()->getManager();
 
             //Remove part
@@ -169,7 +169,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);
@@ -177,7 +177,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/Form/AdminPages/ImportType.php

Spacing +2 added lines, -2 removed lines

@@ -66,12 +66,12 @@
         //Disable import if user is not allowed to create elements.
         $entity = new $data['entity_class']();
         $perm_name = "create";
-        $disabled = ! $this->security->isGranted($perm_name, $entity);
+        $disabled = !$this->security->isGranted($perm_name, $entity);
 
         $builder
 
             ->add('format', ChoiceType::class, [
-                'choices' => ['JSON' => 'json', 'XML' => 'xml','CSV' => 'csv' ,'YAML' => 'yaml'],
+                'choices' => ['JSON' => 'json', 'XML' => 'xml', 'CSV' => 'csv', 'YAML' => 'yaml'],
                 'label' => $this->trans->trans('export.format'),
                 'disabled' => $disabled])
             ->add('csv_separator', TextType::class, ['data' => ';',