From d5ee158caa43639f853b13d2d68368c9b90cf731 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julien=20Fastr=C3=A9?= <julien.fastre@champs-libres.coop>
Date: Mon, 5 Jun 2023 16:41:55 +0200
Subject: [PATCH] [export][rector] rector rules to add get form default data on
 export, aggregator, direct export

---
 ...aultDataOnExportFilterAggregatorRector.php |  10 +-
 ...th-no-method-get-form-default-data.php.inc | 133 ++++++++++++++++++
 ...th-no-method-get-form-default-data.php.inc |  77 ++++++++++
 ...th-no-method-get-form-default-data.php.inc |  95 +++++++++++++
 4 files changed, 314 insertions(+), 1 deletion(-)
 create mode 100644 utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/aggregator-with-no-method-get-form-default-data.php.inc
 create mode 100644 utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/direct-export-with-no-method-get-form-default-data.php.inc
 create mode 100644 utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/export-with-no-method-get-form-default-data.php.inc

diff --git a/utils/rector/src/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector.php b/utils/rector/src/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector.php
index c41ecfa91..5fe993180 100644
--- a/utils/rector/src/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector.php
+++ b/utils/rector/src/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector.php
@@ -11,6 +11,9 @@ declare(strict_types=1);
 
 namespace Utils\Rector\Rector;
 
+use Chill\MainBundle\Export\AggregatorInterface;
+use Chill\MainBundle\Export\DirectExportInterface;
+use Chill\MainBundle\Export\ExportInterface;
 use Chill\MainBundle\Export\FilterInterface;
 use PhpParser\Node;
 use Rector\Core\Rector\AbstractRector;
@@ -43,7 +46,12 @@ class ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector extends Abstra
             return null;
         }
 
