Part-DB / Part-DB-symfony

src/Controller/GroupController.php

Spacing +1 added lines, -1 removed lines

@@ -86,7 +86,7 @@
      *
      * @return Response
      */
-    public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Group $entity = null): Response
+    public function new(Request $request, EntityManagerInterface $em, EntityImporter $importer, ?Group $entity = null) : Response
     {
         return $this->_new($request, $em, $importer, $entity);
     }

src/EventSubscriber/UserSystem/SetUserTimezoneSubscriber.php

Spacing +2 added lines, -2 removed lines

@@ -50,12 +50,12 @@
 
         //Check if the user has set a timezone
         $user = $this->security->getUser();
-        if ($user instanceof User && ! empty($user->getTimezone())) {
+        if ($user instanceof User && !empty($user->getTimezone())) {
             $timezone = $user->getTimezone();
         }
 
         //Fill with default value if needed
-        if (null === $timezone && ! empty($this->default_timezone)) {
+        if (null === $timezone && !empty($this->default_timezone)) {
             $timezone = $this->default_timezone;
         }
 

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/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,