Part-DB / Part-DB-symfony

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/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/Command/CleanAttachmentsCommand.php

Spacing +3 added lines, -3 removed lines

@@ -49,7 +49,7 @@ 
         $io = new SymfonyStyle($input, $output);
 
         $mediaPath = $this->pathResolver->getMediaPath();
-        $io->note("The media path is " . $mediaPath);
+        $io->note("The media path is ".$mediaPath);
 
         $finder = new Finder();
         //We look for files in the media folder only
@@ -106,8 +106,8 @@ 
      */
     protected function removeEmptySubFolders($path)
     {
-        $empty=true;
-        foreach (glob($path . DIRECTORY_SEPARATOR . "*") as $file)
+        $empty = true;
+        foreach (glob($path.DIRECTORY_SEPARATOR."*") as $file)
         {
             $empty &= is_dir($file) && $this->removeEmptySubFolders($file);
         }

src/Services/Attachments/AttachmentPathResolver.php

Spacing +3 added lines, -3 removed lines

@@ -114,7 +114,7 @@ 
         }
 
         //Otherwise prepend the project path
-        $tmp = realpath($this->project_dir . DIRECTORY_SEPARATOR . $param_path);
+        $tmp = realpath($this->project_dir.DIRECTORY_SEPARATOR.$param_path);
 
         //If path does not exist then disable the placeholder
         if ($tmp === false) {
@@ -138,7 +138,7 @@ 
 
         foreach ($array as $item) {
             $item = str_replace(['\\'], ['/'], $item);
-            $ret[] = '/' . preg_quote($item, '/') . '/';
+            $ret[] = '/'.preg_quote($item, '/').'/';
         }
 
         return $ret;
@@ -157,7 +157,7 @@ 
         //Older path entries are given via %BASE% which was the project root
 
         $count = 0;
-        $placeholder_path = preg_replace($this->placeholders_regex, $this->pathes, $placeholder_path,-1,$count);
+        $placeholder_path = preg_replace($this->placeholders_regex, $this->pathes, $placeholder_path, -1, $count);
 
         //A valid placeholder can have only one
         if ($count !== 1) {

src/Services/AttachmentHelper.php

Spacing +3 added lines, -3 removed lines

@@ -147,7 +147,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];
     }
 
     /**
@@ -164,7 +164,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;
     }
 
@@ -185,7 +185,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();

src/Migrations/Version1.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
     {
         try {
             //Check if we can use this migration method:
-            $version = (int)$this->connection->fetchColumn("SELECT keyValue AS version FROM `internal` WHERE `keyName` = 'dbVersion'");
+            $version = (int) $this->connection->fetchColumn("SELECT keyValue AS version FROM `internal` WHERE `keyName` = 'dbVersion'");
             $this->skipIf(true, "Old Part-DB Database detected! Continue with upgrade...");
         } catch (DBALException $ex) {
             //when the table was not found, we can proceed, because we have an empty DB!
@@ -85,7 +85,7 @@ 
          */
 
         //Create table for user logs:
-        $sql =  $updateSteps[] = "CREATE TABLE `log` ".
+        $sql = $updateSteps[] = "CREATE TABLE `log` ".
             "( `id` INT NOT NULL AUTO_INCREMENT , `datetime` TIMESTAMP NOT NULL DEFAULT CURRENT_TIMESTAMP() ,".
             " `id_user` INT NOT NULL ,".
             " `level` TINYINT NOT NULL ,".

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 IGNORE 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 IGNORE 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 IGNORE 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 IGNORE 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 <> ''"
         );