Part-DB / Part-DB-symfony

src/EventSubscriber/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/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/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/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

src/Entity/LogSystem/ElementDeletedLogEntry.php

Spacing +1 added lines, -1 removed lines

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