Part-DB / Part-DB-symfony

src/DataTables/Filters/Constraints/InstanceOfConstraint.php

Spacing +5 added lines, -5 removed lines

@@ -90,25 +90,25 @@
         }
 
         //Ensure we have an valid operator
-        if(!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
-            throw new \RuntimeException('Invalid operator '. $this->operator . ' provided. Valid operators are '. implode(', ', self::ALLOWED_OPERATOR_VALUES));
+        if (!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
+            throw new \RuntimeException('Invalid operator '.$this->operator.' provided. Valid operators are '.implode(', ', self::ALLOWED_OPERATOR_VALUES));
         }
 
         $expressions = [];
 
         if ($this->operator === 'ANY' || $this->operator === 'NONE') {
-            foreach($this->value as $value) {
+            foreach ($this->value as $value) {
                 //We cannnot use an paramater here, as this is the only way to pass the FCQN to the query (via binded params, we would need to use ClassMetaData). See: https://github.com/doctrine/orm/issues/4462
                 $expressions[] = ($queryBuilder->expr()->isInstanceOf($this->property, $value));
             }
 
-            if($this->operator === 'ANY') {
+            if ($this->operator === 'ANY') {
                 $queryBuilder->andWhere($queryBuilder->expr()->orX(...$expressions));
             } else { //NONE
                 $queryBuilder->andWhere($queryBuilder->expr()->not($queryBuilder->expr()->orX(...$expressions)));
             }
         } else {
-            throw new \RuntimeException('Unknown operator '. $this->operator . ' provided. Valid operators are '. implode(', ', self::ALLOWED_OPERATOR_VALUES));
+            throw new \RuntimeException('Unknown operator '.$this->operator.' provided. Valid operators are '.implode(', ', self::ALLOWED_OPERATOR_VALUES));
         }
     }
 }
\ No newline at end of file

src/DataTables/Filters/Constraints/EntityConstraint.php

Spacing +8 added lines, -8 removed lines

@@ -111,7 +111,7 @@ 
     public function setValue(?AbstractDBElement $value): void
     {
         if (!$value instanceof $this->class) {
-            throw new \InvalidArgumentException('The value must be an instance of ' . $this->class);
+            throw new \InvalidArgumentException('The value must be an instance of '.$this->class);
         }
 
         $this->value = $value;
@@ -155,13 +155,13 @@ 
         }
 
         //Ensure we have an valid operator
-        if(!in_array($this->operator, $this->getAllowedOperatorValues(), true)) {
-            throw new \RuntimeException('Invalid operator '. $this->operator . ' provided. Valid operators are '. implode(', ', $this->getAllowedOperatorValues()));
+        if (!in_array($this->operator, $this->getAllowedOperatorValues(), true)) {
+            throw new \RuntimeException('Invalid operator '.$this->operator.' provided. Valid operators are '.implode(', ', $this->getAllowedOperatorValues()));
         }
 
         //We need to handle null values differently, as they can not be compared with == or !=
         if ($this->value === null) {
-            if($this->operator === '=' || $this->operator === 'INCLUDING_CHILDREN') {
+            if ($this->operator === '=' || $this->operator === 'INCLUDING_CHILDREN') {
                 $queryBuilder->andWhere(sprintf("%s IS NULL", $this->property));
                 return;
             }
@@ -171,16 +171,16 @@ 
                 return;
             }
 
-            throw new \RuntimeException('Unknown operator '. $this->operator . ' provided. Valid operators are '. implode(', ', $this->getAllowedOperatorValues()));
+            throw new \RuntimeException('Unknown operator '.$this->operator.' provided. Valid operators are '.implode(', ', $this->getAllowedOperatorValues()));
         }
 
-        if($this->operator === '=' || $this->operator === '!=') {
+        if ($this->operator === '=' || $this->operator === '!=') {
            $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier, $this->operator, $this->value);
            return;
         }
 
         //Otherwise retrieve the children list and apply the operator to it
-        if($this->isStructural()) {
+        if ($this->isStructural()) {
             $list = $this->nodesListBuilder->getChildrenFlatList($this->value);
             //Add the element itself to the list
             $list[] = $this->value;
@@ -195,7 +195,7 @@ 
                 return;
             }
         } else {
-            throw new \RuntimeException('Cannot apply operator '. $this->operator . ' to non-structural type');
+            throw new \RuntimeException('Cannot apply operator '.$this->operator.' to non-structural type');
         }
 
     }

src/DataTables/Filters/Constraints/NumberConstraint.php

Spacing +5 added lines, -5 removed lines

@@ -117,19 +117,19 @@
         }
 
         //Ensure we have an valid operator
