Part-DB / Part-DB-symfony

src/Services/Attachments/AttachmentSubmitHandler.php

Spacing +9 added lines, -9 removed lines

@@ -72,7 +72,7 @@ 
     protected $httpClient;
     protected $mimeTypes;
 
-    public function __construct (AttachmentPathResolver $pathResolver, bool $allow_attachments_downloads,
+    public function __construct(AttachmentPathResolver $pathResolver, bool $allow_attachments_downloads,
                                 HttpClientInterface $httpClient, MimeTypesInterface $mimeTypes)
     {
         $this->pathResolver = $pathResolver;
@@ -121,7 +121,7 @@ 
             $attachment->getName()
         );
 
-        return $safeName . '-' . uniqid('', false) . '.' . $extension;
+        return $safeName.'-'.uniqid('', false).'.'.$extension;
     }
 
     /**
@@ -141,7 +141,7 @@ 
         //Ensure the given attachment class is known to mapping
         if (!isset($this->folder_mapping[get_class($attachment)])) {
             throw new \InvalidArgumentException(
-                'The given attachment class is not known! The passed class was: ' . get_class($attachment)
+                'The given attachment class is not known! The passed class was: '.get_class($attachment)
             );
         }
         //Ensure the attachment has an assigned element
@@ -153,8 +153,8 @@ 
 
         //Build path
         return
-            $base_path . DIRECTORY_SEPARATOR //Base path
-            . $this->folder_mapping[get_class($attachment)] . DIRECTORY_SEPARATOR . $attachment->getElement()->getID();
+            $base_path.DIRECTORY_SEPARATOR //Base path
+            . $this->folder_mapping[get_class($attachment)].DIRECTORY_SEPARATOR.$attachment->getElement()->getID();
     }
 
     /**
@@ -227,7 +227,7 @@ 
 
         $ext = pathinfo($filename, PATHINFO_EXTENSION);
         $new_path = $this->generateAttachmentPath($attachment, $secure_location)
-            . DIRECTORY_SEPARATOR . $this->generateAttachmentFilename($attachment, $ext);
+            . DIRECTORY_SEPARATOR.$this->generateAttachmentFilename($attachment, $ext);
 
         //Move file to new directory
         $fs = new Filesystem();
@@ -257,7 +257,7 @@ 
 
         $fs = new Filesystem();
         $attachment_folder = $this->generateAttachmentPath($attachment, $options['secure_attachment']);
-        $tmp_path = $attachment_folder . DIRECTORY_SEPARATOR . $this->generateAttachmentFilename($attachment, 'tmp');
+        $tmp_path = $attachment_folder.DIRECTORY_SEPARATOR.$this->generateAttachmentFilename($attachment, 'tmp');
 
         try {
             $response = $this->httpClient->request('GET', $url, [
@@ -265,7 +265,7 @@ 
             ]);
 
             if (200 !== $response->getStatusCode()) {
-                throw new AttachmentDownloadException('Status code: ' . $response->getStatusCode());
+                throw new AttachmentDownloadException('Status code: '.$response->getStatusCode());
             }
 
             //Open a temporary file in the attachment folder
@@ -306,7 +306,7 @@ 
             }
 
             //Rename the file to its new name and save path to attachment entity
-            $new_path = $attachment_folder . DIRECTORY_SEPARATOR . $this->generateAttachmentFilename($attachment, $new_ext);
+            $new_path = $attachment_folder.DIRECTORY_SEPARATOR.$this->generateAttachmentFilename($attachment, $new_ext);
             $fs->rename($tmp_path, $new_path);
 
             //Make our file path relative to %BASE%

src/Command/CleanAttachmentsCommand.php

Spacing +4 added lines, -4 removed lines

@@ -49,9 +49,9 @@ 
         $io = new SymfonyStyle($input, $output);
 
         $mediaPath = $this->pathResolver->getMediaPath();
-        $io->note("The media path is " . $mediaPath);
+        $io->note("The media path is ".$mediaPath);
         $securePath = $this->pathResolver->getSecurePath();
-        $io->note("The secure media path is ". $securePath);
+        $io->note("The secure media path is ".$securePath);
 
         $finder = new Finder();
         //We look for files in the media folder only
@@ -110,8 +110,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/Controller/HomepageController.php

Spacing +1 added lines, -1 removed lines

@@ -55,7 +55,7 @@
         $banner = $this->getParameter('banner');
         if (empty($banner)) {
             $banner_path = $this->kernel->getProjectDir()
-                . DIRECTORY_SEPARATOR . 'config' . DIRECTORY_SEPARATOR . 'banner.md';
+                . DIRECTORY_SEPARATOR.'config'.DIRECTORY_SEPARATOR.'banner.md';
 
             return file_get_contents($banner_path);
         }

src/Services/Attachments/AttachmentManager.php

Spacing +1 added lines, -1 removed lines

@@ -173,6 +173,6 @@
 
         $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];
     }
 }
\ No newline at end of file

src/Services/Attachments/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);
     }

src/Helpers/UTCDateTimeType.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
             self::$utc_timezone
         );
 
-        if (! $converted) {
+        if (!$converted) {
             throw ConversionException::conversionFailedFormat(
                 $value,
                 $this->getName(),

src/Controller/AdminPages/BaseAdminController.php

Spacing +9 added lines, -9 removed lines

@@ -115,7 +115,7 @@ 
                 } catch (AttachmentDownloadException $ex) {
                     $this->addFlash(
                         'error',
-                        $this->translator->trans('attachment.download_failed') . ' ' . $ex->getMessage()
+                        $this->translator->trans('attachment.download_failed').' '.$ex->getMessage()
                     );
                 }
             }
@@ -127,7 +127,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'));
         }
 
@@ -171,7 +171,7 @@ 
                 } catch (AttachmentDownloadException $ex) {
                     $this->addFlash(
                         'error',
-                        $this->translator->trans('attachment.download_failed') . ' ' . $ex->getMessage()
+                        $this->translator->trans('attachment.download_failed').' '.$ex->getMessage()
                     );
                 }
             }
@@ -180,10 +180,10 @@ 
             $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'));
         }
 
@@ -203,7 +203,7 @@ 
 
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -222,7 +222,7 @@ 
             //Show errors to user:
             foreach ($errors as $name => $error) {
                 /** @var $error ConstraintViolationList */
