Part-DB / Part-DB-symfony

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/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' => ';',

src/Form/AdminPages/MassCreationForm.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         //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('lines', TextareaType::class, ['data' => '',

src/Form/AdminPages/BaseEntityAdminForm.php

Spacing +2 added lines, -2 removed lines

@@ -83,7 +83,7 @@ 
                 'label' =>  $this->trans->trans('entity.edit.not_selectable'),
                 'help' =>  $this->trans->trans('entity.edit.not_selectable.help'),
                 'label_attr' => ['class' => 'checkbox-custom'],
-                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity) ])
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity)])
 
             ->add('comment', CKEditorType::class, ['required' => false, 'empty_data' => '',
                 'label' =>  $this->trans->trans('comment.label'),
@@ -94,7 +94,7 @@ 
 
             //Buttons
             $builder->add('save', SubmitType::class, [
-                'label' =>  $is_new ?  $this->trans->trans('entity.create') :  $this->trans->trans('entity.edit.save'),
+                'label' =>  $is_new ? $this->trans->trans('entity.create') : $this->trans->trans('entity.edit.save'),
                 'attr' => ['class' => $is_new ? 'btn-success' : ''],
                 'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity)])
             ->add('reset', ResetType::class, ['label' => 'entity.edit.reset',

src/Form/Part/PartBaseType.php

Spacing +2 added lines, -2 removed lines

@@ -217,12 +217,12 @@
             'label' => $this->trans->trans('part.edit.master_attachment'),
             'class' => PartAttachment::class,
             'attr' => ['class' => 'selectpicker'],
-            'choice_attr' => function ($choice, $key, $value) {
+            'choice_attr' => function($choice, $key, $value) {
                 /** @var Attachment $choice */
                 return ['data-subtext' => $choice->getFilename() ?? "URL"];
             },
             'choice_label' => 'name',
-            'query_builder' => function (EntityRepository $er) use ($part) {
+            'query_builder' => function(EntityRepository $er) use ($part) {
                 if ($part->getID() == null) {
                     //This query is always false, so we get empty results
                     return $er->createQueryBuilder('u')->where('0 = 2');

src/DataTables/PartsDataTable.php

Spacing +7 added lines, -7 removed lines

@@ -133,12 +133,12 @@ 
         }
 
         if (isset($options['tag'])) {
-            $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%' . $options['tag'] . '%');
+            $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%'.$options['tag'].'%');
         }
 
         if (isset($options['search'])) {
             $builder->AndWhere('part.name LIKE :search')->orWhere('part.description LIKE :search')->orWhere('part.comment LIKE :search')
-                ->setParameter('search', '%' . $options['search'] . '%');
+                ->setParameter('search', '%'.$options['search'].'%');
         }
     }
 
@@ -151,7 +151,7 @@ 
         $dataTable
             ->add('name', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.name'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return sprintf(
                         '<a href="%s">%s</a>',
                         $this->urlGenerator->infoURL($context),
@@ -180,7 +180,7 @@ 
             ])
             ->add('storelocation', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.storeLocations'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $tmp = array();
                     foreach ($context->getPartLots() as $lot) {
                         //Ignore lots without storelocation
@@ -199,7 +199,7 @@ 
             ])
             ->add('amount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.amount'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $amount = $context->getAmountSum();
                     return $this->amountFormatter->format($amount, $context->getPartUnit());
                 }
@@ -207,7 +207,7 @@ 
             ->add('minamount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.minamount'),
                 'visible' => false,
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return $this->amountFormatter->format($value, $context->getPartUnit());
                 }
             ])
@@ -271,7 +271,7 @@ 
                 },
                 'entity' => Part::class,
                 'criteria' => [
-                    function (QueryBuilder $builder) use ($options) {
+                    function(QueryBuilder $builder) use ($options) {
                         $this->buildCriteria($builder, $options);
                     },
                     new SearchCriteriaProvider()