CacheWarmers

Author: ProklUng

DependencyInjection/CompilerPass/WarmersConfiguratorCompilerPass.php

@@ -38,9 +38,9 @@ public function process(ContainerBuilder $container)
         }
 
         // Нет в контейнере параметра warming_pages - прогрев только главной страницы.
-        if (!$container->hasParameter('warming_pages')) {
+        if ($container->hasParameter('warming_pages')) {
             $definition = $container->getDefinition('bitrix_ordinary_tools.bitrix_page_cache_warmer');
-            $definition->replaceArgument(2, ['/']);
+            $definition->replaceArgument(2, $container->getParameter('warming_pages'));
         }
    }
 }

Resources/config/warmers.yaml

@@ -23,6 +23,6 @@ services:
     arguments:
       - '@http_client'
       - '%kernel.site.host%'
-      - ['%warming_pages%']
+      - ['/']
     class: Prokl\BitrixOrdinaryToolsBundle\Services\CacheWarmer\BitrixCacheWarmer
     tags: [kernel.cache_warmer]

Tests/Cases/CompilerPass/WarmersConfiguratorCompilerPassTest.php

@@ -87,7 +87,42 @@ public function testNoPageParameters() : void
         $newDefinition = $this->container->getDefinition('bitrix_ordinary_tools.bitrix_page_cache_warmer');
         $newParam = $newDefinition->getArguments();
 
-        $this->assertSame(['/'], $newParam[2]);
+        $this->assertSame(['/xxx/'], $newParam[2]);
+    }
+
+    /**
+     * Задан параметр warming_pages.
+     *
+     * @return void
+     */
+    public function testHasPageParameters() : void
+    {
+        $this->container->setDefinition(
+            'http_client',
+            new Definition(get_class($this->fakeService))
+        );
+
+        $this->container->setParameter(
+            'warming_pages',
+            ['/page/']
+        );
+
+        $definition = new Definition(get_class($this->fakeService));
+        $definition->addArgument($this->container->getDefinition('http_client'));
+        $definition->addArgument('localhost');
+        $definition->addArgument(['/xxx/']);
+
+        $this->container->setDefinition(
+            'bitrix_ordinary_tools.bitrix_page_cache_warmer',
+            $definition
+        );
+
+        $this->obTestObject->process($this->container);
+
+        $newDefinition = $this->container->getDefinition('bitrix_ordinary_tools.bitrix_page_cache_warmer');
+        $newParam = $newDefinition->getArguments();
+
+        $this->assertSame(['/page/'], $newParam[2]);
     }
 
     /**