-        if(!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
-            throw new \RuntimeException('Invalid operator '. $this->operator . ' provided. Valid operators are '. implode(', ', self::ALLOWED_OPERATOR_VALUES));
+        if (!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
+            throw new \RuntimeException('Invalid operator '.$this->operator.' provided. Valid operators are '.implode(', ', self::ALLOWED_OPERATOR_VALUES));
         }
 
         if ($this->operator !== 'BETWEEN') {
             $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier, $this->operator, $this->value1);
-        }  else {
+        } else {
             if ($this->value2 === null) {
                 throw new RuntimeException("Cannot use operator BETWEEN without value2!");
             }
 
-            $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier . '1', '>=', $this->value1);
-            $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier . '2', '<=', $this->value2);
+            $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier.'1', '>=', $this->value1);
+            $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier.'2', '<=', $this->value2);
         }
     }
 }
\ No newline at end of file

Braces +1 added lines, -1 removed lines

@@ -123,7 +123,7 @@
 
         if ($this->operator !== 'BETWEEN') {
             $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier, $this->operator, $this->value1);
-        }  else {
+        } else {
             if ($this->value2 === null) {
                 throw new RuntimeException("Cannot use operator BETWEEN without value2!");
             }

src/DataTables/Filters/Constraints/ChoiceConstraint.php

Spacing +3 added lines, -3 removed lines

@@ -87,8 +87,8 @@ 
         }
 
         //Ensure we have an valid operator
-        if(!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
-            throw new \RuntimeException('Invalid operator '. $this->operator . ' provided. Valid operators are '. implode(', ', self::ALLOWED_OPERATOR_VALUES));
+        if (!in_array($this->operator, self::ALLOWED_OPERATOR_VALUES, true)) {
+            throw new \RuntimeException('Invalid operator '.$this->operator.' provided. Valid operators are '.implode(', ', self::ALLOWED_OPERATOR_VALUES));
         }
 
         if ($this->operator === 'ANY') {
@@ -96,7 +96,7 @@ 
         } elseif ($this->operator === 'NONE') {
             $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier, 'NOT IN', $this->value);
         } else {
-            throw new \RuntimeException('Unknown operator '. $this->operator . ' provided. Valid operators are '. implode(', ', self::ALLOWED_OPERATOR_VALUES));
+            throw new \RuntimeException('Unknown operator '.$this->operator.' provided. Valid operators are '.implode(', ', self::ALLOWED_OPERATOR_VALUES));
         }
     }
 }
\ No newline at end of file

src/DataTables/Filters/PartSearchFilter.php

Spacing +12 added lines, -12 removed lines

