chore: fix coding style (#1091)

DependencyInjection/APYDataGridExtension.php

@@ -20,7 +20,7 @@
 
 class APYDataGridExtension extends Extension
 {
-    public function load(array $configs, ContainerBuilder $container):void
+    public function load(array $configs, ContainerBuilder $container): void
     {
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);

DependencyInjection/Compiler/GridExtensionPass.php

@@ -18,7 +18,7 @@
 
 class GridExtensionPass implements CompilerPassInterface
 {
-    public function process(ContainerBuilder $container):void
+    public function process(ContainerBuilder $container): void
     {
         if (false === $container->hasDefinition('grid')) {
             return;

Grid/Columns.php

@@ -22,7 +22,7 @@ class Columns implements \IteratorAggregate, \Countable
     protected $columns = [];
     protected $extensions = [];
 
-    const MISSING_COLUMN_EX_MSG = 'Column with id "%s" doesn\'t exists';
+    private const MISSING_COLUMN_EX_MSG = 'Column with id "%s" doesn\'t exists';
 
     /**
      * @var AuthorizationCheckerInterface
@@ -42,7 +42,7 @@ public function __construct(AuthorizationCheckerInterface $authorizationChecker)
      *
      * @return ColumnsIterator
      */
-    public function getIterator($showOnlySourceColumns = false):ColumnsIterator
+    public function getIterator($showOnlySourceColumns = false): ColumnsIterator
     {
         return new ColumnsIterator(new \ArrayIterator($this->columns), $showOnlySourceColumns);
     }
@@ -128,7 +128,7 @@ public function getPrimaryColumn()
     /**
      * @return int
      */
-    public function count():int
+    public function count(): int
     {
         return count($this->columns);
     }

Grid/Export/ExcelExport.php

@@ -12,6 +12,8 @@
 
 namespace APY\DataGridBundle\Grid\Export;
 
+use APY\DataGridBundle\Grid\Grid;
+
 /**
  * Excel (This export produces a warning with new Office Excel).
  */

Grid/Export/Export.php

@@ -61,14 +61,13 @@ abstract class Export implements ExportInterface, ContainerAwareInterface
      *
      * @return \APY\DataGridBundle\Grid\Export\Export
      */
-    public function __construct( $title, $fileName = 'export', $params = [], $charset = 'UTF-8', $role = null)
+    public function __construct($title, $fileName = 'export', $params = [], $charset = 'UTF-8', $role = null)
     {
         $this->title = $title;
         $this->fileName = $fileName;
         $this->params = $params;
         $this->charset = $charset;
         $this->role = $role;
-        // $this->twig = $twig;
     }
 
     /**
@@ -82,14 +81,12 @@ public function setContainer(ContainerInterface $container = null)
     {
         $this->container = $container;
 
-
-
         return $this;
     }
 
     public function setTwig(Environment $twig) 
     {
-        $this->twig=$twig;
+        $this->twig = $twig;
     }
     /**
      * gets the Container associated with this Controller.

Grid/Export/JSONExport.php

@@ -12,6 +12,8 @@
 
 namespace APY\DataGridBundle\Grid\Export;
 
+use APY\DataGridBundle\Grid\Grid;
+
 /**
  * JSON.
  */

Grid/Export/PHPExcel5Export.php

@@ -12,6 +12,8 @@
 
 namespace APY\DataGridBundle\Grid\Export;
 
+use APY\DataGridBundle\Grid\Grid;
+
 /**
  * PHPExcel 5 Export (97-2003) (.xls)
  * 52 columns maximum.

Grid/Export/XMLExport.php

@@ -12,6 +12,7 @@
 
 namespace APY\DataGridBundle\Grid\Export;
 
+use APY\DataGridBundle\Grid\Grid;
 use Symfony\Component\Serializer\Encoder\XmlEncoder;
 use Symfony\Component\Serializer\Normalizer\GetSetMethodNormalizer;
 use Symfony\Component\Serializer\Serializer;

Grid/Grid.php

@@ -65,7 +65,7 @@ class Grid implements GridInterface
     public const TWEAK_NOT_DEFINED_EX_MSG = 'Tweak %s is not defined.';
 
     /**
-     * @var \Symfony\Component\DependencyInjection\Container
+     * @var Container
      */
     protected $container;
 
@@ -80,11 +80,10 @@ class Grid implements GridInterface
     protected $session;
 
     /**
-     * @var \Symfony\Component\HttpFoundation\Request
+     * @var Request
      */
     protected $request;
 
-
     protected AuthorizationCheckerInterface $securityContext;
 
     protected Environment $twig;
@@ -109,9 +108,6 @@ class Grid implements GridInterface
      */
     protected $routeParameters;
 
-    /**
-     * @var \APY\DataGridBundle\Grid\Source\Source
-     */
     protected ?Source $source = null;
 
     /**
@@ -140,22 +136,22 @@ class Grid implements GridInterface
     protected $limits = [];
 
     /**
-     * @var \APY\DataGridBundle\Grid\Columns|\APY\DataGridBundle\Grid\Column\Column[]
+     * @var Columns|Column[]
      */
     protected $columns;
 
     /**
-     * @var \APY\DataGridBundle\Grid\Rows
+     * @var Rows
      */
     protected $rows;
 
     /**
-     * @var \APY\DataGridBundle\Grid\Action\MassAction[]
+     * @var Action\MassAction[]
      */
     protected $massActions = [];
 
     /**
-     * @var \APY\DataGridBundle\Grid\Action\RowAction[]
+     * @var Action\RowAction[]
      */
     protected $rowActions = [];
 
@@ -205,7 +201,7 @@ class Grid implements GridInterface
     protected $noResultMessage;
 
     /**
-     * @var \APY\DataGridBundle\Grid\Export\Export[]
+     * @var Export[]
      */
     protected $exports = [];
 

Grid/GridBuilder.php

@@ -19,15 +19,16 @@ class GridBuilder extends GridConfigBuilder implements GridBuilderInterface
     /**
      * The container.
      */
-    private \Symfony\Component\DependencyInjection\Container $container;
+    private Container $container;
 
     private AuthorizationCheckerInterface $securityContext;
 
     private Environment $twig;
+
     /**
      * The factory.
      */
-    private \APY\DataGridBundle\Grid\GridFactoryInterface $factory;
+    private GridFactoryInterface $factory;
 
     /**
      * Columns of the grid builder.

Grid/GridFactory.php

@@ -20,13 +20,13 @@ class GridFactory implements GridFactoryInterface
     /**
      * The service container.
      */
-    private \Symfony\Component\DependencyInjection\Container $container;
+    private Container $container;
 
     private AuthorizationCheckerInterface $securityContext;
 
     private Environment $twig;
 
-    private \APY\DataGridBundle\Grid\GridRegistryInterface $registry;
+    private GridRegistryInterface $registry;
 
     /**
      * Constructor.

Grid/GridManager.php

@@ -20,7 +20,6 @@
 use Symfony\Component\HttpFoundation\Response;
 use Twig\Environment;
 
-
 class GridManager implements IteratorAggregate, Countable
 {
     protected $container;

Grid/Helper/ColumnsIterator.php

@@ -28,7 +28,7 @@ public function __construct(\Iterator $iterator, $showOnlySourceColumns)
         $this->showOnlySourceColumns = $showOnlySourceColumns;
     }
 
-    public function accept():bool
+    public function accept(): bool
     {
         $current = $this->getInnerIterator()->current();
 

Grid/Mapping/Metadata/DriverHeap.php

@@ -21,7 +21,7 @@ class DriverHeap extends \SplPriorityQueue
      *
      * @see SplPriorityQueue::compare()
      */
-    public function compare($priority1, $priority2):int
+    public function compare($priority1, $priority2): int
     {
         if ($priority1 === $priority2) {
             return 0;

Grid/Rows.php

@@ -58,7 +58,7 @@ public function addRow(Row $row)
      *
      * @see Countable::count()
      */
-    public function count():int
+    public function count(): int
     {
         return $this->rows->count();
     }