wip

Kreyu · Nov 23, 2024 · 2e48fb3 · 2e48fb3
1 parent 77340b0
commit 2e48fb3
Show file tree

Hide file tree

Showing 5 changed files with 1,197 additions and 37 deletions.
diff --git a/src/Column/Type/ColumnType.php b/src/Column/Type/ColumnType.php
@@ -25,9 +25,17 @@ public function __construct(
 
     public function buildColumn(ColumnBuilderInterface $builder, array $options): void
     {
+        $sortPropertyPath = null;
+
+        if (true === $options['sort']) {
+            $sortPropertyPath = $builder->getName();
+        } elseif (is_string($options['sort'])) {
+            $sortPropertyPath = $options['sort'];
+        }
+
         $builder
-            ->setPropertyPath($options['property_path'] ?: null)
-            ->setSortPropertyPath(is_string($options['sort']) ? $options['sort'] : null)
+            ->setPropertyPath($options['property_path'] ?? $builder->getName() ?: null)
+            ->setSortPropertyPath($sortPropertyPath)
             ->setPriority($options['priority'])
             ->setVisible($options['visible'])
             ->setPersonalizable($options['personalizable'])
@@ -77,14 +85,22 @@ public function buildValueView(ColumnValueView $view, ColumnInterface $column, a
             $attr = $attr($normData, $rowData);
         }
 
+        $translationParameters = $options['value_translation_parameters'];
+
+        if (is_callable($translationParameters)) {
+            $translationParameters = $translationParameters($normData, $rowData);
+        }
+
         $view->vars = array_replace($view->vars, [
+            'name' => $column->getName(),
+            'column' => $view,
             'row' => $view->parent,
             'data_table' => $view->parent->parent,
             'block_prefixes' => $this->getColumnBlockPrefixes($column, $options),
             'data' => $view->data,
             'value' => $view->value,
             'translation_domain' => $options['value_translation_domain'] ?? $view->parent->parent->vars['translation_domain'] ?? null,
-            'translation_parameters' => $options['value_translation_parameters'] ?? [],
+            'translation_parameters' => $translationParameters ?? [],
             'attr' => $attr,
         ]);
     }
@@ -99,29 +115,21 @@ public function buildExportHeaderView(ColumnHeaderView $view, ColumnInterface $c
             $options['export'] = [];
         }
 
-        $options['export'] += [
-            'getter' => $options['getter'],
-            'property_path' => $options['property_path'],
-            'formatter' => $options['formatter'],
-        ];
+        $options['export']['label'] ??= $options['label'] ?? StringUtil::camelToSentence($column->getName());
+        $options['export']['header_translation_domain'] ??= $options['header_translation_domain'] ?? $view->parent->parent->vars['translation_domain'] ?? false;
+        $options['export']['header_translation_parameters'] ??= $options['header_translation_parameters'] ?? [];
 
-        $label = $options['label'] ?? StringUtil::camelToSentence($column->getName());
+        $label = $options['export']['label'];
 
         if ($this->translator) {
             if ($label instanceof TranslatableInterface) {
                 $label = $label->trans($this->translator, $this->translator->getLocale());
             } else {
-                $translationDomain = $options['export']['header_translation_domain']
-                    ?? $options['header_translation_domain']
-                    ?? $view->parent->parent->vars['translation_domain']
-                    ?? false;
+                $translationDomain = $options['export']['header_translation_domain'];
+                $translationParameters = $options['export']['header_translation_parameters'];
 
                 if ($translationDomain) {
-                    $label = $this->translator->trans(
-                        id: $label,
-                        parameters: $options['header_translation_parameters'],
-                        domain: $translationDomain,
-                    );
+                    $label = $this->translator->trans($label, $translationParameters, $translationDomain);
                 }
             }
         }
@@ -139,35 +147,43 @@ public function buildExportValueView(ColumnValueView $view, ColumnInterface $col
             $options['export'] = [];
         }
 
-        $options['export'] += [
-            'getter' => $options['getter'],
-            'property_path' => $options['property_path'],
-            'property_accessor' => $options['property_accessor'],
-            'formatter' => $options['formatter'],
-        ];
+        $options['export']['getter'] ??= $options['getter'];
+        $options['export']['property_path'] ??= $options['property_path'];
+        $options['export']['property_accessor'] ??= $options['property_accessor'];
+        $options['export']['formatter'] ??= $options['formatter'];
+        $options['export']['value_translation_domain'] ??= $options['value_translation_domain'] ?? $view->parent->parent->vars['translation_domain'] ?? false;
+        $options['export']['value_translation_parameters'] ??= $options['value_translation_parameters'] ?? [];
 
         $rowData = $view->parent->data;
 
         $normData = $this->getNormDataFromRowData($rowData, $column, $options['export']);
         $viewData = $this->getViewDataFromNormData($normData, $rowData, $column, $options['export']);
 
-        if ($this->translator && is_string($viewData)) {
-            $translationDomain = $options['export']['value_translation_domain']
-                ?? $options['value_translation_domain']
-                ?? $view->parent->parent->vars['translation_domain']
-                ?? false;
-
-            if ($translationDomain) {
-                $viewData = $this->translator->trans(
-                    id: $viewData,
-                    parameters: $options['value_translation_parameters'],
-                    domain: $translationDomain,
-                );
+        if ($this->translator && (is_string($viewData) || $viewData instanceof TranslatableInterface)) {
+            if ($viewData instanceof TranslatableInterface) {
+                $viewData = $viewData->trans($this->translator, $this->translator->getLocale());
+            } else {
+                $translationDomain = $options['export']['value_translation_domain'];
+                $translationParameters = $options['export']['value_translation_parameters'];
+
+                if (is_callable($translationParameters)) {
+                    $translationParameters = $translationParameters($normData, $rowData);
+                }
+
+                if ($translationDomain) {
+                    $viewData = $this->translator->trans(
+                        id: $viewData,
+                        parameters: $translationParameters,
+                        domain: $translationDomain,
+                    );
+                }
             }
         }
 
+        $view->data = $normData;
         $view->value = $viewData;
 
+        $view->vars['data'] = $normData;
         $view->vars['value'] = $viewData;
     }
 
@@ -198,7 +214,7 @@ public function configureOptions(OptionsResolver $resolver): void
             ->setAllowedTypes('header_translation_domain', ['null', 'bool', 'string'])
             ->setAllowedTypes('header_translation_parameters', ['null', 'array'])
             ->setAllowedTypes('value_translation_domain', ['null', 'bool', 'string'])
-            ->setAllowedTypes('value_translation_parameters', 'array')
+            ->setAllowedTypes('value_translation_parameters', ['array', 'callable'])
             ->setAllowedTypes('block_name', ['null', 'string'])
             ->setAllowedTypes('block_prefix', ['null', 'string'])
             ->setAllowedTypes('sort', ['bool', 'string'])

diff --git a/src/Test/Column/Type/ColumnTypeTestCase.php b/src/Test/Column/Type/ColumnTypeTestCase.php
@@ -0,0 +1,72 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Kreyu\Bundle\DataTableBundle\Test\Column\Type;
+
+use Kreyu\Bundle\DataTableBundle\Column\ColumnFactoryInterface;
+use Kreyu\Bundle\DataTableBundle\Column\ColumnRegistry;
+use Kreyu\Bundle\DataTableBundle\Column\Type\ColumnTypeInterface;
+use Kreyu\Bundle\DataTableBundle\Column\Type\ResolvedColumnTypeFactory;
+use Kreyu\Bundle\DataTableBundle\DataTableFactory;
+use Kreyu\Bundle\DataTableBundle\DataTableFactoryInterface;
+use Kreyu\Bundle\DataTableBundle\DataTableInterface;
+use Kreyu\Bundle\DataTableBundle\DataTableRegistry;
+use Kreyu\Bundle\DataTableBundle\DataTableRegistryInterface;
+use Kreyu\Bundle\DataTableBundle\Query\ArrayProxyQuery;
+use Kreyu\Bundle\DataTableBundle\Tests\Fixtures\Column\TestColumnFactory;
+use Kreyu\Bundle\DataTableBundle\Type\DataTableType;
+use Kreyu\Bundle\DataTableBundle\Type\ResolvedDataTableTypeFactory;
+use PHPUnit\Framework\TestCase;
+
+abstract class ColumnTypeTestCase extends TestCase
+{
+    protected ColumnFactoryInterface $factory;
+    protected DataTableInterface $dataTable;
+
+    protected function setUp(): void
+    {
+        parent::setUp();
+
+        $this->recreateFactoryWithType($this->instantiateType());
+    }
+
+    protected function recreateFactoryWithType(ColumnTypeInterface $type): void
+    {
+        $registry = new ColumnRegistry(
+            types: [$type],
+            typeExtensions: [],
+            resolvedTypeFactory: new ResolvedColumnTypeFactory(),
+        );
+
+        $this->factory = new TestColumnFactory($registry);
+        $this->factory->setDataTable($this->getDataTable());
+    }
+
+    protected function instantiateType(): ColumnTypeInterface
+    {
+        return new ($this->getTestedType());
+    }
+
+    abstract protected function getTestedType(): string;
+
+    protected function createDataTableRegistry(): DataTableRegistryInterface
+    {
+        return new DataTableRegistry(
+            types: [new DataTableType()],
+            typeExtensions: [],
+            proxyQueryFactories: [],
+            resolvedTypeFactory: new ResolvedDataTableTypeFactory(),
+        );
+    }
+
+    protected function createDataTableFactory(): DataTableFactoryInterface
+    {
+        return new DataTableFactory($this->createDataTableRegistry());
+    }
+
+    protected function getDataTable(): DataTableInterface
+    {
+        return $this->dataTable ??= $this->createDataTableFactory()->create(DataTableType::class, new ArrayProxyQuery([]));
+    }
+}
diff --git a/tests/Fixtures/Column/TestColumnFactory.php b/tests/Fixtures/Column/TestColumnFactory.php
@@ -0,0 +1,36 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Kreyu\Bundle\DataTableBundle\Tests\Fixtures\Column;
+
+use Kreyu\Bundle\DataTableBundle\Column\ColumnFactory;
+use Kreyu\Bundle\DataTableBundle\Column\ColumnInterface;
+use Kreyu\Bundle\DataTableBundle\Column\Type\ColumnType;
+use Kreyu\Bundle\DataTableBundle\DataTableInterface;
+
+class TestColumnFactory extends ColumnFactory
+{
+    private DataTableInterface $dataTable;
+
+    public function create(string $type = ColumnType::class, array $options = []): ColumnInterface
+    {
+        $column = parent::create($type, $options);
+        $column->setDataTable($this->dataTable);
+
+        return $column;
+    }
+
+    public function createNamed(string $name, string $type = ColumnType::class, array $options = []): ColumnInterface
+    {
+        $column = parent::createNamed($name, $type, $options);
+        $column->setDataTable($this->dataTable);
+
+        return $column;
+    }
+
+    public function setDataTable(DataTableInterface $dataTable): void
+    {
+        $this->dataTable = $dataTable;
+    }
+}
diff --git a/tests/Fixtures/Model/User.php b/tests/Fixtures/Model/User.php
@@ -0,0 +1,20 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Kreyu\Bundle\DataTableBundle\Tests\Fixtures\Model;
+
+use Symfony\Contracts\Translation\TranslatableInterface;
+
+class User
+{
+    public function __construct(
+        public null|string|TranslatableInterface $firstName = null,
+    ) {
+    }
+
+    public function getFirstNameUppercased(): string
+    {
+        return strtoupper($this->firstName);
+    }
+}