fix for ContainerInterface is null

yupmin · May 31, 2019 · 7bf6fb4 · 7bf6fb4
1 parent 8f06680
commit 7bf6fb4
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/library/Odesk/Phystrix/AbstractCommand.php b/library/Odesk/Phystrix/AbstractCommand.php
@@ -64,7 +64,7 @@ abstract class AbstractCommand
     private $commandMetricsFactory;
 
     /**
-     * @var ContainerInterface
+     * @var ContainerInterface|null
      */
     protected $container;
 
@@ -292,9 +292,9 @@ protected function processExecutionEvent($eventName)
     /**
      * Sets service container instance, for injecting custom dependencies into the command
      *
-     * @param ContainerInterface $container
+     * @param ContainerInterface|null $container
      */
-    public function setContainer(ContainerInterface $container)
+    public function setContainer(ContainerInterface $container = null)
     {
         $this->container = $container;
     }

diff --git a/library/Odesk/Phystrix/CommandFactory.php b/library/Odesk/Phystrix/CommandFactory.php
@@ -63,12 +63,11 @@ class CommandFactory
      * Constructor
      *
      * @param Config $config
-     * @param LocatorInterface $serviceLocator
      * @param CircuitBreakerFactory $circuitBreakerFactory
      * @param CommandMetricsFactory $commandMetricsFactory
-     * @param RequestCache $requestCache
-     * @param RequestLog $requestLog
-     * @param ContainerInterface $container
+     * @param RequestCache|null $requestCache
+     * @param RequestLog|null $requestLog
+     * @param ContainerInterface|null $container
      */
     public function __construct(
         Config $config,