Part-DB / Part-DB-symfony

src/Form/AdminPages/SupplierForm.php

Spacing +2 added lines, -2 removed lines

@@ -71,7 +71,7 @@ 
             'required' => false,
             'label' => 'supplier.edit.default_currency',
             'disable_not_selectable' => true,
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
         ]);
 
         $builder->add('shipping_costs', BigDecimalMoneyType::class, [
@@ -79,7 +79,7 @@ 
             'currency' => $this->default_currency,
             'scale' => 3,
             'label' => 'supplier.shipping_costs.label',
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'move', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'move', $entity),
         ]);
     }
 }

src/Controller/SelectAPIController.php

Spacing +2 added lines, -2 removed lines

@@ -107,12 +107,12 @@
         foreach ($nodes_list as $node) {
             /** @var AbstractStructuralDBElement $node */
             $entry = [
-                'text' => str_repeat('   ', $node->getLevel()) . htmlspecialchars($node->getName()),
+                'text' => str_repeat('   ', $node->getLevel()).htmlspecialchars($node->getName()),
                 'value' => $node->getID(),
                 'data-subtext' => $node->getParent() ? $node->getParent()->getFullPath() : null,
             ];
 
-            $entries[] =  $entry;
+            $entries[] = $entry;
         }
 
         return $entries;

src/Services/Parts/PartsTableActionHandler.php

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
                     break;
 
                 default:
-                    throw new \InvalidArgumentException('The given action is unknown! (' . $action . ')');
+                    throw new \InvalidArgumentException('The given action is unknown! ('.$action.')');
             }
         }
     }

src/DataTables/AttachmentDataTable.php

Spacing +8 added lines, -8 removed lines

