Part-DB / Part-DB-symfony

src/Entity/Parts/PartLot.php

Spacing +2 added lines, -2 removed lines

@@ -115,7 +115,7 @@ 
      */
     public function getIDString(): string
     {
-        return 'PL' . $this->getID();
+        return 'PL'.$this->getID();
     }
 
     /**
@@ -268,7 +268,7 @@ 
 
     public function setAmount(float $new_amount): PartLot
     {
-        $this->amount  = $new_amount;
+        $this->amount = $new_amount;
         return $this;
     }
 

src/EntityListeners/TreeCacheInvalidationListener.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
 
         /* If any group change, then invalidate all cached trees. Users Permissions can be inherited from groups,
             so a change in any group can cause big permisssion changes for users. So to be sure, invalidate all trees */
-        if($element instanceof Group) {
+        if ($element instanceof Group) {
             $tag = "groups";
             $this->cache->invalidateTags([$tag]);
         }

src/Services/UserCacheKeyGenerator.php

Spacing +2 added lines, -2 removed lines

@@ -64,10 +64,10 @@
         //If the user is null, then treat it as anonymous user.
         //When the anonymous user is passed as user then use this path too.
         if ($user === null || $user->getID() === User::ID_ANONYMOUS) {
-            return 'user$_' . User::ID_ANONYMOUS;
+            return 'user$_'.User::ID_ANONYMOUS;
         }
 
         //In the most cases we can just use the username (its unique)
-        return "user_" . $user->getUsername();
+        return "user_".$user->getUsername();
     }
 }
\ No newline at end of file

src/Services/TreeBuilder.php

Spacing +3 added lines, -3 removed lines

@@ -102,7 +102,7 @@ 
 
         $tree_node = new TreeViewNode($element->__toString(), $href, $children_nodes);
 
-        if($children_nodes != null) {
+        if ($children_nodes != null) {
             $tree_node->addTag((string) count($children_nodes));
         }
 
@@ -177,9 +177,9 @@ 
         $parent_id = $parent != null ? $parent->getID() : "0";
         // Backslashes are not allowed in cache keys
         $secure_class_name = str_replace("\\", '_', $class_name);
-        $key = "list_" . $this->keyGenerator->generateKey() . "_" . $secure_class_name . $parent_id;
+        $key = "list_".$this->keyGenerator->generateKey()."_".$secure_class_name.$parent_id;
 
-        $ret = $this->cache->get($key, function (ItemInterface $item) use ($class_name, $parent, $secure_class_name) {
+        $ret = $this->cache->get($key, function(ItemInterface $item) use ($class_name, $parent, $secure_class_name) {
             // Invalidate when groups, a element with the class or the user changes
             $item->tag(['groups', 'tree_list', $this->keyGenerator->generateKey(), $secure_class_name]);
             /**

src/Services/ToolsTreeBuilder.php

Spacing +2 added lines, -2 removed lines

@@ -66,9 +66,9 @@
      */
     public function getTree() : array
     {
-        $key = "tree_tools_" .  $this->keyGenerator->generateKey();
+        $key = "tree_tools_".$this->keyGenerator->generateKey();
 
-        return $this->cache->get($key, function (ItemInterface $item) {
+        return $this->cache->get($key, function(ItemInterface $item) {
             //Invalidate tree, whenever group or the user changes
             $item->tag(["tree_tools", "groups", $this->keyGenerator->generateKey()]);
 

src/Migrations/Version20190812154222.php

Spacing +4 added lines, -4 removed lines

@@ -30,15 +30,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/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