Part-DB / Part-DB-symfony

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

src/Migrations/Version20190902140506.php

Spacing +5 added lines, -5 removed lines

@@ -25,7 +25,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->abortIf($version !== 26, "This database migration can only be used if the database version is 26! Install Part-DB 0.5.6 and update database there!");
         } catch (DBALException $ex) {
             //when the table was not found, then you can not use this migration
@@ -46,15 +46,15 @@ 
 
         /** Migrate the part locations for parts with known instock */
         $this->addSql(
-            'INSERT IGNORE 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 IGNORE 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 IGNORE 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 IGNORE 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/Controller/TypeaheadController.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         $normalizers = [
             new ObjectNormalizer()
         ];
-        $encoders =  [
+        $encoders = [
             new JsonEncoder()
         ];
         $serializer = new Serializer($normalizers, $encoders);

src/Services/BuiltinAttachmentsFinder.php

Spacing +3 added lines, -3 removed lines

@@ -56,7 +56,7 @@ 
     protected function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefaults([
-            'limit' => 15,  //Given only 15 entries
+            'limit' => 15, //Given only 15 entries
             //'allowed_extensions' => [], //Filter the filenames. For example ['jpg', 'jpeg'] to only get jpegs.
             //'placeholders' => Attachment::BUILTIN_PLACEHOLDER, //By default use all builtin ressources,
             'empty_returns_all' => false //Return the whole list of ressources when empty keyword is given
@@ -72,7 +72,7 @@ 
     public function getListOfRessources() : array
     {
         try {
-            return $this->cache->get('attachment_builtin_ressources', function () {
+            return $this->cache->get('attachment_builtin_ressources', function() {
                 $results = [];
 
                 $finder = new Finder();
@@ -146,7 +146,7 @@ 
         } */
 
         //Ignore case
-        $regex = '/.*' . $keyword . '.*/i';
+        $regex = '/.*'.$keyword.'.*/i';
 
         return preg_grep($regex, $base_list);
     }