Merge branch 'tests/fixes-2022-09-05' into 111_exports_suite

src/Bundle/ChillPersonBundle/Export/Aggregator/HouseholdAggregators/ChildrenNumberAggregator.php

@@ -1,15 +1,26 @@
 <?php
 
+/**
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
 namespace Chill\PersonBundle\Export\Aggregator\HouseholdAggregators;
 
 use Chill\MainBundle\Export\AggregatorInterface;
 use Chill\MainBundle\Form\Type\ChillDateType;
 use Chill\PersonBundle\Export\Declarations;
+use DateTime;
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Query\Expr\Andx;
 use Doctrine\ORM\QueryBuilder;
 use Symfony\Component\Form\FormBuilderInterface;
 use Symfony\Contracts\Translation\TranslatorInterface;
+use function in_array;
 
 class ChildrenNumberAggregator implements AggregatorInterface
 {
@@ -21,64 +32,14 @@ class ChildrenNumberAggregator implements AggregatorInterface
         $this->translator = $translator;
     }
 
-    /**
-     * @inheritDoc
-     */
-    public function getLabels($key, array $values, $data)
-    {
-        return function ($value): string {
-
-            if ($value === '_header') {
-                return 'Number of children';
-            }
-
-            return $this->translator->trans(
-                'household_composition.numberOfChildren children in household', [
-                'numberOfChildren' => $value
-            ]);
-        };
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function getQueryKeys($data): array
-    {
-        return ['childrennumber_aggregator'];
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function buildForm(FormBuilderInterface $builder)
-    {
-        $builder->add('on_date', ChillDateType::class, [
-            'data' => new \DateTime('now'),
-        ]);
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function getTitle(): string
-    {
-        return 'Group by number of children';
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function addRole()
+    public function addRole(): ?string
     {
         return null;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function alterQuery(QueryBuilder $qb, $data)
     {
-        if (!in_array('composition', $qb->getAllAliases())) {
+        if (!in_array('composition', $qb->getAllAliases(), true)) {
             $qb->join('household.compositions', 'composition');
         }
 
@@ -113,11 +74,41 @@ class ChildrenNumberAggregator implements AggregatorInterface
         $qb->setParameter('ondate', $data['on_date'], Types::DATE_MUTABLE);
     }
 
-    /**
-     * @inheritDoc
-     */
     public function applyOn(): string
     {
         return Declarations::HOUSEHOLD_TYPE;
     }
-}
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+        $builder->add('on_date', ChillDateType::class, [
+            'data' => new DateTime('now'),
+        ]);
+    }
+
+    public function getLabels($key, array $values, $data)
+    {
+        return function ($value): string {
+            if ('_header' === $value) {
+                return 'Number of children';
+            }
+
+            return $this->translator->trans(
+                'household_composition.numberOfChildren children in household',
+                [
+                    'numberOfChildren' => $value,
+                ]
+            );
+        };
+    }
+
+    public function getQueryKeys($data): array
+    {
+        return ['childrennumber_aggregator'];
+    }
+
+    public function getTitle(): string
+    {
+        return 'Group by number of children';
+    }
+}

src/Bundle/ChillPersonBundle/Export/Aggregator/HouseholdAggregators/CompositionAggregator.php

@@ -1,5 +1,14 @@
 <?php
 
+/**
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+declare(strict_types=1);
+
 namespace Chill\PersonBundle\Export\Aggregator\HouseholdAggregators;
 
 use Chill\MainBundle\Export\AggregatorInterface;
@@ -7,17 +16,19 @@ use Chill\MainBundle\Form\Type\ChillDateType;
 use Chill\MainBundle\Templating\TranslatableStringHelper;
 use Chill\PersonBundle\Export\Declarations;
 use Chill\PersonBundle\Repository\Household\HouseholdCompositionTypeRepository;
+use DateTime;
 use Doctrine\DBAL\Types\Types;
 use Doctrine\ORM\Query\Expr\Andx;
 use Doctrine\ORM\QueryBuilder;
 use Symfony\Component\Form\FormBuilderInterface;
+use function in_array;
 
 class CompositionAggregator implements AggregatorInterface
 {
-    private HouseholdCompositionTypeRepository $typeRepository;
-
     private TranslatableStringHelper $translatableStringHelper;
 
+    private HouseholdCompositionTypeRepository $typeRepository;
+
     public function __construct(
         HouseholdCompositionTypeRepository $typeRepository,
         TranslatableStringHelper $translatableStringHelper
@@ -26,64 +37,14 @@ class CompositionAggregator implements AggregatorInterface
         $this->translatableStringHelper = $translatableStringHelper;
     }
 
-    /**
-     * @inheritDoc
-     */
-    public function getLabels($key, array $values, $data)
-    {
-        return function ($value): string {
-            if ($value === '_header') {
-                return 'Composition';
-            }
-
-            $c = $this->typeRepository->find($value);
-
-            return $this->translatableStringHelper->localize(
-                $c->getLabel()
-            );
-        };
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function getQueryKeys($data): array
-    {
-        return ['composition_aggregator'];
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function buildForm(FormBuilderInterface $builder)
-    {
-        $builder->add('on_date', ChillDateType::class, [
-            'data' => new \DateTime('now'),
-        ]);
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function getTitle(): string
-    {
-        return 'Group by composition';
-    }
-
-    /**
-     * @inheritDoc
-     */
-    public function addRole()
+    public function addRole(): ?string
     {
         return null;
     }
 
-    /**
-     * @inheritDoc
-     */
     public function alterQuery(QueryBuilder $qb, $data)
     {
-        if (!in_array('composition', $qb->getAllAliases())) {
+        if (!in_array('composition', $qb->getAllAliases(), true)) {
             $qb->join('household.compositions', 'composition');
         }
 
@@ -118,11 +79,40 @@ class CompositionAggregator implements AggregatorInterface
         $qb->setParameter('ondate', $data['on_date'], Types::DATE_MUTABLE);
     }
 
-    /**
-     * @inheritDoc
-     */
     public function applyOn(): string
     {
         return Declarations::HOUSEHOLD_TYPE;
     }
-}
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+        $builder->add('on_date', ChillDateType::class, [
+            'data' => new DateTime('now'),
+        ]);
+    }
+
+    public function getLabels($key, array $values, $data)
+    {
+        return function ($value): string {
+            if ('_header' === $value) {
+                return 'Composition';
+            }
+
+            $c = $this->typeRepository->find($value);
+
+            return $this->translatableStringHelper->localize(
+                $c->getLabel()
+            );
+        };
+    }
+
+    public function getQueryKeys($data): array
+    {
+        return ['composition_aggregator'];
+    }
+
+    public function getTitle(): string
+    {
+        return 'Group by composition';
+    }
+}