Skip to content

Commit

Permalink
Adding constructor to Router.
Browse files Browse the repository at this point in the history
Adding __setPrefixes to initialize prefixes and merge Routing.admin with new Routing.prefixes.
  • Loading branch information
markstory committed Sep 27, 2009
1 parent 2f46f39 commit 3904c6f
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 3 deletions.
28 changes: 26 additions & 2 deletions cake/libs/router.php
Expand Up @@ -164,6 +164,31 @@ class Router {
*/
var $__defaultsMapped = false;

/**
* Constructor for Router.
* Builds __prefixes
*
* @return void
**/
function Router() {
$this->__setPrefixes();
}

/**
* Sets the Routing prefixes.
*
* @return void
* @access private
**/
function __setPrefixes() {
$routing = Configure::read('Routing');
if (isset($routing['admin'])) {
$this->__prefixes[] = $this->__admin = $routing['admin'];
}
if (isset($routing['prefixes'])) {
$this->__prefixes = array_merge($this->__prefixes, $routing['prefixes']);
}
}
/**
* Gets a reference to the Router object instance
*
Expand All @@ -176,7 +201,6 @@ function &getInstance() {

if (!$instance) {
$instance[0] =& new Router();
$instance[0]->__admin = Configure::read('Routing.admin');
}
return $instance[0];
}
Expand Down Expand Up @@ -722,7 +746,7 @@ function reload() {
foreach (get_class_vars('Router') as $key => $val) {
$_this->{$key} = $val;
}
$_this->__admin = Configure::read('Routing.admin');
$_this->__setPrefixes();
}

/**
Expand Down
2 changes: 1 addition & 1 deletion cake/tests/cases/libs/router.test.php
Expand Up @@ -1655,7 +1655,7 @@ function testPassedArgsOrder() {
$this->assertEqual($result, $expected);

$result = Router::prefixes();
$expected = array('protected', 'admin');
$expected = array('admin', 'protected');
$this->assertEqual($result, $expected);
}

Expand Down

0 comments on commit 3904c6f

Please sign in to comment.