php-cs-fixer

Kreyu · Jul 20, 2023 · aa3053c · aa3053c
1 parent e64817c
commit aa3053c
Show file tree

Hide file tree

Showing 27 changed files with 61 additions and 60 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,7 +1,5 @@
 # 0.13
 
-## What's changed
-
 - **[Feature]** Batch actions ([see more](https://data-table-bundle.swroblewski.pl/features/actions/batch-actions/))
 - **[Feature]** Improved DX with row actions ([see more](https://data-table-bundle.swroblewski.pl/features/actions/row-actions/))
 - **[Feature]** Actions `visible` option ([see more](https://data-table-bundle.swroblewski.pl/reference/actions/types/action/#visible))

diff --git a/src/Action/ActionBuilder.php b/src/Action/ActionBuilder.php
@@ -176,4 +176,4 @@ private function createBuilderLockedException(): BadMethodCallException
     {
         return new BadMethodCallException('ActionConfigBuilder methods cannot be accessed anymore once the builder is turned into a ActionConfigInterface instance.');
     }
-}
+}
diff --git a/src/Action/Type/ActionType.php b/src/Action/Type/ActionType.php
@@ -77,7 +77,7 @@ public function buildView(ActionView $view, ActionInterface $action, array $opti
             'icon_attr' => $options['icon_attr'],
             'confirmation' => $options['confirmation'],
             'context' => $action->getConfig()->getContext(),
-            'batch' => $action->getConfig()->getContext() === ActionContext::Batch,
+            'batch' => ActionContext::Batch === $action->getConfig()->getContext(),
             'visible' => $options['visible'],
         ]);
     }

diff --git a/src/Action/Type/ResolvedActionType.php b/src/Action/Type/ResolvedActionType.php
@@ -72,10 +72,7 @@ public function createBuilder(ActionFactoryInterface $factory, string $name, arr
         try {
             $options = $this->getOptionsResolver()->resolve($options);
         } catch (ExceptionInterface $exception) {
-            throw new $exception(sprintf(
-                'An error has occurred resolving the options of the action "%s": ',
-                get_debug_type($this->getInnerType())).$exception->getMessage(), $exception->getCode(), $exception
-            );
+            throw new $exception(sprintf('An error has occurred resolving the options of the action "%s": ', get_debug_type($this->getInnerType())).$exception->getMessage(), $exception->getCode(), $exception);
         }
 
         return new ActionBuilder($name, $this, $options);

diff --git a/src/Bridge/Doctrine/Orm/Filter/Type/BooleanFilterType.php b/src/Bridge/Doctrine/Orm/Filter/Type/BooleanFilterType.php
@@ -13,11 +13,10 @@
 use Symfony\Component\Form\Extension\Core\Type as Form;
 use Symfony\Component\OptionsResolver\Options;
 use Symfony\Component\OptionsResolver\OptionsResolver;
+use Symfony\Component\Translation\TranslatableMessage;
 
 use function Symfony\Component\Translation\t;
 
-use Symfony\Component\Translation\TranslatableMessage;
-
 class BooleanFilterType extends AbstractFilterType
 {
     /**

diff --git a/src/Bridge/Doctrine/Orm/Filter/Type/DateRangeFilterType.php b/src/Bridge/Doctrine/Orm/Filter/Type/DateRangeFilterType.php
@@ -10,8 +10,8 @@
 use Kreyu\Bundle\DataTableBundle\Filter\Form\Type\DateRangeType;
 use Kreyu\Bundle\DataTableBundle\Query\ProxyQueryInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
-
 use Symfony\Component\Translation\TranslatableMessage;
+
 use function Symfony\Component\Translation\t;
 
 class DateRangeFilterType extends AbstractFilterType

diff --git a/src/Bridge/Doctrine/Orm/Query/DoctrineOrmProxyQuery.php b/src/Bridge/Doctrine/Orm/Query/DoctrineOrmProxyQuery.php
@@ -4,7 +4,6 @@
 
 namespace Kreyu\Bundle\DataTableBundle\Bridge\Doctrine\Orm\Query;
 
-use Doctrine\ORM\Query;
 use Doctrine\ORM\QueryBuilder;
 use Doctrine\ORM\Tools\Pagination\CountWalker;
 use Doctrine\ORM\Tools\Pagination\Paginator;
@@ -13,7 +12,6 @@
 use Kreyu\Bundle\DataTableBundle\Pagination\PaginationData;
 use Kreyu\Bundle\DataTableBundle\Pagination\PaginationInterface;
 use Kreyu\Bundle\DataTableBundle\Query\ProxyQueryInterface;
-use Kreyu\Bundle\DataTableBundle\Sorting\Direction;
 use Kreyu\Bundle\DataTableBundle\Sorting\SortingData;
 
 /**

diff --git a/src/Bridge/OpenSpout/Exporter/Type/AbstractExporterType.php b/src/Bridge/OpenSpout/Exporter/Type/AbstractExporterType.php
@@ -4,7 +4,6 @@
 
 namespace Kreyu\Bundle\DataTableBundle\Bridge\OpenSpout\Exporter\Type;
 
-use Kreyu\Bundle\DataTableBundle\Column\ColumnHeaderView;
 use Kreyu\Bundle\DataTableBundle\DataTableView;
 use Kreyu\Bundle\DataTableBundle\Exporter\ExportFile;
 use Kreyu\Bundle\DataTableBundle\Exporter\Type\AbstractExporterType as BaseAbstractExporterType;
@@ -18,7 +17,7 @@
 abstract class AbstractExporterType extends BaseAbstractExporterType
 {
     public function __construct(
-        private ?TranslatorInterface $translator = null
+        private ?TranslatorInterface $translator = null,
     ) {
     }
 
@@ -77,7 +76,7 @@ public function export(DataTableView $view, string $filename, array $options = [
         return new ExportFile($path, "$filename.$extension");
     }
 
-    protected abstract function getExtension(): string;
+    abstract protected function getExtension(): string;
 
-    protected abstract function getWriter(array $options): WriterInterface;
-}
+    abstract protected function getWriter(array $options): WriterInterface;
+}
diff --git a/src/Bridge/OpenSpout/Exporter/Type/CsvExporterType.php b/src/Bridge/OpenSpout/Exporter/Type/CsvExporterType.php
@@ -44,4 +44,4 @@ public function configureOptions(OptionsResolver $resolver): void
             ->setAllowedTypes('flush_threshold', 'int')
         ;
     }
-}
+}
diff --git a/src/Bridge/OpenSpout/Exporter/Type/OdsExporterType.php b/src/Bridge/OpenSpout/Exporter/Type/OdsExporterType.php
@@ -5,9 +5,9 @@
 namespace Kreyu\Bundle\DataTableBundle\Bridge\OpenSpout\Exporter\Type;
 
 use OpenSpout\Common\Entity\Style\Style;
-use OpenSpout\Writer\WriterInterface;
 use OpenSpout\Writer\ODS\Options;
 use OpenSpout\Writer\ODS\Writer;
+use OpenSpout\Writer\WriterInterface;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
 class OdsExporterType extends AbstractExporterType
@@ -45,4 +45,4 @@ public function configureOptions(OptionsResolver $resolver): void
             ->setAllowedTypes('default_row_height', ['null', 'float'])
         ;
     }
-}
+}
diff --git a/src/Bridge/OpenSpout/Exporter/Type/XlsxExporterType.php b/src/Bridge/OpenSpout/Exporter/Type/XlsxExporterType.php
@@ -45,4 +45,4 @@ public function configureOptions(OptionsResolver $resolver): void
             ->setAllowedTypes('default_row_height', ['null', 'float'])
         ;
     }
-}
+}
diff --git a/src/Column/Type/ActionsColumnType.php b/src/Column/Type/ActionsColumnType.php
@@ -73,7 +73,7 @@ public function configureOptions(OptionsResolver $resolver): void
     private function resolveAction(
         string $name,
         array|ActionBuilderInterface|ActionInterface $action,
-        ColumnHeaderView|ColumnValueView $view
+        ColumnHeaderView|ColumnValueView $view,
     ): ?ActionInterface {
         if ($action instanceof ActionInterface) {
             return $action;

diff --git a/src/Column/Type/NumberColumnType.php b/src/Column/Type/NumberColumnType.php
@@ -6,10 +6,8 @@
 
 use Kreyu\Bundle\DataTableBundle\Column\ColumnInterface;
 use Kreyu\Bundle\DataTableBundle\Column\ColumnValueView;
-use Symfony\Component\OptionsResolver\Options;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 use Symfony\Component\Translation\Formatter\IntlFormatter;
-use Twig\Extra\Intl\IntlExtension;
 
 class NumberColumnType extends AbstractColumnType
 {
@@ -30,12 +28,12 @@ public function configureOptions(OptionsResolver $resolver): void
                     $resolver
                         ->setDefaults([
                             'attrs' => [],
-                            'style' => 'decimal'
+                            'style' => 'decimal',
                         ])
                         ->setAllowedTypes('attrs', 'array')
                         ->setAllowedTypes('style', 'string')
                     ;
-                }
+                },
             ])
             ->setAllowedTypes('use_intl_formatter', 'bool')
         ;

diff --git a/src/DataTable.php b/src/DataTable.php
@@ -260,11 +260,11 @@ public function sort(SortingData $data): void
 
             $sortField = $column->getOptions()['sort'];
 
-            if ($sortField === false) {
+            if (false === $sortField) {
                 continue;
             }
 
-            if ($sortField === true) {
+            if (true === $sortField) {
                 $sortField = $column->getOptions()['property_path'] ?: $column->getName();
             }
 

diff --git a/src/DataTableBuilder.php b/src/DataTableBuilder.php
@@ -314,7 +314,7 @@ class DataTableBuilder implements DataTableBuilderInterface
      */
     private bool $locked = false;
 
-    public function __construct(string $name, ?ProxyQueryInterface $query = null, array $options = [])
+    public function __construct(string $name, ProxyQueryInterface $query = null, array $options = [])
     {
         $this->name = $name;
         $this->query = $query;

diff --git a/src/DataTableBuilderInterface.php b/src/DataTableBuilderInterface.php
@@ -28,7 +28,7 @@ public function getAction(string $name): ActionBuilderInterface;
     public function hasAction(string $name): bool;
 
     /**
-     * @param null|class-string<ActionTypeInterface> $type
+     * @param class-string<ActionTypeInterface>|null $type
      */
     public function addAction(ActionBuilderInterface|string $action, string $type = null, array $options = []): static;
 
@@ -47,7 +47,7 @@ public function getBatchAction(string $name): ActionBuilderInterface;
     public function hasBatchAction(string $name): bool;
 
     /**
-     * @param null|class-string<ActionTypeInterface> $type
+     * @param class-string<ActionTypeInterface>|null $type
      */
     public function addBatchAction(ActionBuilderInterface|string $action, string $type = null, array $options = []): static;
 
@@ -70,7 +70,7 @@ public function getRowAction(string $name): ActionBuilderInterface;
     public function hasRowAction(string $name): bool;
 
     /**
-     * @param null|class-string<ActionTypeInterface> $type
+     * @param class-string<ActionTypeInterface>|null $type
      */
     public function addRowAction(ActionBuilderInterface|string $action, string $type = null, array $options = []): static;
 

diff --git a/src/DataTableConfigBuilderInterface.php b/src/DataTableConfigBuilderInterface.php
@@ -5,7 +5,6 @@
 namespace Kreyu\Bundle\DataTableBundle;
 
 use Kreyu\Bundle\DataTableBundle\Action\ActionFactoryInterface;
-use Kreyu\Bundle\DataTableBundle\Action\Type\ActionTypeInterface;
 use Kreyu\Bundle\DataTableBundle\Column\ColumnFactoryInterface;
 use Kreyu\Bundle\DataTableBundle\Column\Type\ColumnTypeInterface;
 use Kreyu\Bundle\DataTableBundle\Exporter\ExportData;

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -32,7 +32,7 @@ public function getConfigTreeBuilder(): TreeBuilder
                     ->setDeprecated(
                         'kreyu/data-table-bundle',
                         '0.12.0',
-                        'The child node "%node%" at path "%path%" is deprecated, use "defaults.themes" instead.'
+                        'The child node "%node%" at path "%path%" is deprecated, use "defaults.themes" instead.',
                     )
                 ->end()
                 ->arrayNode('defaults')

diff --git a/src/Pagination/CurrentPageOutOfRangeException.php b/src/Pagination/CurrentPageOutOfRangeException.php
@@ -7,4 +7,4 @@
 class CurrentPageOutOfRangeException extends \Exception
 {
     // ...
-}
+}
diff --git a/src/Persistence/CachePersistenceAdapter.php b/src/Persistence/CachePersistenceAdapter.php
@@ -6,13 +6,12 @@
 
 use Kreyu\Bundle\DataTableBundle\DataTableInterface;
 use Psr\Cache\InvalidArgumentException;
-
-use function Symfony\Component\String\u;
-
 use Symfony\Contracts\Cache\CacheInterface;
 use Symfony\Contracts\Cache\ItemInterface;
 use Symfony\Contracts\Cache\TagAwareCacheInterface;
 
+use function Symfony\Component\String\u;
+
 class CachePersistenceAdapter implements PersistenceAdapterInterface
 {
     public const TAG_PREFIX = 'kreyu_data_table_persistence_';

diff --git a/src/RowIterator.php b/src/RowIterator.php
@@ -45,4 +45,4 @@ public function rewind(): void
     {
         $this->recreateGenerator();
     }
-}
+}
diff --git a/src/Sorting/Direction.php b/src/Sorting/Direction.php
@@ -8,4 +8,4 @@ enum Direction: string
 {
     case ASC = 'asc';
     case DESC = 'desc';
-}
+}
diff --git a/src/Sorting/SortingColumnData.php b/src/Sorting/SortingColumnData.php
@@ -16,8 +16,7 @@ public function __construct(
     }
 
     /**
-     * @param  array{name: string, direction: string} $data
-     * @return self
+     * @param array{name: string, direction: string} $data
      */
     public static function fromArray(array $data): self
     {