Part-DB / Part-DB-symfony

src/Entity/Parts/Part.php

Spacing +1 added lines, -1 removed lines

@@ -282,7 +282,7 @@
      */
     public function getIDString(): string
     {
-        return 'P' . sprintf('%06d', $this->getID());
+        return 'P'.sprintf('%06d', $this->getID());
     }
 
     /*********************************************************************************

src/Services/SIFormatter.php

Spacing +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@ 
      */
     public function getPrefixByMagnitude(int $magnitude) : array
     {
-        $prefixes_pos = ['' ,'k', 'M', 'G', 'T', 'P', 'E', 'Z', 'Y'];
-        $prefixes_neg = ['' ,'m', 'μ', 'n', 'p', 'f', 'a', 'z', 'y'];
+        $prefixes_pos = ['', 'k', 'M', 'G', 'T', 'P', 'E', 'Z', 'Y'];
+        $prefixes_neg = ['', 'm', 'μ', 'n', 'p', 'f', 'a', 'z', 'y'];
 
         //Determine nearest prefix index.
         $nearest = (int) round(abs($magnitude) / 3);
@@ -85,7 +85,7 @@ 
         [$divisor, $symbol] = $this->getPrefixByMagnitude($this->getMagnitude($value));
         $value /= $divisor;
         //Build the format string, e.g.: %.2d km
-        $format_string = '%.' . $decimals . 'f ' . $symbol . $unit;
+        $format_string = '%.'.$decimals.'f '.$symbol.$unit;
 
         return sprintf($format_string, $value);
     }

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