Part-DB / Part-DB-symfony

src/DataTables/Filters/Constraints/Part/ParameterConstraint.php

Spacing +11 added lines, -11 removed lines

@@ -53,8 +53,8 @@ 
         //The alias has to be uniq for each subquery, so generate a random one
         $this->alias = uniqid('param_', false);
 
-        $this->value_text = new TextConstraint($this->alias . '.value_text');
-        $this->value = new ParameterValueConstraint($this->alias );
+        $this->value_text = new TextConstraint($this->alias.'.value_text');
+        $this->value = new ParameterValueConstraint($this->alias);
     }
 
     public function isEnabled(): bool
@@ -69,26 +69,26 @@ 
 
 
 
-        $subqb->select('COUNT(' . $this->alias . ')')
+        $subqb->select('COUNT('.$this->alias.')')
             ->from(PartParameter::class, $this->alias)
-            ->where($this->alias . '.element = part');
+            ->where($this->alias.'.element = part');
 
         if (!empty($this->name)) {
             $paramName = $this->generateParameterIdentifier('params.name');
-            $subqb->andWhere($this->alias . '.name = :' . $paramName);
-            $queryBuilder->setParameter($paramName,  $this->name);
+            $subqb->andWhere($this->alias.'.name = :'.$paramName);
+            $queryBuilder->setParameter($paramName, $this->name);
         }
 
         if (!empty($this->symbol)) {
             $paramName = $this->generateParameterIdentifier('params.symbol');
-            $subqb->andWhere($this->alias . '.symbol = :' . $paramName);
-            $queryBuilder->setParameter($paramName,  $this->symbol);
+            $subqb->andWhere($this->alias.'.symbol = :'.$paramName);
+            $queryBuilder->setParameter($paramName, $this->symbol);
         }
 
         if (!empty($this->unit)) {
             $paramName = $this->generateParameterIdentifier('params.unit');
-            $subqb->andWhere($this->alias . '.unit = :' . $paramName);
-            $queryBuilder->setParameter($paramName,  $this->unit);
+            $subqb->andWhere($this->alias.'.unit = :'.$paramName);
+            $queryBuilder->setParameter($paramName, $this->unit);
         }
 
         //Apply all subfilters
@@ -101,7 +101,7 @@ 
             $queryBuilder->setParameter($parameter->getName(), $parameter->getValue());
         }
 
-        $queryBuilder->andWhere('(' . $subqb->getDQL() . ') > 0');
+        $queryBuilder->andWhere('('.$subqb->getDQL().') > 0');
     }
 
     /**

src/DataTables/Filters/Constraints/Part/TagsConstraint.php

Spacing +12 added lines, -12 removed lines

@@ -102,34 +102,34 @@
      */
     protected function getExpressionForTag(QueryBuilder $queryBuilder, string $tag): Expr\Orx
     {
-        $tag_identifier_prefix = uniqid($this->identifier . '_', false);
+        $tag_identifier_prefix = uniqid($this->identifier.'_', false);
 
         $expr = $queryBuilder->expr();
 
         $tmp = $expr->orX(
-            $expr->like($this->property, ':' . $tag_identifier_prefix . '_1'),
-            $expr->like($this->property, ':' . $tag_identifier_prefix . '_2'),
-            $expr->like($this->property, ':' . $tag_identifier_prefix . '_3'),
-            $expr->eq($this->property, ':' . $tag_identifier_prefix . '_4'),
+            $expr->like($this->property, ':'.$tag_identifier_prefix.'_1'),
+            $expr->like($this->property, ':'.$tag_identifier_prefix.'_2'),
+            $expr->like($this->property, ':'.$tag_identifier_prefix.'_3'),
+            $expr->eq($this->property, ':'.$tag_identifier_prefix.'_4'),
         );
 
         //Set the parameters for the LIKE expression, in each variation of the tag (so with a comma, at the end, at the beginning, and on both ends, and equaling the tag)
-        $queryBuilder->setParameter($tag_identifier_prefix . '_1', '%,' . $tag . ',%');
-        $queryBuilder->setParameter($tag_identifier_prefix . '_2', '%,' . $tag);
-        $queryBuilder->setParameter($tag_identifier_prefix . '_3', $tag . ',%');
-        $queryBuilder->setParameter($tag_identifier_prefix . '_4', $tag);
+        $queryBuilder->setParameter($tag_identifier_prefix.'_1', '%,'.$tag.',%');
+        $queryBuilder->setParameter($tag_identifier_prefix.'_2', '%,'.$tag);
+        $queryBuilder->setParameter($tag_identifier_prefix.'_3', $tag.',%');
+        $queryBuilder->setParameter($tag_identifier_prefix.'_4', $tag);
 
         return $tmp;
     }
 
     public function apply(QueryBuilder $queryBuilder): void
     {
-        if(!$this->isEnabled()) {
+        if (!$this->isEnabled()) {
             return;
         }
 
-        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));
         }
 
         $tagsExpressions = [];

src/DataTables/Filters/Constraints/Part/ParameterValueConstraint.php

Spacing +2 added lines, -2 removed lines

@@ -37,13 +37,13 @@
     public function __construct(string $alias) {
         $this->alias = $alias;
 
-        parent::__construct($alias . '.value_typical');
+        parent::__construct($alias.'.value_typical');
     }
 
     public function apply(QueryBuilder $queryBuilder): void
     {
         //Skip if not enabled
-        if(!$this->isEnabled()) {
+        if (!$this->isEnabled()) {
             return;
         }
 

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;
                     }
                 }
             }