Clearing deprecation notices

Author: pmishev

src/CacheWarmer/MetadataCacheWarmer.php

@@ -16,8 +16,8 @@ public function isOptional(): bool
         return true;
     }
 
-    public function warmUp(string $cacheDir): array
+    public function warmUp(string $cacheDir, ?string $buildDir = null): array
     {
-        return $this->metadataCollector->warmUp($cacheDir);
+        return $this->metadataCollector->warmUp($cacheDir, $buildDir);
     }
 }

src/Command/IndexBuildCommand.php

@@ -3,6 +3,7 @@
 namespace Sineflow\ElasticsearchBundle\Command;
 
 use Sineflow\ElasticsearchBundle\Manager\IndexManagerRegistry;
+use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -12,10 +13,9 @@
 /**
  * Command for (re)building elasticsearch index.
  */
+#[AsCommand('sineflow:es:index:build', '(Re)builds elasticsearch index.')]
 class IndexBuildCommand extends Command
 {
-    protected static $defaultName = 'sineflow:es:index:build';
-
     public function __construct(private readonly IndexManagerRegistry $indexManagerRegistry)
     {
         parent::__construct();
@@ -29,7 +29,6 @@ protected function configure(): void
         parent::configure();
 
         $this
-            ->setDescription('(Re)builds elasticsearch index.')
             ->addArgument(
                 'index',
                 InputArgument::REQUIRED,

src/Command/IndexCreateCommand.php

@@ -3,6 +3,7 @@
 namespace Sineflow\ElasticsearchBundle\Command;
 
 use Sineflow\ElasticsearchBundle\Manager\IndexManagerRegistry;
+use Symfony\Component\Console\Attribute\AsCommand;
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;
@@ -11,10 +12,9 @@
 /**
  * Command for creating elasticsearch index.
  */
+#[AsCommand('sineflow:es:index:create', 'Creates elasticsearch index.')]
 class IndexCreateCommand extends Command
 {
-    protected static $defaultName = 'sineflow:es:index:create';
-
     public function __construct(private readonly IndexManagerRegistry $indexManagerRegistry)
     {
         parent::__construct();
@@ -28,7 +28,6 @@ protected function configure(): void
         parent::configure();
 
         $this
-            ->setDescription('Creates elasticsearch index.')
             ->addArgument(
                 'index',
                 InputArgument::REQUIRED,

src/Mapping/DocumentMetadataCollector.php

@@ -57,7 +57,7 @@ public function __construct(
      *
      * @throws InvalidArgumentException
      */
-    public function warmUp(string $cacheDir): array
+    public function warmUp(string $cacheDir, ?string $buildDir = null): array
     {
         // force cache generation
         foreach ($this->documentClassToIndexManagerNames as $documentClass => $indexManagerName) {

src/SineflowElasticsearchBundle.php

@@ -16,7 +16,7 @@ class SineflowElasticsearchBundle extends Bundle
     /**
      * {@inheritdoc}
      */
-    public function build(ContainerBuilder $container)
+    public function build(ContainerBuilder $container): void
     {
         parent::build($container);
 

tests/AbstractContainerAwareTestCase.php

@@ -22,10 +22,7 @@ abstract class AbstractContainerAwareTestCase extends KernelTestCase
  */
 abstract class AbstractContainerAwareTestCase extends KernelTestCase
 {
-    /**
-     * @var ContainerInterface
-     */
-    private $cachedContainer;
+    private ?ContainerInterface $cachedContainer;
 
     /**
      * {@inheritDoc}

tests/AbstractElasticsearchTestCase.php

@@ -32,10 +32,8 @@ abstract class AbstractElasticsearchTestCase extends AbstractContainerAwareTestC
      *              ]
      *          ]
      *      ]
-     *
-     * @return array
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [];
     }

tests/Functional/Document/Provider/ElasticsearchProviderTest.php

@@ -11,7 +11,7 @@ class ElasticsearchProviderTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Document/Provider/ProviderRegistryTest.php

@@ -14,15 +14,12 @@
 
 class ProviderRegistryTest extends AbstractElasticsearchTestCase
 {
-    /**
-     * @var ProviderRegistry
-     */
-    private $providerRegistry;
+    private ProviderRegistry $providerRegistry;
 
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Document/RepositoryTest.php

@@ -13,30 +13,18 @@ class RepositoryTest extends AbstractElasticsearchTestCase
 {
     use AssertThrows;
 
-    /**
-     * @var Repository
-     */
-    private $repository;
+    private Repository $repository;
 
-    /**
-     * @var IndexManager
-     */
-    private $indexManager;
+    private IndexManager $indexManager;
 
-    /**
-     * @var Finder
-     */
-    private $finder;
+    private Finder $finder;
 
-    /**
-     * @var DocumentMetadataCollector
-     */
-    private $metadataCollector;
+    private DocumentMetadataCollector $metadataCollector;
 
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Finder/Adapter/KnpPaginatorAdapterTest.php

@@ -15,7 +15,7 @@ class KnpPaginatorAdapterTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Finder/Adapter/ScrollAdapterTest.php

@@ -13,7 +13,7 @@ class ScrollAdapterTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Finder/FinderTest.php

@@ -21,7 +21,7 @@ class FinderTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Manager/IndexManagerTest.php

@@ -25,7 +25,7 @@ class IndexManagerTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Functional/Profiler/ElasticsearchProfilerTest.php

@@ -16,7 +16,7 @@ class ElasticsearchProfilerTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'default' => [

tests/Functional/Result/DocumentIteratorTest.php

@@ -18,7 +18,7 @@ class DocumentIteratorTest extends AbstractElasticsearchTestCase
     /**
      * {@inheritdoc}
      */
-    protected function getDataArray()
+    protected function getDataArray(): array
     {
         return [
             'bar' => [

tests/Unit/Annotation/DocumentTest.php

@@ -17,7 +17,7 @@ public function testDump(): void
     {
         $doc = new Document();
 
-        $doc->type = 'product';
+        $doc->repositoryClass = 'some class name';
         $doc->options = [
             'dynamic' => 'strict',
             'foo'     => 'bar',

tests/Unit/Annotation/PropertyTest.php

@@ -27,7 +27,7 @@ public function testDump(): void
             'analyzer' => 'standard',
             'foo'      => 'bar',
         ];
-        $type->foo = 'bar';
+        $type->enumType = 'bar';
 
         $this->assertEquals(
             [