Part-DB / Part-DB-symfony

src/Security/Voter/PartVoter.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
             if (false !== strpos($attribute, '.')) {
                 [$perm, $op] = explode('.', $attribute);
 
-                return $this->resolver->isValidOperation('parts_' . $perm, $op);
+                return $this->resolver->isValidOperation('parts_'.$perm, $op);
             }
 
             return $this->resolver->isValidOperation('parts', $attribute);

src/Security/Voter/StructureVoter.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
      */
     protected function supports($attribute, $subject)
     {
-        if(is_object($subject)) {
+        if (is_object($subject)) {
             $permission_name = $this->instanceToPermissionName($subject);
             //If permission name is null, then the subject is not supported
             return ($permission_name !== null) && $this->resolver->isValidOperation($permission_name, $attribute);

src/Form/Permissions/PermissionGroupType.php

Spacing +2 added lines, -2 removed lines

@@ -81,13 +81,13 @@
     {
         parent::configureOptions($resolver);
 
-        $resolver->setDefault('group_name', function (Options $options) {
+        $resolver->setDefault('group_name', function(Options $options) {
             return trim($options['name']);
         });
 
         $resolver->setDefault('inherit', false);
 
-        $resolver->setDefault('label', function (Options $options) {
+        $resolver->setDefault('label', function(Options $options) {
             if (!empty($this->perm_structure['groups'][$options['group_name']]['label'])) {
                 return $this->perm_structure['groups'][$options['group_name']]['label'];
             }

src/DataTables/Column/EntityColumn.php

Spacing +4 added lines, -4 removed lines

@@ -71,12 +71,12 @@
 
         $resolver->setRequired('property');
 
-        $resolver->setDefault('field', function (Options $option) {
-            return $option['property'] . '.name';
+        $resolver->setDefault('field', function(Options $option) {
+            return $option['property'].'.name';
         });
 
-        $resolver->setDefault('render', function (Options $options) {
-            return function ($value, Part $context) use ($options) {
+        $resolver->setDefault('render', function(Options $options) {
+            return function($value, Part $context) use ($options) {
                 /** @var DBElement $entity */
                 $entity = $this->accessor->getValue($context, $options['property']);
 

src/Entity/Parts/Part.php

Spacing +1 added lines, -1 removed lines

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

src/Controller/PartController.php

Spacing +5 added lines, -5 removed lines

@@ -90,7 +90,7 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
 
@@ -99,7 +99,7 @@ 
             $this->addFlash('info', $translator->trans('part.edited_flash'));
             //Reload form, so the SIUnitType entries use the new part unit
             $form = $this->createForm(PartBaseType::class, $part);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $translator->trans('part.edited_flash.invalid'));
         }
 
@@ -121,7 +121,7 @@ 
     {
         $this->denyAccessUnlessGranted('delete', $part);
 
-        if ($this->isCsrfTokenValid('delete' . $part->getId(), $request->request->get('_token'))) {
+        if ($this->isCsrfTokenValid('delete'.$part->getId(), $request->request->get('_token'))) {
             $entityManager = $this->getDoctrine()->getManager();
 
             //Remove part
@@ -165,7 +165,7 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
             $em->persist($new_part);
@@ -175,7 +175,7 @@ 
             return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $translator->trans('part.created_flash.invalid'));
         }
 

src/Controller/UserController.php

Spacing +2 added lines, -2 removed lines

@@ -140,7 +140,7 @@ 
         }
 
         //Show permissions to user
-        $builder = $this->createFormBuilder()->add('permissions',PermissionsType::class, [
+        $builder = $this->createFormBuilder()->add('permissions', PermissionsType::class, [
             'mapped' => false,
             'disabled' => true,
             'inherit' => true,
@@ -164,7 +164,7 @@ 
          */
         $user = $this->getUser();
 
-        if(!$user instanceof User) {
+        if (!$user instanceof User) {
             return new \RuntimeException("This controller only works only for Part-DB User objects!");
         }
 

src/Services/AmountFormatter.php

Spacing +7 added lines, -7 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 false;
             },
-            '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,9 +125,9 @@ 
 
         //Otherwise just output it
         if (!empty($options['unit'])) {
-            $format_string = '%.' . $options['decimals'] . 'f ' . $options['unit'];
+            $format_string = '%.'.$options['decimals'].'f '.$options['unit'];
         } else { //Dont add space after number if no unit was specified
-            $format_string = '%.' . $options['decimals'] . 'f';
+            $format_string = '%.'.$options['decimals'].'f';
         }
         return sprintf($format_string, $value);
     }

src/Services/SIFormatter.php

Spacing +4 added lines, -4 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'];
 
         if ($magnitude >= 0) {
             $nearest = (int) floor(abs($magnitude) / 3);
@@ -105,9 +105,9 @@ 
         $value /= $divisor;
         //Build the format string, e.g.: %.2d km
         if ($unit !== '' || $symbol !== '') {
-            $format_string = '%.' . $decimals . 'f ' . $symbol . $unit;
+            $format_string = '%.'.$decimals.'f '.$symbol.$unit;
         } else {
-            $format_string = '%.' . $decimals . 'f';
+            $format_string = '%.'.$decimals.'f';
         }
 
         return sprintf($format_string, $value);