Part-DB / Part-DB-symfony

src/EventSubscriber/UserSystem/PasswordChangeNeededSubscriber.php

Spacing +4 added lines, -4 removed lines

@@ -74,15 +74,15 @@ 
         $user = $this->security->getUser();
         $request = $event->getRequest();
 
-        if (! $event->isMasterRequest()) {
+        if (!$event->isMasterRequest()) {
             return;
         }
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             return;
         }
 
         //Abort if we dont need to redirect the user.
-        if (! $user->isNeedPwChange() && ! static::TFARedirectNeeded($user)) {
+        if (!$user->isNeedPwChange() && !static::TFARedirectNeeded($user)) {
             return;
         }
 
@@ -125,7 +125,7 @@ 
     {
         $tfa_enabled = $user->isU2FAuthEnabled() || $user->isGoogleAuthenticatorEnabled();
 
-        if (null !== $user->getGroup() && $user->getGroup()->isEnforce2FA() && ! $tfa_enabled) {
+        if (null !== $user->getGroup() && $user->getGroup()->isEnforce2FA() && !$tfa_enabled) {
             return true;
         }
 

src/EventSubscriber/UserSystem/RegisterU2FSubscriber.php

Spacing +2 added lines, -2 removed lines

@@ -80,9 +80,9 @@
     public function onRegister(RegisterEvent $event): void
     {
         //Skip adding of U2F key on demo mode
-        if (! $this->demo_mode) {
+        if (!$this->demo_mode) {
             $user = $event->getUser();
-            if (! $user instanceof User) {
+            if (!$user instanceof User) {
                 throw new \InvalidArgumentException('Only User objects can be registered for U2F!');
             }
 

src/EventSubscriber/LogSystem/EventLoggerSubscriber.php

Spacing +5 added lines, -5 removed lines

@@ -269,11 +269,11 @@ 
     {
         unset($fields['lastModified']);
 
-        if (! $this->hasFieldRestrictions($element)) {
+        if (!$this->hasFieldRestrictions($element)) {
             return $fields;
         }
 
-        return array_filter($fields, function ($value, $key) use ($element) {
+        return array_filter($fields, function($value, $key) use ($element) {
             //Associative array (save changed data) case
             if (is_string($key)) {
                 return $this->shouldFieldBeSaved($element, $key);
@@ -287,7 +287,7 @@ 
     {
         $uow = $em->getUnitOfWork();
 
-        if (! $logEntry instanceof ElementEditedLogEntry && ! $logEntry instanceof ElementDeletedLogEntry) {
+        if (!$logEntry instanceof ElementEditedLogEntry && !$logEntry instanceof ElementDeletedLogEntry) {
             throw new \InvalidArgumentException('$logEntry must be ElementEditedLogEntry or ElementDeletedLogEntry!');
         }
 
@@ -300,7 +300,7 @@ 
         $old_data = $this->filterFieldRestrictions($entity, $old_data);
 
         //Restrict length of string fields, to save memory...
-        $old_data = array_map(function ($value) {
+        $old_data = array_map(function($value) {
             if (is_string($value)) {
                 return mb_strimwidth($value, 0, self::MAX_STRING_LENGTH, '...');
             }
@@ -319,7 +319,7 @@ 
     protected function validEntity(object $entity): bool
     {
         //Dont log logentries itself!
-        if ($entity instanceof AbstractDBElement && ! $entity instanceof AbstractLogEntry) {
+        if ($entity instanceof AbstractDBElement && !$entity instanceof AbstractLogEntry) {
             return true;
         }
 

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/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();
         }