@@ -74,34 +74,34 @@ 
     {
         $fields_to_search = [];
 
-        if($this->name) {
+        if ($this->name) {
             $fields_to_search[] = 'part.name';
         }
-        if($this->category) {
+        if ($this->category) {
             $fields_to_search[] = 'category.name';
         }
-        if($this->description) {
+        if ($this->description) {
             $fields_to_search[] = 'part.description';
         }
-        if($this->tags) {
+        if ($this->tags) {
             $fields_to_search[] = 'part.tags';
         }
-        if($this->storelocation) {
+        if ($this->storelocation) {
             $fields_to_search[] = 'storelocations.name';
         }
-        if($this->ordernr) {
+        if ($this->ordernr) {
             $fields_to_search[] = 'orderdetails.supplierpartnr';
         }
-        if($this->mpn) {
+        if ($this->mpn) {
             $fields_to_search[] = 'part.manufacturer_product_url';
         }
-        if($this->supplier) {
+        if ($this->supplier) {
             $fields_to_search[] = 'suppliers.name';
         }
-        if($this->manufacturer) {
+        if ($this->manufacturer) {
             $fields_to_search[] = 'manufacturer.name';
         }
-        if($this->footprint) {
+        if ($this->footprint) {
             $fields_to_search[] = 'footprint.name';
         }
 
@@ -118,7 +118,7 @@ 
         }
 
         //Convert the fields to search to a list of expressions
-        $expressions = array_map(function (string $field) {
+        $expressions = array_map(function(string $field) {
             if ($this->regex) {
                 return sprintf("REGEXP(%s, :search_query) = 1", $field);
             }
@@ -135,7 +135,7 @@ 
         if ($this->regex) {
             $queryBuilder->setParameter('search_query', $this->keyword);
         } else {
-            $queryBuilder->setParameter('search_query', '%' . $this->keyword . '%');
+            $queryBuilder->setParameter('search_query', '%'.$this->keyword.'%');
         }
     }
 

src/DataTables/Filters/CompoundFilterTrait.php

Spacing +3 added lines, -3 removed lines

@@ -42,15 +42,15 @@
 
             $value = $property->getValue($this);
             //We only want filters (objects implementing FilterInterface)
-            if($value instanceof FilterInterface) {
+            if ($value instanceof FilterInterface) {
                 $filters[$property->getName()] = $value;
             }
 
             //Add filters in collections
             if ($value instanceof Collection) {
                 foreach ($value as $key => $filter) {
-                    if($filter instanceof FilterInterface) {
-                        $filters[$property->getName() . '.' . (string) $key] = $filter;
+                    if ($filter instanceof FilterInterface) {
+                        $filters[$property->getName().'.'.(string) $key] = $filter;
                     }
                 }
             }

src/EntityListeners/TreeCacheInvalidationListener.php

Spacing +1 added lines, -1 removed lines

@@ -57,7 +57,7 @@
             $this->cache->invalidateTags([$secure_class_name]);
 
             //Trigger a sidebar reload for all users (see SidebarTreeUpdater service)
-            if(!$element instanceof LabelProfile) {
+            if (!$element instanceof LabelProfile) {
                 $this->cache->invalidateTags(['sidebar_tree_update']);
             }
         }

src/Form/ProjectSystem/ProjectBOMEntryType.php

Spacing +1 added lines, -1 removed lines

@@ -26,7 +26,7 @@
     {
 
 
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (PreSetDataEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(PreSetDataEvent $event) {
             $form = $event->getForm();
             /** @var ProjectBOMEntry $data */
             $data = $event->getData();

src/Form/Type/PartSelectType.php

Spacing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         //At initialization we have to fill the form element with our selected data, so the user can see it
-        $builder->addEventListener(FormEvents::PRE_SET_DATA, function (PreSetDataEvent $event) {
+        $builder->addEventListener(FormEvents::PRE_SET_DATA, function(PreSetDataEvent $event) {
             $form = $event->getForm();
             $config = $form->getConfig()->getOptions();
             $data = $event->getData() ?? [];
@@ -95,8 +95,8 @@ 
 
         $resolver->setDefaults([
             //Prefill the selected choice with the needed data, so the user can see it without an additional Ajax request
-            'choice_attr' => ChoiceList::attr($this, function (?Part $part) {
-                if($part) {
+            'choice_attr' => ChoiceList::attr($this, function(?Part $part) {
+                if ($part) {
                     //Determine the picture to show:
                     $preview_attachment = $this->previewGenerator->getTablePreviewAttachment($part);
                     if ($preview_attachment !== null) {