EZP-31544: changed $configResolver from private to protected and chan…

…ged variable (#39)
ezsystems · Apr 8, 2020 · f344942 · f344942
1 parent c379931
commit f344942
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/eZ/Bundle/EzPublishCoreBundle/URLChecker/Handler/AbstractConfigResolverBasedURLHandler.php b/eZ/Bundle/EzPublishCoreBundle/URLChecker/Handler/AbstractConfigResolverBasedURLHandler.php
@@ -15,7 +15,7 @@
 abstract class AbstractConfigResolverBasedURLHandler extends AbstractURLHandler
 {
     /** @var \eZ\Publish\Core\MVC\ConfigResolverInterface */
-    private $configResolver;
+    protected $configResolver;
 
     /** @var string */
     private $parameterName;

diff --git a/eZ/Bundle/EzPublishCoreBundle/URLChecker/Handler/HTTPHandler.php b/eZ/Bundle/EzPublishCoreBundle/URLChecker/Handler/HTTPHandler.php
@@ -28,7 +28,7 @@ public function validate(array $urls)
         $handlers = [];
 
         // Batch size can't be larger then number of urls
-        $batchSize = min(count($urls), $this->options['batch_size']);
+        $batchSize = min(count($urls), $options['batch_size']);
         for ($i = 0; $i < $batchSize; ++$i) {
             curl_multi_add_handle(
                 $master,
@@ -116,6 +116,7 @@ public function getOptions(): array
      */
     private function createCurlHandlerForUrl(URL $url, array &$handlers, int $connectionTimeout, int $timeout)
     {
+        $options = $this->getOptions();
         $handler = curl_init();
 
         curl_setopt_array($handler, [
@@ -128,7 +129,7 @@ private function createCurlHandlerForUrl(URL $url, array &$handlers, int $connec
             CURLOPT_NOBODY => true,
         ]);
 
-        if ($this->options['ignore_certificate']) {
+        if ($options['ignore_certificate']) {
             curl_setopt_array($handler, [
                 CURLOPT_SSL_VERIFYPEER => false,
                 CURLOPT_SSL_VERIFYHOST => false,