@@ -79,14 +79,14 @@ 
     {
         $dataTable->add('picture', TextColumn::class, [
             'label' => '',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 if ($context->isPicture()
-                    && ! $context->isExternal()
+                    && !$context->isExternal()
                     && $this->attachmentHelper->isFileExisting($context)) {
 
                     $title = htmlspecialchars($context->getName());
                     if ($context->getFilename()) {
-                        $title .= ' (' . htmlspecialchars($context->getFilename()) . ')';
+                        $title .= ' ('.htmlspecialchars($context->getFilename()).')';
                     }
 
                     return sprintf(
@@ -105,7 +105,7 @@ 
 
         $dataTable->add('name', TextColumn::class, [
             'label' => 'attachment.edit.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 //Link to external source
                 if ($context->isExternal()) {
                     return sprintf(
@@ -130,7 +130,7 @@ 
         $dataTable->add('attachment_type', TextColumn::class, [
             'label' => 'attachment.table.type',
             'field' => 'attachment_type.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 return sprintf(
                     '<a href="%s">%s</a>',
                     $this->entityURLGenerator->editURL($context->getAttachmentType()),
@@ -142,7 +142,7 @@ 
         $dataTable->add('element', TextColumn::class, [
             'label' => 'attachment.table.element',
             //'propertyPath' => 'element.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 return sprintf(
                     '<a href="%s">%s</a>',
                     $this->entityURLGenerator->infoURL($context->getElement()),
@@ -158,7 +158,7 @@ 
 
         $dataTable->add('filesize', TextColumn::class, [
             'label' => $this->translator->trans('attachment.table.filesize'),
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 if ($this->attachmentHelper->isFileExisting($context)) {
                     return $this->attachmentHelper->getHumanFileSize($context);
                 }
@@ -222,7 +222,7 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => Attachment::class,
-            'query' => function (QueryBuilder $builder): void {
+            'query' => function(QueryBuilder $builder): void {
                 $this->getQuery($builder);
             },
         ]);

src/DataTables/PartsDataTable.php

Spacing +13 added lines, -13 removed lines

@@ -119,7 +119,7 @@ 
         $optionsResolver->setAllowedTypes('search', ['null', 'string']);
 
         //Configure search options
-        $optionsResolver->setDefault('search_options', function (OptionsResolver $resolver): void {
+        $optionsResolver->setDefault('search_options', function(OptionsResolver $resolver): void {
             $resolver->setDefaults([
                 'name' => true,
                 'category' => true,
@@ -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 (null === $preview_attachment) {
                         return '';
@@ -163,7 +163,7 @@ 
 
                     $title = htmlspecialchars($preview_attachment->getName());
                     if ($preview_attachment->getFilename()) {
-                        $title .= ' (' . htmlspecialchars($preview_attachment->getFilename()) . ')';
+                        $title .= ' ('.htmlspecialchars($preview_attachment->getFilename()).')';
                     }
 
                     return sprintf(
@@ -178,7 +178,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),
@@ -207,7 +207,7 @@ 
             ])
             ->add('storelocation', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.storeLocations'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     $tmp = [];
                     foreach ($context->getPartLots() as $lot) {
                         //Ignore lots without storelocation
@@ -226,7 +226,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());
@@ -235,7 +235,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());
                 },
             ])
@@ -298,10 +298,10 @@ 
             ->add('edit', IconLinkColumn::class, [
                 'label' => $this->translator->trans('part.table.edit'),
                 'visible' => false,
-                'href' => function ($value, Part $context) {
+                'href' => function($value, Part $context) {
                     return $this->urlGenerator->editURL($context);
                 },
-                'disabled' => function ($value, Part $context) {
+                'disabled' => function($value, Part $context) {
                     return !$this->security->isGranted('edit', $context);
                 },
                 'title' => $this->translator->trans('part.table.edit.title'),
@@ -310,12 +310,12 @@ 
             ->addOrderBy('name')
             ->createAdapter(FetchJoinORMAdapter::class, [
                 'simple_total_query' => true,
-                'query' => function (QueryBuilder $builder): void {
+                'query' => function(QueryBuilder $builder): void {
                     $this->getQuery($builder);
                 },
                 'entity' => Part::class,
                 'criteria' => [
-                    function (QueryBuilder $builder) use ($options): void {
+                    function(QueryBuilder $builder) use ($options): void {
                         $this->buildCriteria($builder, $options);
                     },
                     new SearchCriteriaProvider(),
@@ -396,8 +396,8 @@ 
             $builder->andWhere('part.tags LIKE :tag')->setParameter('tag', '%'.$options['tag'].'%');
         }
 
-        if (! empty($options['search'])) {
-            if (! $options['search_options']['regex']) {
+        if (!empty($options['search'])) {
+            if (!$options['search_options']['regex']) {
                 //Dont show results, if no things are selected
                 $builder->andWhere('0=1');
                 $defined = false;

src/Controller/PartController.php

Spacing +3 added lines, -3 removed lines

@@ -188,7 +188,7 @@ 
 
             //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', 'part.edited_flash.invalid');
         }
 
@@ -232,7 +232,7 @@ 
      * @return Response
      */
     public function new(Request $request, EntityManagerInterface $em, TranslatorInterface $translator,
-        AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null): Response
+        AttachmentSubmitHandler $attachmentSubmitHandler, ?Part $part = null) : Response
     {
         if (null === $part) {
             $new_part = new Part();
@@ -315,7 +315,7 @@ 
             return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'part.created_flash.invalid');
         }
 

tests/Services/Attachments/AttachmentPathResolverTest.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
         $this->assertSame(self::$projectDir_orig, self::$service->parameterToAbsolutePath(self::$projectDir));
 
         //Relative pathes should be resolved
-        $expected = str_replace('\\', '/',self::$projectDir_orig.DIRECTORY_SEPARATOR.'src');
+        $expected = str_replace('\\', '/', self::$projectDir_orig.DIRECTORY_SEPARATOR.'src');
         $this->assertSame($expected, self::$service->parameterToAbsolutePath('src'));
         $this->assertSame($expected, self::$service->parameterToAbsolutePath('./src'));
 

src/Form/AttachmentFormType.php

Spacing +4 added lines, -4 removed lines

@@ -115,7 +115,7 @@ 
             'required' => false,
             'label' => 'attachment.edit.secure_file',
             'mapped' => false,
-            'disabled' => ! $this->security->isGranted('@parts_attachments.show_private'),
+            'disabled' => !$this->security->isGranted('@parts_attachments.show_private'),
             'attr' => [
                 'class' => 'form-control-sm',
             ],
@@ -143,7 +143,7 @@ 
             'required' => false,
             'label' => 'attachment.edit.download_url',
             'mapped' => false,
-            'disabled' => ! $this->allow_attachments_download,
+            'disabled' => !$this->allow_attachments_download,
             'attr' => [
                 'class' => 'form-control-sm',
             ],
@@ -169,7 +169,7 @@ 
             ],
         ]);
 
-        $builder->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event): void {
+        $builder->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event): void {
             $form = $event->getForm();
             $attachment = $form->getData();
 
@@ -188,7 +188,7 @@ 
         //Check the secure file checkbox, if file is in securefile location
         $builder->get('secureFile')->addEventListener(
             FormEvents::PRE_SET_DATA,
-            function (FormEvent $event): void {
+            function(FormEvent $event): void {
                 $attachment = $event->getForm()->getParent()->getData();
                 if ($attachment instanceof Attachment) {
                     $event->setData($attachment->isSecure());

src/Command/UpdateExchangeRatesCommand.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
         $iso_code = $input->getArgument('iso_code');
         $repo = $this->em->getRepository(Currency::class);
 
-        if (! empty($iso_code)) {
+        if (!empty($iso_code)) {
             $candidates = $repo->findBy(['iso_code' => $iso_code]);
         } else {
             $candidates = $repo->findAll();