Part-DB / Part-DB-symfony

src/Services/PricedetailHelper.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
             } else {
                 // We have to sort the pricedetails manually
                 $array = $pricedetails->map(
-                    function (Pricedetail $pricedetail) {
+                    function(Pricedetail $pricedetail) {
                         return $pricedetail->getMinDiscountQuantity();
                     }
                 )->toArray();

src/Twig/AppExtension.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
     public function getTests()
     {
         return [
-            new TwigTest('instanceof', function ($var, $instance) {
+            new TwigTest('instanceof', function($var, $instance) {
                 return $var instanceof $instance;
             }),
         ];

src/Entity/Base/StructuralDBElement.php

Indentation -1 removed lines

@@ -176,7 +176,6 @@
 
     /**
      *  Get the comment of the element.
-
      *
      * @return string the comment
      */

Spacing +3 added lines, -3 removed lines

@@ -117,7 +117,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!');
         }
 
@@ -201,7 +201,7 @@ 
      */
     public function getFullPath(string $delimiter = self::PATH_DELIMITER_ARROW): string
     {
-        if (! \is_array($this->full_path_strings)) {
+        if (!\is_array($this->full_path_strings)) {
             $this->full_path_strings = [];
             $this->full_path_strings[] = $this->getName();
             $element = $this;
@@ -233,7 +233,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;
         }
 

src/DataTables/Column/EntityColumn.php

Spacing +3 added lines, -3 removed lines

@@ -60,12 +60,12 @@
 
         $resolver->setRequired('property');
 
-        $resolver->setDefault('field', function (Options $option) {
+        $resolver->setDefault('field', function(Options $option) {
             return $option['property'].'.name';
         });
 
-        $resolver->setDefault('render', function (Options $options) {
-            return function ($value, Part $context) use ($options) {
+        $resolver->setDefault('render', function(Options $options) {
+            return function($value, Part $context) use ($options) {
                 /** @var DBElement $entity */
                 $entity = $this->accessor->getValue($context, $options['property']);
 

src/Services/Trees/ToolsTreeBuilder.php

Spacing +1 added lines, -1 removed lines

@@ -77,7 +77,7 @@
     {
         $key = 'tree_tools_'.$this->keyGenerator->generateKey();
 
-        return $this->cache->get($key, function (ItemInterface $item) {
+        return $this->cache->get($key, function(ItemInterface $item) {
             //Invalidate tree, whenever group or the user changes
             $item->tag(['tree_tools', 'groups', $this->keyGenerator->generateKey()]);
 

tests/Controller/AdminPages/AbstractAdminControllerTest.php

Spacing +3 added lines, -3 removed lines

@@ -71,7 +71,7 @@ 
         $crawler = $client->request('GET', static::$base_path.'/new');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     /**
@@ -96,7 +96,7 @@ 
         $crawler = $client->request('GET', static::$base_path.'/1');
         $this->assertFalse($client->getResponse()->isRedirect());
         $this->assertSame($read, $client->getResponse()->isSuccessful(), 'Controller was not successful!');
-        $this->assertSame($read, ! $client->getResponse()->isForbidden(), 'Permission Checking not working!');
+        $this->assertSame($read, !$client->getResponse()->isForbidden(), 'Permission Checking not working!');
     }
 
     public function deleteDataProvider()
@@ -133,6 +133,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!');
     }
 }

src/Controller/AttachmentFileController.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
             throw new \RuntimeException('You can not download external attachments!');
         }
 
-        if (! $helper->isFileExisting($attachment)) {
+        if (!$helper->isFileExisting($attachment)) {
             throw new \RuntimeException('The file associated with the attachment is not existing!');
         }
 
@@ -82,7 +82,7 @@ 
             throw new \RuntimeException('You can not download external attachments!');
         }
 
-        if (! $helper->isFileExisting($attachment)) {
+        if (!$helper->isFileExisting($attachment)) {
             throw new \RuntimeException('The file associated with the attachment is not existing!');
         }
 

src/Controller/RedirectController.php

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,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();
         }
 
@@ -66,7 +66,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();
@@ -84,7 +84,7 @@ 
      */
     public function checkIfModRewriteAvailable()
     {
-        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/Controller/PartController.php

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
             $this->addFlash('info', 'part.edited_flash');
             //Reload form, so the SIUnitType entries use the new part unit
             $form = $this->createForm(PartBaseType::class, $part);
-        } elseif ($form->isSubmitted() && ! $form->isValid()) {
+        } elseif ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'part.edited_flash.invalid');
         }
 
@@ -189,7 +189,7 @@ 
             return $this->redirectToRoute('part_edit', ['id' => $new_part->getID()]);
         }
 
-        if ($form->isSubmitted() && ! $form->isValid()) {
+        if ($form->isSubmitted() && !$form->isValid()) {
             $this->addFlash('error', 'part.created_flash.invalid');
         }