Part-DB / Part-DB-symfony

src/DataTables/ProjectBomEntriesDataTable.php

Spacing +12 added lines, -12 removed lines

@@ -62,8 +62,8 @@ 
             ->add('picture', TextColumn::class, [
                 'label' => '',
                 'className' => 'no-colvis',
-                'render' => function ($value, ProjectBOMEntry $context) {
-                    if($context->getPart() === null) {
+                'render' => function($value, ProjectBOMEntry $context) {
+                    if ($context->getPart() === null) {
                         return '';
                     }
                     return $this->partDataTableHelper->renderPicture($context->getPart());
@@ -78,7 +78,7 @@ 
             ->add('quantity', TextColumn::class, [
                 'label' => $this->translator->trans('project.bom.quantity'),
                 'className' => 'text-center',
-                'render' => function ($value, ProjectBOMEntry $context) {
+                'render' => function($value, ProjectBOMEntry $context) {
                     //If we have a non-part entry, only show the rounded quantity
                     if ($context->getPart() === null) {
                         return round($context->getQuantity());
@@ -91,13 +91,13 @@ 
             ->add('name', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.name'),
                 'orderable' => false,
-                'render' => function ($value, ProjectBOMEntry $context) {
-                    if($context->getPart() === null) {
+                'render' => function($value, ProjectBOMEntry $context) {
+                    if ($context->getPart() === null) {
                         return $context->getName();
                     }
-                    if($context->getPart() !== null) {
+                    if ($context->getPart() !== null) {
                         $tmp = $this->partDataTableHelper->renderName($context->getPart());
-                        if(!empty($context->getName())) {
+                        if (!empty($context->getName())) {
                             $tmp .= '<br><b>'.htmlspecialchars($context->getName()).'</b>';
                         }
                         return $tmp;
@@ -107,8 +107,8 @@ 
 
             ->add('description', MarkdownColumn::class, [
                 'label' => $this->translator->trans('part.table.description'),
-                'data' => function (ProjectBOMEntry $context) {
-                    if($context->getPart() !== null) {
+                'data' => function(ProjectBOMEntry $context) {
+                    if ($context->getPart() !== null) {
                         return $context->getPart()->getDescription();
                     }
                     //For non-part BOM entries show the comment field
@@ -133,7 +133,7 @@ 
 
             ->add('mountnames', TextColumn::class, [
                 'label' => 'project.bom.mountnames',
-                'render' => function ($value, ProjectBOMEntry $context) {
+                'render' => function($value, ProjectBOMEntry $context) {
                     $html = '';
 
                     foreach (explode(',', $context->getMountnames()) as $mountname) {
@@ -156,11 +156,11 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => Attachment::class,
-            'query' => function (QueryBuilder $builder) use ($options): void {
+            'query' => function(QueryBuilder $builder) use ($options): void {
                 $this->getQuery($builder, $options);
             },
             'criteria' => [
-                function (QueryBuilder $builder) use ($options): void {
+                function(QueryBuilder $builder) use ($options): void {
                     $this->buildCriteria($builder, $options);
                 },
                 new SearchCriteriaProvider(),

src/DataTables/AttachmentDataTable.php

Spacing +10 added lines, -10 removed lines

@@ -62,9 +62,9 @@ 
     public function configure(DataTable $dataTable, array $options): void
     {
         $dataTable->add('dont_matter', RowClassColumn::class, [
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 //Mark attachments with missing files yellow
-                if(!$this->attachmentHelper->isFileExisting($context)){
+                if (!$this->attachmentHelper->isFileExisting($context)) {
                     return 'table-warning';
                 }
 
@@ -75,7 +75,7 @@ 
         $dataTable->add('picture', TextColumn::class, [
             'label' => '',
             'className' => 'no-colvis',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 if ($context->isPicture()
                     && !$context->isExternal()
                     && $this->attachmentHelper->isFileExisting($context)) {
@@ -100,7 +100,7 @@ 
 
         $dataTable->add('name', TextColumn::class, [
             'label' => 'attachment.edit.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 //Link to external source
                 if ($context->isExternal()) {
                     return sprintf(
@@ -125,7 +125,7 @@ 
         $dataTable->add('attachment_type', TextColumn::class, [
             'label' => 'attachment.table.type',
             'field' => 'attachment_type.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 return sprintf(
                     '<a href="%s">%s</a>',
                     $this->entityURLGenerator->editURL($context->getAttachmentType()),
@@ -137,7 +137,7 @@ 
         $dataTable->add('element', TextColumn::class, [
             'label' => 'attachment.table.element',
             //'propertyPath' => 'element.name',
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 return sprintf(
                     '<a href="%s">%s</a>',
                     $this->entityURLGenerator->infoURL($context->getElement()),
@@ -153,7 +153,7 @@ 
 
         $dataTable->add('filesize', TextColumn::class, [
             'label' => $this->translator->trans('attachment.table.filesize'),
-            'render' => function ($value, Attachment $context) {
+            'render' => function($value, Attachment $context) {
                 if ($context->isExternal()) {
                     return sprintf(
                         '<span class="badge bg-primary">
@@ -211,11 +211,11 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => Attachment::class,
-            'query' => function (QueryBuilder $builder): void {
+            'query' => function(QueryBuilder $builder): void {
                 $this->getQuery($builder);
             },
             'criteria' => [
-                function (QueryBuilder $builder) use ($options): void {
+                function(QueryBuilder $builder) use ($options): void {
                     $this->buildCriteria($builder, $options);
                 },
                 new SearchCriteriaProvider(),
@@ -237,7 +237,7 @@ 
     {
         //We do the most stuff here in the filter class
         if (isset($options['filter'])) {
-            if(!$options['filter'] instanceof AttachmentFilter) {
+            if (!$options['filter'] instanceof AttachmentFilter) {
                 throw new \Exception('filter must be an instance of AttachmentFilter!');
             }
 

src/DataTables/Column/EntityColumn.php

Spacing +3 added lines, -3 removed lines

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

src/DataTables/Helpers/PartDataTableHelper.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
         }
         if ($context->getBuiltProject() !== null) {
             $icon = sprintf('<i class="fa-solid fa-box-archive fa-fw me-1" title="%s"></i>',
-                $this->translator->trans('part.info.projectBuildPart.hint') . ': ' . $context->getBuiltProject()->getName());
+                $this->translator->trans('part.info.projectBuildPart.hint').': '.$context->getBuiltProject()->getName());
         }
 
 

src/DataTables/Filters/Constraints/FilterTrait.php

Spacing +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
         //Replace all special characters with underscores
         $property = preg_replace('/[^a-zA-Z0-9_]/', '_', $property);
         //Add a random number to the end of the property name for uniqueness
-        return $property . '_' . uniqid("", false);
+        return $property.'_'.uniqid("", false);
     }
 
     /**
@@ -72,7 +72,7 @@ 
             $expression = sprintf("%s %s :%s", $property, $comparison_operator, $parameterIdentifier);
         }
 
-        if($this->useHaving || $this->isAggregateFunctionString($property)) { //If the property is an aggregate function, we have to use the "having" instead of the "where"
+        if ($this->useHaving || $this->isAggregateFunctionString($property)) { //If the property is an aggregate function, we have to use the "having" instead of the "where"
             $queryBuilder->andHaving($expression);
         } else {
             $queryBuilder->andWhere($expression);

src/DataTables/Filters/Constraints/BooleanConstraint.php

Spacing +1 added lines, -1 removed lines

@@ -62,7 +62,7 @@
     public function apply(QueryBuilder $queryBuilder): void
     {
         //Do not apply a filter if value is null (filter is set to ignore)
-        if(!$this->isEnabled()) {
+        if (!$this->isEnabled()) {
             return;
         }
 

src/DataTables/Filters/Constraints/TextConstraint.php

Spacing +7 added lines, -7 removed lines

@@ -86,16 +86,16 @@ 
 
     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));
         }
 
         //Equal and not equal can be handled easily
-        if($this->operator === '=' || $this->operator === '!=') {
+        if ($this->operator === '=' || $this->operator === '!=') {
 
             $this->addSimpleAndConstraint($queryBuilder, $this->property, $this->identifier, $this->operator, $this->value);
             return;
@@ -106,11 +106,11 @@ 
         if ($this->operator === 'LIKE') {
             $like_value = $this->value;
         } else if ($this->operator === 'STARTS') {
-            $like_value = $this->value . '%';
+            $like_value = $this->value.'%';
         } else if ($this->operator === 'ENDS') {
-            $like_value = '%' . $this->value;
+            $like_value = '%'.$this->value;
         } else if ($this->operator === 'CONTAINS') {
-            $like_value = '%' . $this->value . '%';
+            $like_value = '%'.$this->value.'%';
         }
 
         if ($like_value !== null) {

src/DataTables/Filters/Constraints/IntConstraint.php

Spacing +2 added lines, -2 removed lines

@@ -26,10 +26,10 @@
 {
     public function apply(QueryBuilder $queryBuilder): void
     {
-        if($this->value1 !== null) {
+        if ($this->value1 !== null) {
             $this->value1 = (int) $this->value1;
         }
-        if($this->value2 !== null) {
+        if ($this->value2 !== null) {
             $this->value2 = (int) $this->value2;
         }
 

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');
     }
 
     /**