Part-DB / Part-DB-symfony

src/Controller/RedirectController.php

Spacing +3 added lines, -3 removed lines

@@ -62,7 +62,7 @@ 
 
         //Check if a user has set a preferred language setting:
         $user = $this->getUser();
-        if (($user instanceof User) && ! empty($user->getLanguage())) {
+        if (($user instanceof User) && !empty($user->getLanguage())) {
             $locale = $user->getLanguage();
         }
 
@@ -70,7 +70,7 @@ 
         $new_url = $request->getUriForPath('/'.$locale.$request->getPathInfo());
 
         //If either mod_rewrite is not enabled or the index.php version is enforced, add index.php to the string
-        if (($this->enforce_index_php || ! $this->checkIfModRewriteAvailable())
+        if (($this->enforce_index_php || !$this->checkIfModRewriteAvailable())
             && false === strpos($new_url, 'index.php')) {
             //Like Request::getUriForPath only with index.php
             $new_url = $request->getSchemeAndHttpHost().$request->getBaseUrl().'/index.php/'.$locale.$request->getPathInfo();
@@ -88,7 +88,7 @@ 
      */
     public function checkIfModRewriteAvailable(): bool
     {
-        if (! function_exists('apache_get_modules')) {
+        if (!function_exists('apache_get_modules')) {
             //If we can not check for apache modules, we just hope for the best and assume url rewriting is available
             //If you want to enforce index.php versions of the url, you can override this via ENV vars.
             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/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

src/Entity/Parameters/AbstractParameter.php

Spacing +2 added lines, -2 removed lines

@@ -191,7 +191,7 @@ 
      */
     public function setElement(AbstractDBElement $element): self
     {
-        if (! is_a($element, static::ALLOWED_ELEMENT_CLASS)) {
+        if (!is_a($element, static::ALLOWED_ELEMENT_CLASS)) {
             throw new InvalidArgumentException(sprintf('The element associated with a %s must be a %s!', static::class, static::ALLOWED_ELEMENT_CLASS));
         }
 
@@ -409,7 +409,7 @@ 
     protected function formatWithUnit(float $value, string $format = '%g'): string
     {
         $str = \sprintf($format, $value);
-        if (! empty($this->unit)) {
+        if (!empty($this->unit)) {
             return $str.' '.$this->unit;
         }