Part-DB / Part-DB-symfony

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

Spacing +2 added lines, -2 removed lines

@@ -76,13 +76,13 @@
         }
 
         //$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());
 
         //If either mod_rewrite is not enabled or the index.php version is enforced, add index.php to the string
         if (($this->enforce_index_php || !$this->checkIfModRewriteAvailable())
             && strpos($new_url, 'index.php') === false) {
             //Like Request::getUriForPath only with index.php
-            $new_url = $request->getSchemeAndHttpHost(). $request->getBaseUrl().'/index.php/' . $locale . $request->getPathInfo();
+            $new_url = $request->getSchemeAndHttpHost().$request->getBaseUrl().'/index.php/'.$locale.$request->getPathInfo();
         }
         return $this->redirect($new_url);
     }

src/Form/AdminPages/BaseEntityAdminForm.php

Spacing +2 added lines, -2 removed lines

@@ -96,7 +96,7 @@ 
                 'label' =>  $this->trans->trans('entity.edit.not_selectable'),
                 'help' =>  $this->trans->trans('entity.edit.not_selectable.help'),
                 'label_attr' => ['class' => 'checkbox-custom'],
-                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity) ])
+                'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity)])
 
             ->add('comment', CKEditorType::class, ['required' => false, 'empty_data' => '',
                 'label' =>  $this->trans->trans('comment.label'),
@@ -127,7 +127,7 @@ 
 
         //Buttons
         $builder->add('save', SubmitType::class, [
-            'label' =>  $is_new ?  $this->trans->trans('entity.create') :  $this->trans->trans('entity.edit.save'),
+            'label' =>  $is_new ? $this->trans->trans('entity.create') : $this->trans->trans('entity.edit.save'),
             'attr' => ['class' => $is_new ? 'btn-success' : ''],
             'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity)])
             ->add('reset', ResetType::class, ['label' => 'entity.edit.reset',

src/Form/Type/MasterPictureAttachmentType.php

Spacing +6 added lines, -6 removed lines

@@ -51,8 +51,8 @@ 
         $resolver->setDefaults([
             'filter' => 'picture',
             'attr' => ['class' => 'selectpicker'],
-            'choice_attr' => function (Options $options) {
-                return  function ($choice, $key, $value) use ($options) {
+            'choice_attr' => function(Options $options) {
+                return  function($choice, $key, $value) use ($options) {
                     /** @var Attachment $choice */
                     $tmp = ['data-subtext' => $choice->getFilename() ?? 'URL'];
 
@@ -64,13 +64,13 @@ 
                 };
             },
             'choice_label' => 'name',
-            'class' => function (Options $options) {
+            'class' => function(Options $options) {
                 $short_class_name = (new \ReflectionClass($options['entity']))->getShortName();
                 //Category becomes CategoryAttachment
-                return 'App\\Entity\\Attachments\\' . $short_class_name . 'Attachment';
+                return 'App\\Entity\\Attachments\\'.$short_class_name.'Attachment';
             },
-            'query_builder' => function (Options $options) {
-                return function (EntityRepository $er) use ($options) {
+            'query_builder' => function(Options $options) {
+                return function(EntityRepository $er) use ($options) {
                     $entity = $options['entity'];
                     if ($entity->getID() === null) {
                         //This query is always false, so we get empty results

src/DataTables/PartsDataTable.php

Spacing +8 added lines, -8 removed lines

@@ -137,12 +137,12 @@ 
         }
 
         if (isset($options['tag'])) {
-            $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%' . $options['tag'] . '%');
+            $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%'.$options['tag'].'%');
         }
 
         if (isset($options['search'])) {
             $builder->AndWhere('part.name LIKE :search')->orWhere('part.description LIKE :search')->orWhere('part.comment LIKE :search')
-                ->setParameter('search', '%' . $options['search'] . '%');
+                ->setParameter('search', '%'.$options['search'].'%');
         }
     }
 
@@ -155,7 +155,7 @@ 
         $dataTable
             ->add('picture', TextColumn::class, [
                 'label' => '',
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $preview_attachment = $this->previewGenerator->getTablePreviewAttachment($context);
                     if ($preview_attachment === null) {
                         return '';
@@ -171,7 +171,7 @@ 
             ])
             ->add('name', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.name'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return sprintf(
                         '<a href="%s">%s</a>',
                         $this->urlGenerator->infoURL($context),
@@ -200,7 +200,7 @@ 
             ])
             ->add('storelocation', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.storeLocations'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $tmp = array();
                     foreach ($context->getPartLots() as $lot) {
                         //Ignore lots without storelocation
@@ -219,7 +219,7 @@ 
             ])
             ->add('amount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.amount'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $amount = $context->getAmountSum();
                     return $this->amountFormatter->format($amount, $context->getPartUnit());
                 }
@@ -227,7 +227,7 @@ 
             ->add('minamount', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.minamount'),
                 'visible' => false,
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return $this->amountFormatter->format($value, $context->getPartUnit());
                 }
             ])
@@ -291,7 +291,7 @@ 
                 },
                 'entity' => Part::class,
                 'criteria' => [
-                    function (QueryBuilder $builder) use ($options) {
+                    function(QueryBuilder $builder) use ($options) {
                         $this->buildCriteria($builder, $options);
                     },
                     new SearchCriteriaProvider()