Part-DB / Part-DB-symfony

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!');
         }
 

src/Entity/UserSystem/User.php

Spacing +2 added lines, -2 removed lines

@@ -508,7 +508,7 @@ 
     {
         $tmp = $this->getFirstName();
         //Dont add a space, if the name has only one part (it would look strange)
-        if (! empty($this->getFirstName()) && ! empty($this->getLastName())) {
+        if (!empty($this->getFirstName()) && !empty($this->getLastName())) {
             $tmp .= ' ';
         }
         $tmp .= $this->getLastName();
@@ -530,7 +530,7 @@ 
     public function setName(string $new_name): AbstractNamedDBElement
     {
         // Anonymous user is not allowed to change its username
-        if (! $this->isAnonymousUser()) {
+        if (!$this->isAnonymousUser()) {
             $this->name = $new_name;
         }
 

src/Controller/AdminPages/LabelProfileController.php

Spacing +1 added lines, -1 removed lines

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

src/Services/LabelSystem/LabelTextReplacer.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
     public function replace(string $lines, object $target): string
     {
         $patterns = [
-            '/(\[\[[A-Z_]+\]\])/' => function ($match) use ($target) {
+            '/(\[\[[A-Z_]+\]\])/' => function($match) use ($target) {
                 return $this->handlePlaceholder($match[0], $target);
             },
         ];

src/Form/UserAdminForm.php

Spacing +16 added lines, -16 removed lines

@@ -95,7 +95,7 @@ 
                 'attr' => [
                     'placeholder' => 'user.username.placeholder',
                 ],
-                'disabled' => ! $this->security->isGranted('edit_username', $entity),
+                'disabled' => !$this->security->isGranted('edit_username', $entity),
             ])
 
             ->add('group', StructuralEntityType::class, [
@@ -103,7 +103,7 @@ 
                 'required' => false,
                 'label' => 'group.label',
                 'disable_not_selectable' => true,
-                'disabled' => ! $this->security->isGranted('change_group', $entity),
+                'disabled' => !$this->security->isGranted('change_group', $entity),
             ])
 
             ->add('first_name', TextType::class, [
@@ -113,7 +113,7 @@ 
                     'placeholder' => 'user.firstName.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('last_name', TextType::class, [
@@ -123,7 +123,7 @@ 
                     'placeholder' => 'user.lastName.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('email', TextType::class, [
@@ -133,7 +133,7 @@ 
                     'placeholder' => 'user.email.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             ->add('department', TextType::class, [
@@ -143,7 +143,7 @@ 
                     'placeholder' => 'user.department.placeholder',
                 ],
                 'required' => false,
-                'disabled' => ! $this->security->isGranted('edit_infos', $entity),
+                'disabled' => !$this->security->isGranted('edit_infos', $entity),
             ])
 
             //Config section
@@ -156,7 +156,7 @@ 
                 'placeholder' => 'user_settings.language.placeholder',
                 'label' => 'user.language_select',
                 'preferred_choices' => ['en', 'de'],
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->add('timezone', TimezoneType::class, [
                 'required' => false,
@@ -167,12 +167,12 @@ 
                 'placeholder' => 'user_settings.timezone.placeholder',
                 'label' => 'user.timezone.label',
                 'preferred_choices' => ['Europe/Berlin'],
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->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' => [
@@ -181,12 +181,12 @@ 
                 'choice_translation_domain' => false,
                 'placeholder' => 'user_settings.theme.placeholder',
                 'label' => 'user.theme.label',
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
             ->add('currency', CurrencyEntityType::class, [
                 'required' => false,
                 'label' => 'user.currency.label',
-                'disabled' => ! $this->security->isGranted('change_user_settings', $entity),
+                'disabled' => !$this->security->isGranted('change_user_settings', $entity),
             ])
 
             ->add('new_password', RepeatedType::class, [
@@ -200,7 +200,7 @@ 
                 'invalid_message' => 'password_must_match',
                 'required' => false,
                 'mapped' => false,
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
+                'disabled' => !$this->security->isGranted('set_password', $entity),
                 'constraints' => [new Length([
                     'min' => 6,
                     'max' => 128,
@@ -213,7 +213,7 @@ 
                     'class' => 'checkbox-custom',
                 ],
                 'label' => 'user.edit.needs_pw_change',
-                'disabled' => ! $this->security->isGranted('set_password', $entity),
+                'disabled' => !$this->security->isGranted('set_password', $entity),
             ])
 
             ->add('disabled', CheckboxType::class, [
@@ -222,7 +222,7 @@ 
                     'class' => 'checkbox-custom',
                 ],
                 'label' => 'user.edit.user_disabled',
-                'disabled' => ! $this->security->isGranted('set_password', $entity)
+                'disabled' => !$this->security->isGranted('set_password', $entity)
                     || $entity === $this->security->getUser(),
             ])
 
@@ -230,7 +230,7 @@ 
             ->add('permissions', PermissionsType::class, [
                 'mapped' => false,
                 'data' => $builder->getData(),
-                'disabled' => ! $this->security->isGranted('edit_permissions', $entity),
+                'disabled' => !$this->security->isGranted('edit_permissions', $entity),
             ])
         ;
         /*->add('comment', CKEditorType::class, ['required' => false,
@@ -250,7 +250,7 @@ 
                 'data_class' => $options['attachment_class'],
             ],
             'by_reference' => false,
-            'disabled' => ! $this->security->isGranted($is_new ? 'create' : 'edit', $entity),
+            'disabled' => !$this->security->isGranted($is_new ? 'create' : 'edit', $entity),
         ]);
 
         $builder->add('log_comment', TextType::class, [

tests/Controller/AdminPages/LabelProfileControllerTest.php

Spacing +1 added lines, -1 removed lines

@@ -55,6 +55,6 @@
 
         //Page is redirected to '/new', when delete was successful
         $this->assertSame($delete, $client->getResponse()->isRedirect(static::$base_path.'/new'));
-        $this->assertSame($delete, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($delete, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 }