Part-DB / Part-DB-symfony

src/Entity/UserSystem/PermissionsEmbed.php

Spacing +4 added lines, -4 removed lines

@@ -314,7 +314,7 @@ 
      */
     public function getBitValue(string $permission_name, int $bit_n): int
     {
-        if(!$this->isValidPermissionName($permission_name)) {
+        if (!$this->isValidPermissionName($permission_name)) {
             throw new \InvalidArgumentException(sprintf('No permission with the name "%s" is existing!', $permission_name));
         }
 
@@ -358,9 +358,9 @@ 
     public function setPermissionValue(string $permission_name, int $bit_n, ?bool $new_value) : self
     {
         //Determine which bit value the given value is.
-        if($new_value === true) {
+        if ($new_value === true) {
             $bit_value = static::ALLOW;
-        } elseif($new_value === false) {
+        } elseif ($new_value === false) {
             $bit_value = static::DISALLOW;
         } else {
             $bit_value = static::INHERIT;
@@ -380,7 +380,7 @@ 
      */
     public function setBitValue(string $permission_name, int $bit_n, int $new_value) : self
     {
-        if(!$this->isValidPermissionName($permission_name)) {
+        if (!$this->isValidPermissionName($permission_name)) {
             throw new \InvalidArgumentException('No permission with the given name is existing!');
         }
 

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