Part-DB / Part-DB-symfony

src/Entity/Parts/PartLot.php

Spacing +1 added lines, -1 removed lines

@@ -269,7 +269,7 @@
      */
     public function getAmount(): float
     {
-        if ($this->part instanceof Part && ! $this->part->useFloatAmount()) {
+        if ($this->part instanceof Part && !$this->part->useFloatAmount()) {
             return round($this->amount);
         }
 

src/EventSubscriber/U2FRegistrationSubscriber.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/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/Migrations/Version20200311204104.php

Spacing +2 added lines, -2 removed lines

@@ -27,11 +27,11 @@
         $this->addSql('ALTER TABLE `users` ADD perms_parts_parameters SMALLINT NOT NULL');
         $this->addSql('ALTER TABLE log CHANGE level level TINYINT');
 
-        $sql = 'UPDATE `groups`' .
+        $sql = 'UPDATE `groups`'.
             'SET perms_parts_parameters = 341 WHERE (id = 1 AND name = "admins") OR (id = 3 AND name = "users");';
         $this->addSql($sql);
 
-        $sql = 'UPDATE `groups`' .
+        $sql = 'UPDATE `groups`'.
             'SET perms_parts_parameters = 681 WHERE (id = 2 AND name = "readonly");';
         $this->addSql($sql);
 

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