-                $this->addFlash('error', $name . ':' . $error);
+                $this->addFlash('error', $name.':'.$error);
             }
         }
 
@@ -268,7 +268,7 @@ 
             $this->addFlash('error', 'csfr_invalid');
         }
 
-        return $this->redirectToRoute($this->route_base .  '_new');
+        return $this->redirectToRoute($this->route_base.'_new');
     }
 
     protected function _exportAll(EntityManagerInterface $em, EntityExporter $exporter, Request $request)
@@ -279,7 +279,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/DataFixtures/GroupFixtures.php

Spacing +24 added lines, -24 removed lines

@@ -18,14 +18,14 @@ 
         $admins->setName('admins');
         //Perm values taken from Version 1
         $admins->getPermissions()->setRawPermissionValues([
-            'system' => '21','groups' => '1365','users' => '87381','self' => '85','config' => '85',
-            'database' => '21','parts' => '1431655765','parts_name' => '5','parts_description' => '5',
-            'parts_footprint' => '5','parts_manufacturer' => '5','parts_comment' => '5','parts_order' => '5',
-            'parts_orderdetails' => '341','parts_prices' => '341','parts_attachments' => '341','devices' => '5461',
-            'devices_parts' => '325','storelocations' => '5461','footprints' => '5461','categories' => '5461',
-            'suppliers' => '5461','manufacturers' => '5461','attachment_types' => '1365','tools' => '1365',
-            'labels' => '21','parts_category' => '5','parts_minamount' => '5','parts_lots' => '85','parts_tags' => '5',
-            'parts_unit' => '5','parts_mass' => '5','parts_status' => '5','parts_mpn' => '5','currencies' => '5461',
+            'system' => '21', 'groups' => '1365', 'users' => '87381', 'self' => '85', 'config' => '85',
+            'database' => '21', 'parts' => '1431655765', 'parts_name' => '5', 'parts_description' => '5',
+            'parts_footprint' => '5', 'parts_manufacturer' => '5', 'parts_comment' => '5', 'parts_order' => '5',
+            'parts_orderdetails' => '341', 'parts_prices' => '341', 'parts_attachments' => '341', 'devices' => '5461',
+            'devices_parts' => '325', 'storelocations' => '5461', 'footprints' => '5461', 'categories' => '5461',
+            'suppliers' => '5461', 'manufacturers' => '5461', 'attachment_types' => '1365', 'tools' => '1365',
+            'labels' => '21', 'parts_category' => '5', 'parts_minamount' => '5', 'parts_lots' => '85', 'parts_tags' => '5',
+            'parts_unit' => '5', 'parts_mass' => '5', 'parts_status' => '5', 'parts_mpn' => '5', 'currencies' => '5461',
             'measurement_units' => '5461'
         ]);
         $this->setReference(self::ADMINS, $admins);