-        if (!$this->classAnalyzer->hasImplements($node, FilterInterface::class)) {
+        if (
+            !$this->classAnalyzer->hasImplements($node, FilterInterface::class)
+            && !$this->classAnalyzer->hasImplements($node, AggregatorInterface::class)
+            && !$this->classAnalyzer->hasImplements($node, ExportInterface::class)
+            && !$this->classAnalyzer->hasImplements($node, DirectExportInterface::class)
+        ) {
             return null;
         }
 
diff --git a/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/aggregator-with-no-method-get-form-default-data.php.inc b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/aggregator-with-no-method-get-form-default-data.php.inc
new file mode 100644
index 000000000..aa373e629
--- /dev/null
+++ b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/aggregator-with-no-method-get-form-default-data.php.inc
@@ -0,0 +1,133 @@
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\ExportInterface;
+use Symfony\Component\Form\FormBuilderInterface;
+
+class MyClass implements ExportInterface
+{
+    public function buildForm(FormBuilderInterface $builder)
+    {
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+
+    public function getAllowedFormattersTypes()
+    {
+        // TODO: Implement getAllowedFormattersTypes() method.
+    }
+
+    public function getDescription()
+    {
+        // TODO: Implement getDescription() method.
+    }
+
+    public function getLabels($key, array $values, $data)
+    {
+        // TODO: Implement getLabels() method.
+    }
+
+    public function getQueryKeys($data)
+    {
+        // TODO: Implement getQueryKeys() method.
+    }
+
+    public function getResult($query, $data)
+    {
+        // TODO: Implement getResult() method.
+    }
+
+    public function getType()
+    {
+        // TODO: Implement getType() method.
+    }
+
+    public function initiateQuery(array $requiredModifiers, array $acl, array $data = [])
+    {
+        // TODO: Implement initiateQuery() method.
+    }
+
+    public function requiredRole(): string
+    {
+        // TODO: Implement requiredRole() method.
+    }
+
+    public function supportsModifiers()
+    {
+        // TODO: Implement supportsModifiers() method.
+    }
+}
+?>
+-----
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\ExportInterface;
+use Symfony\Component\Form\FormBuilderInterface;
+
+class MyClass implements ExportInterface
+{
+    public function buildForm(FormBuilderInterface $builder)
+    {
+    }
+    public function getFormDefaultData(): array
+    {
+        return [];
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+
+    public function getAllowedFormattersTypes()
+    {
+        // TODO: Implement getAllowedFormattersTypes() method.
+    }
+
+    public function getDescription()
+    {
+        // TODO: Implement getDescription() method.
+    }
+
+    public function getLabels($key, array $values, $data)
+    {
+        // TODO: Implement getLabels() method.
+    }
+
+    public function getQueryKeys($data)
+    {
+        // TODO: Implement getQueryKeys() method.
+    }
+
+    public function getResult($query, $data)
+    {
+        // TODO: Implement getResult() method.
+    }
+
+    public function getType()
+    {
+        // TODO: Implement getType() method.
+    }
+
+    public function initiateQuery(array $requiredModifiers, array $acl, array $data = [])
+    {
+        // TODO: Implement initiateQuery() method.
+    }
+
+    public function requiredRole(): string
+    {
+        // TODO: Implement requiredRole() method.
+    }
+
+    public function supportsModifiers()
+    {
+        // TODO: Implement supportsModifiers() method.
+    }
+}
+?>
diff --git a/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/direct-export-with-no-method-get-form-default-data.php.inc b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/direct-export-with-no-method-get-form-default-data.php.inc
new file mode 100644
index 000000000..d60f62dcb
--- /dev/null
+++ b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/direct-export-with-no-method-get-form-default-data.php.inc
@@ -0,0 +1,77 @@
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\DirectExportInterface;
+use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\HttpFoundation\Response;
+
+class MyClass implements DirectExportInterface
+{
+    public function generate(array $acl, array $data = []): Response
+    {
+        // TODO: Implement generate() method.
+    }
+
+    public function getDescription(): string
+    {
+        // TODO: Implement getDescription() method.
+    }
+
+    public function requiredRole(): string
+    {
+        // TODO: Implement requiredRole() method.
+    }
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+        // TODO: Implement buildForm() method.
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+}
+?>
+-----
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\DirectExportInterface;
+use Symfony\Component\Form\FormBuilderInterface;
+use Symfony\Component\HttpFoundation\Response;
+
+class MyClass implements DirectExportInterface
+{
+    public function generate(array $acl, array $data = []): Response
+    {
+        // TODO: Implement generate() method.
+    }
+
+    public function getDescription(): string
+    {
+        // TODO: Implement getDescription() method.
+    }
+
+    public function requiredRole(): string
+    {
+        // TODO: Implement requiredRole() method.
+    }
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+        // TODO: Implement buildForm() method.
+    }
+    public function getFormDefaultData(): array
+    {
+        return [];
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+}
+?>
diff --git a/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/export-with-no-method-get-form-default-data.php.inc b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/export-with-no-method-get-form-default-data.php.inc
new file mode 100644
index 000000000..ba5a6d4ec
--- /dev/null
+++ b/utils/rector/tests/Rector/ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector/Fixture/export-with-no-method-get-form-default-data.php.inc
@@ -0,0 +1,95 @@
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\AggregatorInterface;
+use Doctrine\ORM\QueryBuilder;
+use Symfony\Component\Form\FormBuilderInterface;
+
+class MyClass implements AggregatorInterface
+{
+    public function getLabels($key, array $values, $data)
+    {
+        // TODO: Implement getLabels() method.
+    }
+
+    public function getQueryKeys($data)
+    {
+        // TODO: Implement getQueryKeys() method.
+    }
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+
+    public function addRole(): ?string
+    {
+        // TODO: Implement addRole() method.
+    }
+
+    public function alterQuery(QueryBuilder $qb, $data)
+    {
+        // TODO: Implement alterQuery() method.
+    }
+
+    public function applyOn()
+    {
+        // TODO: Implement applyOn() method.
+    }
+}
+?>
+-----
+<?php
+
+namespace Utils\Rector\Tests\ChillBundleAddFormDefaultDataOnExportFilterAggregatorRector\Fixture;
+
+use Chill\MainBundle\Export\AggregatorInterface;
+use Doctrine\ORM\QueryBuilder;
+use Symfony\Component\Form\FormBuilderInterface;
+
+class MyClass implements AggregatorInterface
+{
+    public function getLabels($key, array $values, $data)
+    {
+        // TODO: Implement getLabels() method.
+    }
+
+    public function getQueryKeys($data)
+    {
+        // TODO: Implement getQueryKeys() method.
+    }
+
+    public function buildForm(FormBuilderInterface $builder)
+    {
+    }
+    public function getFormDefaultData(): array
+    {
+        return [];
+    }
+
+    public function getTitle()
+    {
+        // TODO: Implement getTitle() method.
+    }
+
+    public function addRole(): ?string
+    {
+        // TODO: Implement addRole() method.
+    }
+
+    public function alterQuery(QueryBuilder $qb, $data)
+    {
+        // TODO: Implement alterQuery() method.
+    }
+
+    public function applyOn()
+    {
+        // TODO: Implement applyOn() method.
+    }
+}
+?>