Part-DB / Part-DB-symfony

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/Controller/RedirectController.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
         }
 
         //$new_url = str_replace($request->getPathInfo(), '/' . $locale . $request->getPathInfo(), $request->getUri());
-        $new_url = $request->getUriForPath('/' . $locale . $request->getPathInfo());
+        $new_url = $request->getUriForPath('/'.$locale.$request->getPathInfo());
         return $this->redirect($new_url);
     }
 }
\ No newline at end of file

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

src/Migrations/Version20190924113252.php

Spacing +4 added lines, -4 removed lines

@@ -50,15 +50,15 @@
 
         //Add a attachment for each footprint attachment
         $this->addSql(
-            'INSERT INTO attachments (element_id, type_id, name, class_name, path, last_modified, datetime_added) ' .
-            "SELECT footprints.id, 1000, 'Footprint', 'Footprint',  REPLACE(footprints.filename, '%BASE%/img/footprints', '%FOOTPRINTS%' ), NOW(), NOW() FROM footprints " .
+            'INSERT INTO attachments (element_id, type_id, name, class_name, path, last_modified, datetime_added) '.
+            "SELECT footprints.id, 1000, 'Footprint', 'Footprint',  REPLACE(footprints.filename, '%BASE%/img/footprints', '%FOOTPRINTS%' ), NOW(), NOW() FROM footprints ".
             "WHERE footprints.filename <> ''"
         );
 
         //Do the same for 3D Footprints
         $this->addSql(
-            'INSERT INTO attachments (element_id, type_id, name, class_name, path, last_modified, datetime_added) ' .
-            "SELECT footprints.id, 1001, 'Footprint 3D', 'Footprint',  REPLACE(footprints.filename_3d, '%BASE%/models', '%FOOTPRINTS3D%' ), NOW(), NOW() FROM footprints " .
+            'INSERT INTO attachments (element_id, type_id, name, class_name, path, last_modified, datetime_added) '.
+            "SELECT footprints.id, 1001, 'Footprint 3D', 'Footprint',  REPLACE(footprints.filename_3d, '%BASE%/models', '%FOOTPRINTS3D%' ), NOW(), NOW() FROM footprints ".
             "WHERE footprints.filename_3d <> ''"
         );
 

src/Controller/AdminPages/BaseAdminController.php

Spacing +9 added lines, -9 removed lines

@@ -101,7 +101,7 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $this->attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $this->attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
             $em->persist($entity);
@@ -111,7 +111,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, ['attachment_class' => $this->attachment_class]);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $this->translator->trans('entity.edit_flash.invalid'));
         }
 
@@ -146,17 +146,17 @@ 
             $attachments = $form['attachments'];
             foreach ($attachments as $attachment) {
                 /** @var $attachment FormInterface */
-                $this->attachmentHelper->upload( $attachment->getData(), $attachment['file']->getData());
+                $this->attachmentHelper->upload($attachment->getData(), $attachment['file']->getData());
             }
 
             $em->persist($new_entity);
             $em->flush();
             $this->addFlash('success', $this->translator->trans('entity.created_flash'));
 
-            return $this->redirectToRoute($this->route_base . '_edit', ['id' => $new_entity->getID()]);
+            return $this->redirectToRoute($this->route_base.'_edit', ['id' => $new_entity->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', $this->translator->trans('entity.created_flash.invalid'));
         }
 
@@ -176,7 +176,7 @@ 
 
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -195,7 +195,7 @@ 
             //Show errors to user:
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -239,7 +239,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)
@@ -250,7 +250,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/Services/AttachmentHelper.php

Spacing +6 added lines, -6 removed lines

@@ -74,10 +74,10 @@ 
         if ($fs->isAbsolutePath($tmp_base_path)) {
             $this->base_path = $tmp_base_path;
         } else {
-            $this->base_path = realpath($kernel->getProjectDir() . DIRECTORY_SEPARATOR . $tmp_base_path);
+            $this->base_path = realpath($kernel->getProjectDir().DIRECTORY_SEPARATOR.$tmp_base_path);
         }
 
-        $this->footprints_path = realpath($kernel->getProjectDir() . "/public/img/footprints");
+        $this->footprints_path = realpath($kernel->getProjectDir()."/public/img/footprints");
         //TODO
         $this->footprints_3d_path = "TODO";
     }
@@ -121,7 +121,7 @@ 
         $placeholder_path = str_replace($placeholders, $targets, $placeholder_path);
 
         //Normalize path and remove ..
-        $placeholder_path = str_replace(['\\','..'], ['/',''], $placeholder_path);
+        $placeholder_path = str_replace(['\\', '..'], ['/', ''], $placeholder_path);
 
         return $placeholder_path;
     }
@@ -220,7 +220,7 @@ 
 
         $sz = 'BKMGTP';
         $factor = (int) floor((strlen($bytes) - 1) / 3);
-        return sprintf("%.{$decimals}f", $bytes / 1024 ** $factor) . @$sz[$factor];
+        return sprintf("%.{$decimals}f", $bytes / 1024 ** $factor).@$sz[$factor];
     }
 
     /**
@@ -237,7 +237,7 @@ 
             MeasurementUnitAttachment::class => 'measurement_unit', StorelocationAttachment::class => 'storelocation',
             SupplierAttachment::class => 'supplier', UserAttachment::class => 'user'];
 
-        $path = $this->base_path . DIRECTORY_SEPARATOR . $mapping[get_class($attachment)] . DIRECTORY_SEPARATOR . $attachment->getElement()->getID();
+        $path = $this->base_path.DIRECTORY_SEPARATOR.$mapping[get_class($attachment)].DIRECTORY_SEPARATOR.$attachment->getElement()->getID();
         return $path;
     }
 
@@ -258,7 +258,7 @@ 
 
         //Sanatize filename
         $originalFilename = pathinfo($file->getClientOriginalName(), PATHINFO_FILENAME);
-        $newFilename = $attachment->getName() . '-' . uniqid('', false) . '.' . $file->guessExtension();
+        $newFilename = $attachment->getName().'-'.uniqid('', false).'.'.$file->guessExtension();
 
         //Move our temporay attachment to its final location
         $file_path = $file->move($folder, $newFilename)->getRealPath();