Refactor dumper config initialization

app/config/services.yaml

@@ -69,6 +69,9 @@ services:
         arguments:
             - !tagged_iterator config.compiler_processor
 
+    config.compiler.dump_output:
+        class: 'Smile\GdprDump\Config\Compiler\Processor\DumpOutputProcessor'
+
     config.compiler.processor.env_var:
         class: 'Smile\GdprDump\Config\Compiler\Processor\EnvVarProcessor'
 

src/Config/Compiler/Processor/DumpOutputProcessor.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Smile\GdprDump\Config\Compiler\Processor;
+
+use Smile\GdprDump\Config\ConfigInterface;
+
+class DumpOutputProcessor implements ProcessorInterface
+{
+    /**
+     * Process date placeholders in dump output (e.g. "dump-{Y-m-d-H.i.s}.sql").
+     */
+    public function process(ConfigInterface $config): void
+    {
+        $dumpSettings = $config->get('dump', []);
+
+        if (array_key_exists('output', $dumpSettings)) {
+            $dumpSettings['output'] = preg_replace_callback(
+                '/{([^}]+)}/',
+                fn (array $matches) => date($matches[1]),
+                $dumpSettings['output']
+            );
+
+            $config->set('dump', $dumpSettings);
+        }
+    }
+}

src/Config/Compiler/Processor/EnvVarProcessor.php

@@ -26,7 +26,7 @@ class EnvVarProcessor implements ProcessorInterface
     ];
 
     /**
-     * @inheritdoc
+     * Replace environment variable placeholders (e.g. "%env(DB_HOST)%")
      */
     public function process(ConfigInterface $config): void
     {

src/Config/Compiler/Processor/VersionProcessor.php

@@ -11,7 +11,7 @@
 class VersionProcessor implements ProcessorInterface
 {
     /**
-     * @inheritdoc
+     * Process the "if_version" parameter.
      */
     public function process(ConfigInterface $config): void
     {

src/Database/TableDependencyResolver.php

@@ -120,6 +120,10 @@ private function buildDependencyTree(): void
      */
     private function isForeignKeyIgnored(ForeignKey $foreignKey): bool
     {
-        return in_array($foreignKey->getConstraintName(), $this->config->getIgnoredForeignKeys(), true);
+        return in_array(
+            $foreignKey->getConstraintName(),
+            $this->config->getFilterPropagationSettings()->getIgnoredForeignKeys(),
+            true
+        );
     }
 }

src/Dumper/Config/Definition/FakerSettings.php

@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Smile\GdprDump\Dumper\Config\Definition;
+
+class FakerSettings
+{
+    public function __construct(private string $locale)
+    {
+    }
+
+    /**
+     * Get the faker locale.
+     */
+    public function getLocale(): string
+    {
+        return $this->locale;
+    }
+}

src/Dumper/Config/Definition/FilterPropagationSettings.php

@@ -0,0 +1,28 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Smile\GdprDump\Dumper\Config\Definition;
+
+class FilterPropagationSettings
+{
+    public function __construct(private bool $enabled, private array $ignoredForeignKeys)
+    {
+    }
+
+    /**
+     * Checker whether filter propagation is enabled.
+     */
+    public function isEnabled(): bool
+    {
+        return $this->enabled;
+    }
+
+    /**
+     * Get foreign keys to ignore when propagating filters to table dependencies.
+     */
+    public function getIgnoredForeignKeys(): array
+    {
+        return $this->ignoredForeignKeys;
+    }
+}

src/Dumper/Config/Table/Filter/Filter.php → src/Dumper/Config/Definition/Table/Filter.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Smile\GdprDump\Dumper\Config\Table\Filter;
+namespace Smile\GdprDump\Dumper\Config\Definition\Table;
 
 use UnexpectedValueException;
 

src/Dumper/Config/Table/Filter/SortOrder.php → src/Dumper/Config/Definition/Table/SortOrder.php

@@ -2,7 +2,7 @@
 
 declare(strict_types=1);
 
-namespace Smile\GdprDump\Dumper\Config\Table\Filter;
+namespace Smile\GdprDump\Dumper\Config\Definition\Table;
 
 use UnexpectedValueException;
 

src/Dumper/Config/Table/TableConfig.php → src/Dumper/Config/Definition/TableConfig.php

@@ -2,16 +2,16 @@
 
 declare(strict_types=1);
 
-namespace Smile\GdprDump\Dumper\Config\Table;
+namespace Smile\GdprDump\Dumper\Config\Definition;
 
-use Smile\GdprDump\Dumper\Config\Table\Filter\Filter;
-use Smile\GdprDump\Dumper\Config\Table\Filter\SortOrder;
+use Smile\GdprDump\Dumper\Config\Definition\Table\Filter;
+use Smile\GdprDump\Dumper\Config\Definition\Table\SortOrder;
+use Smile\GdprDump\Dumper\Config\Validation\ValidationException;
 use Smile\GdprDump\Dumper\Config\Validation\WhereExprValidator;
 use UnexpectedValueException;
 
 class TableConfig
 {
-    private WhereExprValidator $whereExprValidator;
     private string $name;
     private ?string $where = null;
     private ?int $limit = null;
@@ -31,7 +31,6 @@ class TableConfig
 
     public function __construct(string $tableName, array $tableConfig)
     {
-        $this->whereExprValidator = new WhereExprValidator();
         $this->name = $tableName;
         $this->prepareConfig($tableConfig);
     }
@@ -145,6 +144,8 @@ private function prepareConfig(array $tableData): void
 
     /**
      * Prepare the table filters.
+     *
+     * @throws ValidationException
      */
     private function prepareFilters(array $tableData): void
     {
@@ -158,7 +159,8 @@ private function prepareFilters(array $tableData): void
         // New way of declaring table filters (`where` parameter)
         $whereCondition = (string) ($tableData['where'] ?? '');
         if ($whereCondition !== '') {
-            $this->whereExprValidator->validate($whereCondition);
+            $whereExprValidator = new WhereExprValidator();
+            $whereExprValidator->validate($whereCondition);
             $this->where = $whereCondition;
         }
     }

src/Dumper/Config/Definition/TableConfigCollection.php

@@ -0,0 +1,67 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Smile\GdprDump\Dumper\Config\Definition;
+
+use ArrayIterator;
+use IteratorAggregate;
+use Traversable;
+use UnexpectedValueException;
+
+/**
+ * @implements IteratorAggregate<TableConfig>
+ */
+class TableConfigCollection implements IteratorAggregate
+{
+    /**
+     * @var TableConfig[]
+     */
+    private array $items = [];
+
+    /**+
+     * Add an item.
+     */
+    public function add(TableConfig $tableConfig): void
+    {
+        $this->items[$tableConfig->getName()] = $tableConfig;
+    }
+
+    /**
+     * Get an item.
+     *
+     * @throws UnexpectedValueException
+     */
+    public function get(string $name): TableConfig
+    {
+        return $this->has($name)
+            ? $this->items[$name]
+            : throw new UnexpectedValueException(sprintf('The table "%s" is not defined.', $name));
+    }
+
+    /**
+     * Check whether an item exists.
+     */
+    public function has(string $name): bool
+    {
+        return array_key_exists($name, $this->items);
+    }
+
+    /**
+     * Get all items.
+     *
+     * @return TableConfig[]
+     */
+    public function all(): array
+    {
+        return $this->items;
+    }
+
+    /**
+     * @return ArrayIterator<string, TableConfig>
+     */
+    public function getIterator(): Traversable
+    {
+        return new ArrayIterator($this->items);
+    }
+}