Part-DB / Part-DB-symfony

src/Controller/PartController.php

Spacing +3 added lines, -3 removed lines

@@ -182,7 +182,7 @@ 
             $this->addFlash('info', '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', 'part.edited_flash.invalid');
         }
 
@@ -226,7 +226,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();
@@ -304,7 +304,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');
         }
 

src/Form/CollectionTypeExtension.php

Spacing +1 added lines, -1 removed lines

@@ -75,7 +75,7 @@
 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (FormEvent $event) use ($options): void {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(FormEvent $event) use ($options): void {
             $data = $event->getData();
             $config = $event->getForm()->getConfig();
             //If enabled do a reindexing of the collection

src/Form/AttachmentFormType.php

Spacing +3 added lines, -3 removed lines

@@ -106,7 +106,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',
             ],
@@ -134,7 +134,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',
             ],
@@ -163,7 +163,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/Form/Type/StructuralEntityType.php

Spacing +11 added lines, -11 removed lines

@@ -82,9 +82,9 @@ 
     public function buildForm(FormBuilderInterface $builder, array $options): void
     {
         $builder->addModelTransformer(new CallbackTransformer(
-                                          function ($value) use ($options) {
+                                          function($value) use ($options) {
                                               return $this->transform($value, $options);
-                                          }, function ($value) use ($options) {
+                                          }, function($value) use ($options) {
                                               return $this->reverseTransform($value, $options);
                                           }));
     }
@@ -94,21 +94,21 @@ 
         $resolver->setRequired(['class']);
         $resolver->setDefaults([
             'show_fullpath_in_subtext' => true, //When this is enabled, the full path will be shown in subtext
-            'subentities_of' => null,   //Only show entities with the given parent class
-            'disable_not_selectable' => false,  //Disable entries with not selectable property
+            'subentities_of' => null, //Only show entities with the given parent class
+            'disable_not_selectable' => false, //Disable entries with not selectable property
             'choice_value' => 'id', //Use the element id as option value and for comparing items
-            'choice_loader' => function (Options $options) {
-                return new CallbackChoiceLoader(function () use ($options) {
+            'choice_loader' => function(Options $options) {
+                return new CallbackChoiceLoader(function() use ($options) {
                     return $this->getEntries($options);
                 });
             },
-            'choice_label' => function (Options $options) {
-                return function ($choice, $key, $value) use ($options) {
+            'choice_label' => function(Options $options) {
+                return function($choice, $key, $value) use ($options) {
                     return $this->generateChoiceLabels($choice, $key, $value, $options);
                 };
             },
-            'choice_attr' => function (Options $options) {
-                return function ($choice, $key, $value) use ($options) {
+            'choice_attr' => function(Options $options) {
+                return function($choice, $key, $value) use ($options) {
                     return $this->generateChoiceAttr($choice, $key, $value, $options);
                 };
             },
@@ -117,7 +117,7 @@ 
 
         $resolver->setDefault('empty_message', null);
 
-        $resolver->setDefault('attr', function (Options $options) {
+        $resolver->setDefault('attr', function(Options $options) {
             $tmp = [
                 'class' => 'selectpicker',
                 'data-live-search' => true,

src/Form/Type/MasterPictureAttachmentType.php

Spacing +7 added lines, -7 removed lines

@@ -64,14 +64,14 @@ 
             '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'];
 
-                    if ('picture' === $options['filter'] && ! $choice->isPicture()) {
+                    if ('picture' === $options['filter'] && !$choice->isPicture()) {
                         $tmp += ['disabled' => 'disabled'];
-                    } elseif ('3d_model' === $options['filter'] && ! $choice->is3DModel()) {
+                    } elseif ('3d_model' === $options['filter'] && !$choice->is3DModel()) {
                         $tmp += ['disabled' => 'disabled'];
                     }
 
@@ -79,10 +79,10 @@ 
                 };
             },
             'choice_label' => 'name',
-            'choice_loader' => function (Options $options) {
-                return new CallbackChoiceLoader(function () use ($options) {
+            'choice_loader' => function(Options $options) {
+                return new CallbackChoiceLoader(function() use ($options) {
                     $entity = $options['entity'];
-                    if (! $entity instanceof AttachmentContainingDBElement) {
+                    if (!$entity instanceof AttachmentContainingDBElement) {
                         throw new \RuntimeException('$entity must have Attachments! (be of type AttachmentContainingDBElement)');
                     }
 

src/EventSubscriber/LogSystem/LogAccessDeniedSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -49,7 +49,7 @@
             $throwable = $throwable->getPrevious();
         }
         //Ignore everything except AccessDeniedExceptions
-        if (! $throwable instanceof AccessDeniedException) {
+        if (!$throwable instanceof AccessDeniedException) {
             return;
         }
 

src/DataTables/LogDataTable.php

Spacing +13 added lines, -13 removed lines

@@ -102,8 +102,8 @@ 
         $optionsResolver->setAllowedTypes('filter_elements', ['array', 'object']);
         $optionsResolver->setAllowedTypes('mode', 'string');
 
-        $optionsResolver->setNormalizer('filter_elements', function (Options $options, $value) {
-            if (! is_array($value)) {
+        $optionsResolver->setNormalizer('filter_elements', function(Options $options, $value) {
+            if (!is_array($value)) {
                 return [$value];
             }
 
@@ -121,7 +121,7 @@ 
 
         $dataTable->add('symbol', TextColumn::class, [
             'label' => '',
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 switch ($context->getLevelString()) {
                     case LogLevel::DEBUG:
                         $symbol = 'fa-bug';
@@ -182,7 +182,7 @@ 
         $dataTable->add('type', TextColumn::class, [
             'label' => $this->translator->trans('log.type'),
             'propertyPath' => 'type',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $this->translator->trans('log.type.'.$value);
             },
         ]);
@@ -191,14 +191,14 @@ 
             'label' => $this->translator->trans('log.level'),
             'visible' => 'system_log' === $options['mode'],
             'propertyPath' => 'levelString',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $value;
             },
         ]);
 
         $dataTable->add('user', TextColumn::class, [
             'label' => $this->translator->trans('log.user'),
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $user = $context->getUser();
 
                 return sprintf(
@@ -212,7 +212,7 @@ 
         $dataTable->add('target_type', TextColumn::class, [
             'label' => $this->translator->trans('log.target_type'),
             'visible' => false,
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $class = $context->getTargetClass();
                 if (null !== $class) {
                     return $this->elementTypeNameGenerator->getLocalizedTypeLabel($class);
@@ -234,7 +234,7 @@ 
         $dataTable->add('timeTravel', IconLinkColumn::class, [
             'label' => '',
             'icon' => 'fas fa-fw fa-eye',
-            'href' => function ($value, AbstractLogEntry $context) {
+            'href' => function($value, AbstractLogEntry $context) {
                 if (
                     ($context instanceof TimeTravelInterface
                         && $context->hasOldDataInformations())
@@ -252,10 +252,10 @@ 
 
                 return null;
             },
-            'disabled' => function ($value, AbstractLogEntry $context) {
+            'disabled' => function($value, AbstractLogEntry $context) {
                 return
-                    ! $this->security->isGranted('@tools.timetravel')
-                    || ! $this->security->isGranted('show_history', $context->getTargetClass());
+                    !$this->security->isGranted('@tools.timetravel')
+                    || !$this->security->isGranted('show_history', $context->getTargetClass());
             },
         ]);
 
@@ -267,7 +267,7 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => AbstractLogEntry::class,
-            'query' => function (QueryBuilder $builder) use ($options): void {
+            'query' => function(QueryBuilder $builder) use ($options): void {
                 $this->getQuery($builder, $options);
             },
         ]);
@@ -293,7 +293,7 @@ 
             ]);
         }
 
-        if (! empty($options['filter_elements'])) {
+        if (!empty($options['filter_elements'])) {
             foreach ($options['filter_elements'] as $element) {
                 /** @var AbstractDBElement $element */
 

src/Services/Attachments/AttachmentURLGenerator.php

Spacing +2 added lines, -2 removed lines

@@ -126,11 +126,11 @@
      */
     public function getThumbnailURL(Attachment $attachment, string $filter_name = 'thumbnail_sm'): string
     {
-        if (! $attachment->isPicture()) {
+        if (!$attachment->isPicture()) {
             throw new InvalidArgumentException('Thumbnail creation only works for picture attachments!');
         }
 
-        if ($attachment->isExternal() && ! empty($attachment->getURL())) {
+        if ($attachment->isExternal() && !empty($attachment->getURL())) {
             return $attachment->getURL();
         }
 

src/Entity/LogSystem/SecurityEventLogEntry.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
     public function setTargetElement(?AbstractDBElement $element): AbstractLogEntry
     {
-        if (! $element instanceof User) {
+        if (!$element instanceof User) {
             throw new \InvalidArgumentException('Target element must be a User object!');
         }