Part-DB / Part-DB-symfony

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