@@ -34,14 +34,14 @@ 
         $readonly = new Group();
         $readonly->setName('readonly');
         $readonly->getPermissions()->setRawPermissionValues([
-            'system' => '2','groups' => '2730','users' => '43690','self' => '25','config' => '170',
-            'database' => '42','parts' => '2778027689','parts_name' => '9','parts_description' => '9',
-            'parts_footprint' => '9','parts_manufacturer' => '9','parts_comment' => '9','parts_order' => '9',
-            'parts_orderdetails' => '681','parts_prices' => '681','parts_attachments' => '681','devices' => '1705',
-            'devices_parts' => '649','storelocations' => '1705','footprints' => '1705','categories' => '1705',
-            'suppliers' => '1705','manufacturers' => '1705','attachment_types' => '681','tools' => '1366',
-            'labels' => '165','parts_category' => '9','parts_minamount' => '9','parts_lots' => '169','parts_tags' => '9',
-            'parts_unit' => '9','parts_mass' => '9','parts_status' => '9','parts_mpn' => '9','currencies' => '9897',
+            'system' => '2', 'groups' => '2730', 'users' => '43690', 'self' => '25', 'config' => '170',
+            'database' => '42', 'parts' => '2778027689', 'parts_name' => '9', 'parts_description' => '9',
+            'parts_footprint' => '9', 'parts_manufacturer' => '9', 'parts_comment' => '9', 'parts_order' => '9',
+            'parts_orderdetails' => '681', 'parts_prices' => '681', 'parts_attachments' => '681', 'devices' => '1705',
+            'devices_parts' => '649', 'storelocations' => '1705', 'footprints' => '1705', 'categories' => '1705',
+            'suppliers' => '1705', 'manufacturers' => '1705', 'attachment_types' => '681', 'tools' => '1366',
+            'labels' => '165', 'parts_category' => '9', 'parts_minamount' => '9', 'parts_lots' => '169', 'parts_tags' => '9',
+            'parts_unit' => '9', 'parts_mass' => '9', 'parts_status' => '9', 'parts_mpn' => '9', 'currencies' => '9897',
             'measurement_units' => '9897'
         ]);
         $this->setReference(self::READONLY, $readonly);
@@ -50,14 +50,14 @@ 
         $users = new Group();
         $users->setName('users');
         $users->getPermissions()->setRawPermissionValues([
-            'system' => '42','groups' => '2730','users' => '43690','self' => '89','config' => '105',
-            'database' => '41','parts' => '1431655765','parts_name' => '5','parts_description' => '5',
-            'parts_footprint' => '5','parts_manufacturer' => '5','parts_comment' => '5','parts_order' => '5',
-            'parts_orderdetails' => '341','parts_prices' => '341','parts_attachments' => '341','devices' => '5461',
-            'devices_parts' => '325','storelocations' => '5461','footprints' => '5461','categories' => '5461',
-            'suppliers' => '5461','manufacturers' => '5461','attachment_types' => '1365','tools' => '1365',
-            'labels' => '85','parts_category' => '5','parts_minamount' => '5','parts_lots' => '85','parts_tags' => '5',
-            'parts_unit' => '5','parts_mass' => '5','parts_status' => '5','parts_mpn' => '5','currencies' => '5461',
+            'system' => '42', 'groups' => '2730', 'users' => '43690', 'self' => '89', 'config' => '105',
+            'database' => '41', 'parts' => '1431655765', 'parts_name' => '5', 'parts_description' => '5',
+            'parts_footprint' => '5', 'parts_manufacturer' => '5', 'parts_comment' => '5', 'parts_order' => '5',
+            'parts_orderdetails' => '341', 'parts_prices' => '341', 'parts_attachments' => '341', 'devices' => '5461',
+            'devices_parts' => '325', 'storelocations' => '5461', 'footprints' => '5461', 'categories' => '5461',
+            'suppliers' => '5461', 'manufacturers' => '5461', 'attachment_types' => '1365', 'tools' => '1365',
+            'labels' => '85', 'parts_category' => '5', 'parts_minamount' => '5', 'parts_lots' => '85', 'parts_tags' => '5',
+            'parts_unit' => '5', 'parts_mass' => '5', 'parts_status' => '5', 'parts_mpn' => '5', 'currencies' => '5461',
             'measurement_units' => '5461'
         ]);
         $this->setReference(self::USERS, $users);

src/Entity/UserSystem/User.php

Spacing +1 added lines, -1 removed lines

@@ -588,6 +588,6 @@
     public function __toString()
     {
         $tmp = $this->isDisabled() ? ' [DISABLED]' : '';
-        return $this->getFullName(true) . $tmp;
+        return $this->getFullName(true).$tmp;
     }
 }