Part-DB / Part-DB-symfony

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/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/Form/Type/StructuralEntityType.php

Spacing +9 added lines, -9 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,16 +86,16 @@ 
         $resolver->setRequired(['class']);
         $resolver->setDefaults(['attr' => ['class' => 'selectpicker', 'data-live-search' => true],
             '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);
             }
         ]);
@@ -130,7 +130,7 @@ 
 
 
         $tmp = str_repeat('   ', $choice->getLevel()); //Use 3 spaces for intendation
-        $tmp .=  htmlspecialchars($choice->getName($parent));
+        $tmp .= htmlspecialchars($choice->getName($parent));
         return $tmp;
     }
 

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)