Part-DB / Part-DB-symfony

src/Security/Voter/StructureVoter.php

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
      */
     protected function instanceToPermissionName($subject): ?string
     {
-        if (! is_string($subject)) {
+        if (!is_string($subject)) {
             $class_name = get_class($subject);
         } else {
             $class_name = $subject;

src/DataTables/LogDataTable.php

Spacing +11 added lines, -11 removed lines

@@ -109,7 +109,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';
@@ -170,7 +170,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);
             },
         ]);
@@ -179,14 +179,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(
@@ -200,7 +200,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);
@@ -221,7 +221,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())
@@ -239,10 +239,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());
             },
         ]);
 
@@ -254,7 +254,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);
             },
         ]);
@@ -280,7 +280,7 @@ 
             ]);
         }
 
-        if (! empty($options['filter_elements'])) {
+        if (!empty($options['filter_elements'])) {
             foreach ($options['filter_elements'] as $element) {
                 /** @var AbstractDBElement $element */
 

src/DataTables/PartsDataTable.php

Spacing +10 added lines, -10 removed lines

@@ -113,7 +113,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,
@@ -149,7 +149,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 '';
@@ -166,7 +166,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),
@@ -195,7 +195,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
@@ -214,7 +214,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());
@@ -223,7 +223,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());
                 },
             ])
@@ -287,12 +287,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(),
@@ -375,8 +375,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/DataTables/Column/RevertLogColumn.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             return '';
         }
 
-        $disabled = ! $this->security->isGranted('revert_element', $context->getTargetClass());
+        $disabled = !$this->security->isGranted('revert_element', $context->getTargetClass());
 
         $tmp = '<div class="btn-group btn-group-sm">';
         $tmp .= sprintf(

src/Controller/UserSettingsController.php

Spacing +10 added lines, -10 removed lines

@@ -84,7 +84,7 @@ 
         //When user change its settings, he should be logged  in fully.
         $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
 
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             return new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
@@ -115,7 +115,7 @@ 
         //When user change its settings, he should be logged  in fully.
         $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
 
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             throw new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
@@ -166,7 +166,7 @@ 
         //When user change its settings, he should be logged  in fully.
         $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
 
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             return new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
@@ -196,7 +196,7 @@ 
         //When user change its settings, he should be logged  in fully.
         $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
 
-        if (! $user instanceof User) {
+        if (!$user instanceof User) {
             throw new RuntimeException('This controller only works only for Part-DB User objects!');
         }
 
@@ -208,7 +208,7 @@ 
 
         $form->handleRequest($request);
 
-        if (! $this->demo_mode && $form->isSubmitted() && $form->isValid()) {
+        if (!$this->demo_mode && $form->isSubmitted() && $form->isValid()) {
             //Check if user theme setting has changed
             if ($user->getTheme() !== $em->getUnitOfWork()->getOriginalEntityData($user)['theme']) {
                 $page_need_reload = true;
@@ -268,7 +268,7 @@ 
         $pw_form->handleRequest($request);
 
         //Check if password if everything was correct, then save it to User and DB
-        if (! $this->demo_mode && $pw_form->isSubmitted() && $pw_form->isValid()) {
+        if (!$this->demo_mode && $pw_form->isSubmitted() && $pw_form->isValid()) {
             $password = $passwordEncoder->encodePassword($user, $pw_form['new_password']->getData());
             $user->setPassword($password);
 
@@ -283,14 +283,14 @@ 
         //Handle 2FA things
         $google_form = $this->createForm(TFAGoogleSettingsType::class, $user);
         $google_enabled = $user->isGoogleAuthenticatorEnabled();
-        if (! $google_enabled && ! $form->isSubmitted()) {
+        if (!$google_enabled && !$form->isSubmitted()) {
             $user->setGoogleAuthenticatorSecret($googleAuthenticator->generateSecret());
             $google_form->get('googleAuthenticatorSecret')->setData($user->getGoogleAuthenticatorSecret());
         }
         $google_form->handleRequest($request);
 
-        if (! $this->demo_mode && $google_form->isSubmitted() && $google_form->isValid()) {
-            if (! $google_enabled) {
+        if (!$this->demo_mode && $google_form->isSubmitted() && $google_form->isValid()) {
+            if (!$google_enabled) {
                 //Save 2FA settings (save secrets)
                 $user->setGoogleAuthenticatorSecret($google_form->get('googleAuthenticatorSecret')->getData());
                 $backupCodeManager->enableBackupCodes($user);
@@ -318,7 +318,7 @@ 
         ])->getForm();
 
         $backup_form->handleRequest($request);
-        if (! $this->demo_mode && $backup_form->isSubmitted() && $backup_form->isValid()) {
+        if (!$this->demo_mode && $backup_form->isSubmitted() && $backup_form->isValid()) {
             $backupCodeManager->regenerateBackupCodes($user);
             $em->flush();
             $this->addFlash('success', 'user.settings.2fa.backup_codes.regenerated');

src/Controller/UserController.php

Spacing +1 added lines, -1 removed lines

@@ -163,7 +163,7 @@
         //If no user id was passed, then we show info about the current user
         if (null === $user) {
             $tmp = $this->getUser();
-            if (! $tmp instanceof User) {
+            if (!$tmp instanceof User) {
                 throw new InvalidArgumentException('Userinfo only works for database users!');
             }
             $user = $tmp;

src/Controller/LogController.php

Spacing +1 added lines, -1 removed lines

@@ -146,7 +146,7 @@
             $this->dbRepository->changeID($element, $logEntry->getTargetID());
         }
 
-        if (! $element instanceof AbstractDBElement) {
+        if (!$element instanceof AbstractDBElement) {
             $this->addFlash('error', 'log.undo.target_not_found');
 
             return;

src/Entity/Base/AbstractStructuralDBElement.php

Spacing +3 added lines, -3 removed lines

@@ -127,7 +127,7 @@ 
 
         //Check if both elements compared, are from the same type
         // (we have to check inheritance, or we get exceptions when using doctrine entities (they have a proxy type):
-        if (! is_a($another_element, $class_name) && ! is_a($this, get_class($another_element))) {
+        if (!is_a($another_element, $class_name) && !is_a($this, get_class($another_element))) {
             throw new InvalidArgumentException('isChildOf() only works for objects of the same type!');
         }
 
@@ -243,7 +243,7 @@ 
         $tmp[] = $this;
 
         //We only allow 20 levels depth
-        while (! end($tmp)->isRoot() && count($tmp) < 20) {
+        while (!end($tmp)->isRoot() && count($tmp) < 20) {
             $tmp[] = end($tmp)->parent;
         }
 
@@ -322,7 +322,7 @@ 
      */
     public function setChildren($elements): self
     {
-        if (! is_array($elements) && ! $elements instanceof Collection) {
+        if (!is_array($elements) && !$elements instanceof Collection) {
             throw new InvalidArgumentException('$elements must be an array or Collection!');
         }
 

src/Entity/LogSystem/ElementEditedLogEntry.php

Spacing +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
 
     public function hasOldDataInformations(): bool
     {
-        return ! empty($this->extra['d']);
+        return !empty($this->extra['d']);
     }
 
     public function getOldData(): array