New cs fixer rule

Author: XWB

.php_cs.dist

@@ -22,6 +22,7 @@ return PhpCsFixer\Config::create()
         'ordered_imports' => true,
         'phpdoc_no_empty_return' => false,
         'ternary_to_null_coalescing' => true,
+        'nullable_type_declaration_for_default_null_value' => true,
     ])
     ->setUsingCache(true)
     ->setFinder(

src/DataCollector/ElasticaDataCollector.php

@@ -30,7 +30,7 @@ public function __construct(ElasticaLogger $logger)
         $this->logger = $logger;
     }
 
-    public function collect(Request $request, Response $response, \Throwable $exception = null)
+    public function collect(Request $request, Response $response, ?\Throwable $exception = null)
     {
         $this->data['nb_queries'] = $this->logger->getNbQueries();
         $this->data['queries'] = $this->logger->getQueries();

src/Doctrine/Listener.php

@@ -80,7 +80,7 @@ public function __construct(
         ObjectPersisterInterface $objectPersister,
         IndexableInterface $indexable,
         array $config = [],
-        LoggerInterface $logger = null
+        ?LoggerInterface $logger = null
     ) {
         $this->config = array_merge([
             'identifier' => 'id',

src/Doctrine/RepositoryManager.php

@@ -47,7 +47,7 @@ public function __construct(ManagerRegistry $managerRegistry, RepositoryManagerI
     /**
      * {@inheritdoc}
      */
-    public function addIndex(string $indexName, FinderInterface $finder, string $repositoryName = null): void
+    public function addIndex(string $indexName, FinderInterface $finder, ?string $repositoryName = null): void
     {
         throw new \LogicException(__METHOD__.' should not be called. Call addIndex on the main repository manager');
     }

src/Finder/FinderInterface.php

@@ -16,8 +16,8 @@ interface FinderInterface
     /**
      * Searches for query results within a given limit.
      *
-     * @param mixed    $query   Can be a string, an array or an \Elastica\Query object
-     * @param int|null $limit   How many results to get
+     * @param mixed    $query Can be a string, an array or an \Elastica\Query object
+     * @param int|null $limit How many results to get
      *
      * @return array results
      */

src/Finder/PaginatedFinderInterface.php

@@ -23,7 +23,7 @@ interface PaginatedFinderInterface extends FinderInterface
     /**
      * Searches for query results and returns them wrapped in a paginator.
      *
-     * @param mixed $query   Can be a string, an array or an \Elastica\Query object
+     * @param mixed $query Can be a string, an array or an \Elastica\Query object
      *
      * @return Pagerfanta paginated results
      */

src/Manager/RepositoryManagerInterface.php

@@ -26,7 +26,7 @@ interface RepositoryManagerInterface
      * Adds index name and its finder.
      * Custom repository class name can also be added.
      */
-    public function addIndex(string $indexName, FinderInterface $finder, string $repositoryName = null): void;
+    public function addIndex(string $indexName, FinderInterface $finder, ?string $repositoryName = null): void;
 
     /**
      * Return repository for entity.

tests/Unit/Doctrine/AbstractListenerTest.php

@@ -294,7 +294,7 @@ private function getMockPersister(Entity $object, $indexName)
         return $mock;
     }
 
-    private function getMockIndexable($indexName, Entity $object = null, $return = null)
+    private function getMockIndexable($indexName, ?Entity $object = null, $return = null)
     {
         $mock = $this->createMock(IndexableInterface::class);
 

tests/Unit/Elastica/ClientTest.php

@@ -94,7 +94,7 @@ private function getConnectionMock()
         return $connection;
     }
 
-    private function getClientMock(Response $response = null, $connection = null)
+    private function getClientMock(?Response $response = null, $connection = null)
     {
         $transport = new NullTransport();
         if ($response) {

tests/Unit/Subscriber/PaginateElasticaQuerySubscriberTest.php

@@ -295,7 +295,7 @@ protected function getResultSetMock()
         return $this->createMock(PartialResultsInterface::class);
     }
 
-    private function getRequestStack(Request $request = null)
+    private function getRequestStack(?Request $request = null)
     {
         $stack = new RequestStack();