Part-DB / Part-DB-symfony

src/DataTables/LogDataTable.php

Spacing +12 added lines, -12 removed lines

@@ -90,7 +90,7 @@ 
         $optionsResolver->setAllowedTypes('mode', 'string');
         $optionsResolver->setAllowedTypes('filter', ['null', LogFilter::class]);
 
-        $optionsResolver->setNormalizer('filter_elements', static function (Options $options, $value) {
+        $optionsResolver->setNormalizer('filter_elements', static function(Options $options, $value) {
             if (!is_array($value)) {
                 return [$value];
             }
@@ -109,7 +109,7 @@ 
 
         //This special $$rowClass column is used to set the row class depending on the log level. The class gets set by the frontend controller
         $dataTable->add('dont_matter', RowClassColumn::class, [
-            'render' => static function ($value, AbstractLogEntry $context) {
+            'render' => static function($value, AbstractLogEntry $context) {
                 switch ($context->getLevel()) {
                     case AbstractLogEntry::LEVEL_EMERGENCY:
                     case AbstractLogEntry::LEVEL_ALERT:
@@ -129,7 +129,7 @@ 
         $dataTable->add('symbol', TextColumn::class, [
             'label' => '',
             'className' => 'no-colvis',
-            'render' => static function ($value, AbstractLogEntry $context) {
+            'render' => static function($value, AbstractLogEntry $context) {
                 switch ($context->getLevelString()) {
                     case LogLevel::DEBUG:
                         $symbol = 'fa-bug';
@@ -190,13 +190,13 @@ 
         $dataTable->add('type', TextColumn::class, [
             'label' => $this->translator->trans('log.type'),
             'propertyPath' => 'type',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 $text = $this->translator->trans('log.type.'.$value);
 
                 if ($context instanceof PartStockChangedLogEntry) {
                     $text .= sprintf(
                         ' (<i>%s</i>)',
-                        $this->translator->trans('log.part_stock_changed.' . $context->getInstockChangeType())
+                        $this->translator->trans('log.part_stock_changed.'.$context->getInstockChangeType())
                     );
                 }
 
@@ -208,14 +208,14 @@ 
             'label' => $this->translator->trans('log.level'),
             'visible' => 'system_log' === $options['mode'],
             'propertyPath' => 'levelString',
-            'render' => function (string $value, AbstractLogEntry $context) {
+            'render' => function(string $value, AbstractLogEntry $context) {
                 return $this->translator->trans('log.level.'.$value);
             },
         ]);
 
         $dataTable->add('user', TextColumn::class, [
             'label' => $this->translator->trans('log.user'),
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $user = $context->getUser();
 
                 //If user was deleted, show the info from the username field
@@ -238,7 +238,7 @@ 
         $dataTable->add('target_type', TextColumn::class, [
             'label' => $this->translator->trans('log.target_type'),
             'visible' => false,
-            'render' => function ($value, AbstractLogEntry $context) {
+            'render' => function($value, AbstractLogEntry $context) {
                 $class = $context->getTargetClass();
                 if (null !== $class) {
                     return $this->elementTypeNameGenerator->getLocalizedTypeLabel($class);
@@ -260,7 +260,7 @@ 
         $dataTable->add('timeTravel', IconLinkColumn::class, [
             'label' => '',
             'icon' => 'fas fa-fw fa-eye',
-            'href' => function ($value, AbstractLogEntry $context) {
+            'href' => function($value, AbstractLogEntry $context) {
                 if (
                     ($context instanceof TimeTravelInterface
                         && $context->hasOldDataInformations())
@@ -278,7 +278,7 @@ 
 
                 return null;
             },
-            'disabled' => function ($value, AbstractLogEntry $context) {
+            'disabled' => function($value, AbstractLogEntry $context) {
                 return !$this->security->isGranted('show_history', $context->getTargetClass());
             },
         ]);
@@ -291,11 +291,11 @@ 
 
         $dataTable->createAdapter(ORMAdapter::class, [
             'entity' => AbstractLogEntry::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/PartsDataTable.php

Spacing +12 added lines, -12 removed lines

@@ -75,7 +75,7 @@ 
     private $urlGenerator;
 
     public function __construct(EntityURLGenerator $urlGenerator, TranslatorInterface $translator,
-        NodesListBuilder $treeBuilder, AmountFormatter $amountFormatter,PartDataTableHelper $partDataTableHelper,
+        NodesListBuilder $treeBuilder, AmountFormatter $amountFormatter, PartDataTableHelper $partDataTableHelper,
         AttachmentURLGenerator $attachmentURLGenerator, Security $security)
     {
         $this->urlGenerator = $urlGenerator;
@@ -107,7 +107,7 @@ 
         $dataTable
             //Color the table rows depending on the review and favorite status
             ->add('dont_matter', RowClassColumn::class, [
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     if ($context->isNeedsReview()) {
                         return 'table-secondary';
                     }
@@ -123,13 +123,13 @@ 
             ->add('picture', TextColumn::class, [
                 'label' => '',
                 'className' => 'no-colvis',
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return $this->partDataTableHelper->renderPicture($context);
                 },
             ])
             ->add('name', TextColumn::class, [
                 'label' => $this->translator->trans('part.table.name'),
-                'render' => function ($value, Part $context) {
+                'render' => function($value, Part $context) {
                     return $this->partDataTableHelper->renderName($context);
                 },
             ])
@@ -167,7 +167,7 @@ 
         if ($this->security->isGranted('@storelocations.read')) {
             $dataTable->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
@@ -188,7 +188,7 @@ 
 
         $dataTable->add('amount', TextColumn::class, [
             'label' => $this->translator->trans('part.table.amount'),
-            'render' => function ($value, Part $context) {
+            'render' => function($value, Part $context) {
                 $amount = $context->getAmountSum();
                 $expiredAmount = $context->getExpiredAmountSum();
 
@@ -209,7 +209,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());
                 },
             ]);
@@ -271,10 +271,10 @@ 
             ->add('edit', IconLinkColumn::class, [
                 'label' => $this->translator->trans('part.table.edit'),
                 'visible' => false,
-                'href' => function ($value, Part $context) {
+                'href' => function($value, Part $context) {
                     return $this->urlGenerator->editURL($context);
                 },
-                'disabled' => function ($value, Part $context) {
+                'disabled' => function($value, Part $context) {
                     return !$this->security->isGranted('edit', $context);
                 },
                 'title' => $this->translator->trans('part.table.edit.title'),
@@ -283,12 +283,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(),
@@ -314,7 +314,7 @@ 
             ->addSelect(
                 '(
                     SELECT IFNULL(SUM(partLot.amount), 0.0)
-                    FROM '. PartLot::class. ' partLot
+                    FROM '. PartLot::class.' partLot
                     WHERE partLot.part = part.id
                     AND partLot.instock_unknown = false
                     AND (partLot.expiration_date IS NULL OR partLot.expiration_date > CURRENT_DATE())

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) {