From 9434cc8f40b6815ce39c6e51b91fd701663bdc6f Mon Sep 17 00:00:00 2001 From: Alexander Makarov Date: Sat, 19 Oct 2013 13:30:31 +0400 Subject: [PATCH] Moved Application::$controllerNamespace init into variable declaration, reordered Module::__construct. --- framework/yii/base/Application.php | 11 +++++++---- framework/yii/base/Module.php | 3 +-- 2 files changed, 8 insertions(+), 6 deletions(-) diff --git a/framework/yii/base/Application.php b/framework/yii/base/Application.php index 5de7969..c871548 100644 --- a/framework/yii/base/Application.php +++ b/framework/yii/base/Application.php @@ -54,6 +54,13 @@ abstract class Application extends Module * @event ActionEvent an event raised after executing a controller action. */ const EVENT_AFTER_ACTION = 'afterAction'; + + /** + * @var string the namespace that controller classes are in. If not set, + * it will use the "app\controllers" namespace. + */ + public $controllerNamespace = 'app\\controllers'; + /** * @var string the application name. */ @@ -140,10 +147,6 @@ abstract class Application extends Module $this->registerCoreComponents(); Component::__construct($config); - - if ($this->controllerNamespace === null) { - $this->controllerNamespace = 'app\\controllers'; - } } /** diff --git a/framework/yii/base/Module.php b/framework/yii/base/Module.php index 0e5c592..996d0f2 100644 --- a/framework/yii/base/Module.php +++ b/framework/yii/base/Module.php @@ -83,7 +83,6 @@ abstract class Module extends Component * it will use the "controllers" sub-namespace under the namespace of this module. * For example, if the namespace of this module is "foo\bar", then the default * controller namespace would be "foo\bar\controllers". - * If the module is an application, it will default to "app\controllers". */ public $controllerNamespace; /** @@ -127,9 +126,9 @@ abstract class Module extends Component */ public function __construct($id, $parent = null, $config = []) { + parent::__construct($config); $this->id = $id; $this->module = $parent; - parent::__construct($config); if ($this->controllerNamespace === null) { $class = get_class($this); if (($pos = strrpos($class, '\\')) !== false) {