Change getPools to protected (#77)

php-cache · Aug 7, 2016 · 83b9862 · 83b9862
1 parent bbc23d3
commit 83b9862
Showing 1 changed file with 34 additions and 34 deletions.
diff --git a/CachePoolChain.php b/CachePoolChain.php
@@ -58,28 +58,6 @@ public function __construct(array $pools, array $options = [])
         $this->options = $resolver->resolve($options);
     }
 
-    /**
-     * @param string             $poolKey
-     * @param string             $operation
-     * @param CachePoolException $exception
-     *
-     * @throws PoolFailedException
-     */
-    private function handleException($poolKey, $operation, CachePoolException $exception)
-    {
-        if (!$this->options['skip_on_failure']) {
-            throw $exception;
-        }
-
-        $this->log(
-            'warning',
-            sprintf('Removing pool "%s" from chain because it threw an exception when executing "%s"', $poolKey, $operation),
-            ['exception' => $exception]
-        );
-
-        unset($this->pools[$poolKey]);
-    }
-
     /**
      * @param OptionsResolver $resolver
      */
@@ -90,18 +68,6 @@ public function configureOptions(OptionsResolver $resolver)
         ]);
     }
 
-    /**
-     * @return array|\Psr\Cache\CacheItemPoolInterface[]
-     */
-    public function getPools()
-    {
-        if (empty($this->pools)) {
-            throw new NoPoolAvailableException('No valid cache pool available for the chain.');
-        }
-
-        return $this->pools;
-    }
-
     /**
      * {@inheritdoc}
      */
@@ -328,4 +294,38 @@ protected function log($level, $message, array $context = [])
             $this->logger->log($level, $message, $context);
         }
     }
+
+    /**
+     * @return array|\Psr\Cache\CacheItemPoolInterface[]
+     */
+    protected function getPools()
+    {
+        if (empty($this->pools)) {
+            throw new NoPoolAvailableException('No valid cache pool available for the chain.');
+        }
+
+        return $this->pools;
+    }
+
+    /**
+     * @param string             $poolKey
+     * @param string             $operation
+     * @param CachePoolException $exception
+     *
+     * @throws PoolFailedException
+     */
+    private function handleException($poolKey, $operation, CachePoolException $exception)
+    {
+        if (!$this->options['skip_on_failure']) {
+            throw $exception;
+        }
+
+        $this->log(
+            'warning',
+            sprintf('Removing pool "%s" from chain because it threw an exception when executing "%s"', $poolKey, $operation),
+            ['exception' => $exception]
+        );
+
+        unset($this->pools[$poolKey]);
+    }
 }