Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

remove surplus call deep namespace #3473

Closed
wants to merge 2 commits into
from
Jump to file or symbol
Failed to load files and symbols.
+3 −3
Split
@@ -72,12 +72,12 @@ class Catchall implements RouteInterface
protected $assembledParams = array();
/**
- * @var \Zend\Validator\ValidatorChain
+ * @var ValidatorChain
*/
protected $validators;
/**
- * @var \Zend\Filter\FilterChain
+ * @var FilterChain
*/
protected $filters;
@@ -25,7 +25,7 @@ class ControllerLoaderFactory implements FactoryInterface
/**
* Create the controller loader service
*
- * Creates and returns an instance of Controller\ControllerManager. The
+ * Creates and returns an instance of ControllerManager. The
@Maks3w

Maks3w Jan 19, 2013

Member

I prefer to have the entire name to avoid conflicts between two namespaces with the same class name

@samsonasik

samsonasik Jan 19, 2013

Contributor

@Maks3w : should I change to \Zend\Mvc\Controller\ControllerManager ?

@samsonasik

samsonasik Jan 21, 2013

Contributor

@Maks3w : in use statement, it already defined as 'use Zend\Mvc\Controller\ControllerManager', cmiiw.

@weierophinney

weierophinney Jan 21, 2013

Owner

It's correct in here as-is, as the ControllerManager class is imported into the current file.

* only controllers this manager will allow are those defined in the
* application configuration's "controllers" array. If a controller is
* matched, the scoped manager will attempt to load the controller.