Part-DB / Part-DB-symfony

src/Services/GitVersionInfo.php

Spacing +3 added lines, -3 removed lines

@@ -49,8 +49,8 @@ 
      */
     public function getGitBranchName()
     {
-        if (file_exists($this->project_dir . '/.git/HEAD')) {
-            $git = file($this->project_dir . '/.git/HEAD');
+        if (file_exists($this->project_dir.'/.git/HEAD')) {
+            $git = file($this->project_dir.'/.git/HEAD');
             $head = explode('/', $git[0], 3);
             return trim($head[2]);
         }
@@ -65,7 +65,7 @@ 
      */
     public function getGitCommitHash(int $length = 7)
     {
-        $filename = $this->project_dir . '/.git/refs/remotes/origin/' . $this->getGitBranchName();
+        $filename = $this->project_dir.'/.git/refs/remotes/origin/'.$this->getGitBranchName();
         if (file_exists($filename)) {
             $head = file($filename);
             $hash = $head[0];

src/Controller/UserController.php

Spacing +2 added lines, -2 removed lines

@@ -142,7 +142,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,
@@ -168,7 +168,7 @@ 
 
         $page_need_reload = false;
 
-        if(!$user instanceof User) {
+        if (!$user instanceof User) {
             return new \RuntimeException("This controller only works only for Part-DB User objects!");
         }
 

src/Form/UserSettingsType.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
                 'required' => false,
                 'attr' => ['class' => 'selectpicker'],
                 'choices' => User::AVAILABLE_THEMES,
-                'choice_label' => function ($entity, $key, $value) {
+                'choice_label' => function($entity, $key, $value) {
                     return $value;
                 },
                 'placeholder' => $this->trans->trans('user_settings.theme.placeholder'),

src/Form/UserAdminForm.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
             ->add('theme', ChoiceType::class, [
                 'required' => false,
                 'choices' => User::AVAILABLE_THEMES,
-                'choice_label' => function ($entity, $key, $value) {
+                'choice_label' => function($entity, $key, $value) {
                     return $value;
                 },
                 'attr' => ['class' => 'selectpicker'],

src/Services/AttachmentHelper.php

Spacing +1 added lines, -1 removed lines

@@ -165,7 +165,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];
     }
 
 

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);
             }
         }
 
@@ -266,7 +266,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)
@@ -277,7 +277,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/Controller/PartController.php

Spacing +5 added lines, -5 removed lines

@@ -104,7 +104,7 @@ 
                 } catch (AttachmentDownloadException $ex) {
                     $this->addFlash(
                         'error',
-                        $translator->trans('attachment.download_failed') . ' ' . $ex->getMessage()
+                        $translator->trans('attachment.download_failed').' '.$ex->getMessage()
                     );
                 }
             }
@@ -115,7 +115,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'));
         }
 
@@ -137,7 +137,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
@@ -190,7 +190,7 @@ 
                 } catch (AttachmentDownloadException $ex) {
                     $this->addFlash(
                         'error',
-                        $translator->trans('attachment.download_failed') . ' ' . $ex->getMessage()
+                        $translator->trans('attachment.download_failed').' '.$ex->getMessage()
                     );
                 }
             }
@@ -202,7 +202,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/Form/AttachmentFormType.php

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
         //Check the secure file checkbox, if file is in securefile location
         $builder->get('secureFile')->addEventListener(
             FormEvents::PRE_SET_DATA,
-            function (FormEvent $event) {
+            function(FormEvent $event) {
                 $attachment = $event->getForm()->getParent()->getData();
                 if ($attachment instanceof Attachment) {
                     $event->setData($attachment->isSecure());

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%