Updating Scaffold to work with any single prefix. You can set $scaffold = to any prefix defined in Routing.prefixes.

Tests added.
This commit is contained in:
mark_story 2009-10-31 16:06:20 -04:00
parent 15a98fac5b
commit 3a59bf5ed6
2 changed files with 80 additions and 24 deletions

View file

@ -434,10 +434,7 @@ class Scaffold extends Object {
/** /**
* When methods are now present in a controller * When methods are now present in a controller
* scaffoldView is used to call default Scaffold methods if: * scaffoldView is used to call default Scaffold methods if:
* <code> * `var $scaffold;` is placed in the controller's class definition.
* var $scaffold;
* </code>
* is placed in the controller's class definition.
* *
* @param array $params Parameters for scaffolding * @param array $params Parameters for scaffolding
* @return mixed A rendered view of scaffold action, or showing the error * @return mixed A rendered view of scaffold action, or showing the error
@ -445,23 +442,30 @@ class Scaffold extends Object {
*/ */
function __scaffold($params) { function __scaffold($params) {
$db = &ConnectionManager::getDataSource($this->ScaffoldModel->useDbConfig); $db = &ConnectionManager::getDataSource($this->ScaffoldModel->useDbConfig);
$admin = Configure::read('Routing.admin'); $prefixes = Configure::read('Routing.prefixes');
$scaffoldPrefix = $this->scaffoldActions;
if (isset($db)) { if (isset($db)) {
if (empty($this->scaffoldActions)) { if (empty($this->scaffoldActions)) {
$this->scaffoldActions = array( $this->scaffoldActions = array(
'index', 'list', 'view', 'add', 'create', 'edit', 'update', 'delete' 'index', 'list', 'view', 'add', 'create', 'edit', 'update', 'delete'
); );
} elseif (!empty($admin) && $this->scaffoldActions === $admin) { } elseif (!empty($prefixes) && in_array($this->scaffoldActions, $prefixes)) {
$this->scaffoldActions = array( $this->scaffoldActions = array(
$admin .'_index', $admin .'_list', $admin .'_view', $admin .'_add', $scaffoldPrefix . '_index',
$admin .'_create', $admin .'_edit', $admin .'_update', $admin .'_delete' $scaffoldPrefix . '_list',
$scaffoldPrefix . '_view',
$scaffoldPrefix . '_add',
$scaffoldPrefix . '_create',
$scaffoldPrefix . '_edit',
$scaffoldPrefix . '_update',
$scaffoldPrefix . '_delete'
); );
} }
if (in_array($params['action'], $this->scaffoldActions)) { if (in_array($params['action'], $this->scaffoldActions)) {
if (!empty($admin)) { if (!empty($prefixes)) {
$params['action'] = str_replace($admin . '_', '', $params['action']); $params['action'] = str_replace($scaffoldPrefix . '_', '', $params['action']);
} }
switch ($params['action']) { switch ($params['action']) {
case 'index': case 'index':
@ -556,10 +560,15 @@ class ScaffoldView extends ThemeView {
$name = $this->action; $name = $this->action;
} }
$name = Inflector::underscore($name); $name = Inflector::underscore($name);
$admin = Configure::read('Routing.admin'); $prefixes = Configure::read('Routing.prefixes');
if (!empty($admin) && strpos($name, $admin . '_') !== false) { if (!empty($prefixes)) {
$name = substr($name, strlen($admin) + 1); foreach ($prefixes as $prefix) {
if (strpos($name, $prefix . '_') !== false) {
$name = substr($name, strlen($prefix) + 1);
break;
}
}
} }
if ($name === 'add') { if ($name === 'add') {

View file

@ -315,8 +315,8 @@ class ScaffoldViewTest extends CakeTestCase {
* @return void * @return void
*/ */
function testGetViewFilename() { function testGetViewFilename() {
$_admin = Configure::read('Routing.admin'); $_admin = Configure::read('Routing.prefixes');
Configure::write('Routing.admin', 'admin'); Configure::write('Routing.prefixes', array('admin'));
$this->Controller->action = 'index'; $this->Controller->action = 'index';
$ScaffoldView =& new TestScaffoldView($this->Controller); $ScaffoldView =& new TestScaffoldView($this->Controller);
@ -385,7 +385,7 @@ class ScaffoldViewTest extends CakeTestCase {
. DS .'test_plugin' . DS . 'views' . DS . 'tests' . DS . 'scaffold.edit.ctp'; . DS .'test_plugin' . DS . 'views' . DS . 'tests' . DS . 'scaffold.edit.ctp';
$this->assertEqual($result, $expected); $this->assertEqual($result, $expected);
Configure::write('Routing.admin', $_admin); Configure::write('Routing.prefixes', $_admin);
} }
/** /**
@ -499,7 +499,7 @@ class ScaffoldViewTest extends CakeTestCase {
new Scaffold($this->Controller, $params); new Scaffold($this->Controller, $params);
$result = ob_get_clean(); $result = ob_get_clean();
$this->assertPattern('/<form id="ScaffoldMockEditForm" method="post" action="\/scaffold_mock\/edit\/1">/', $result); $this->assertPattern('/<form id="ScaffoldMockEditForm" method="post" action="\/scaffold_mock\/edit\/1"/', $result);
$this->assertPattern('/<legend>Edit Scaffold Mock<\/legend>/', $result); $this->assertPattern('/<legend>Edit Scaffold Mock<\/legend>/', $result);
$this->assertPattern('/input type="hidden" name="data\[ScaffoldMock\]\[id\]" value="1" id="ScaffoldMockId"/', $result); $this->assertPattern('/input type="hidden" name="data\[ScaffoldMock\]\[id\]" value="1" id="ScaffoldMockId"/', $result);
@ -509,6 +509,7 @@ class ScaffoldViewTest extends CakeTestCase {
$this->assertPattern('/textarea name="data\[ScaffoldMock\]\[body\]" cols="30" rows="6" id="ScaffoldMockBody"/', $result); $this->assertPattern('/textarea name="data\[ScaffoldMock\]\[body\]" cols="30" rows="6" id="ScaffoldMockBody"/', $result);
$this->assertPattern('/<li><a href="\/scaffold_mock\/delete\/1"[^>]*>Delete<\/a>\s*<\/li>/', $result); $this->assertPattern('/<li><a href="\/scaffold_mock\/delete\/1"[^>]*>Delete<\/a>\s*<\/li>/', $result);
} }
/** /**
* Test Admin Index Scaffolding. * Test Admin Index Scaffolding.
* *
@ -516,9 +517,9 @@ class ScaffoldViewTest extends CakeTestCase {
* @return void * @return void
**/ **/
function testAdminIndexScaffold() { function testAdminIndexScaffold() {
$_backAdmin = Configure::read('Routing.admin'); $_backAdmin = Configure::read('Routing.prefixes');
Configure::write('Routing.admin', 'admin'); Configure::write('Routing.prefixes', array('admin'));
$params = array( $params = array(
'plugin' => null, 'plugin' => null,
'pass' => array(), 'pass' => array(),
@ -551,7 +552,7 @@ class ScaffoldViewTest extends CakeTestCase {
//TODO: add testing for table generation //TODO: add testing for table generation
$this->assertPattern('/<li><a href="\/admin\/scaffold_mock\/add\/">New Scaffold Mock<\/a><\/li>/', $result); $this->assertPattern('/<li><a href="\/admin\/scaffold_mock\/add\/">New Scaffold Mock<\/a><\/li>/', $result);
Configure::write('Routing.admin', $_backAdmin); Configure::write('Routing.prefixes', $_backAdmin);
} }
/** /**
@ -561,9 +562,9 @@ class ScaffoldViewTest extends CakeTestCase {
* @return void * @return void
**/ **/
function testAdminEditScaffold() { function testAdminEditScaffold() {
$_backAdmin = Configure::read('Routing.admin'); $_backAdmin = Configure::read('Routing.prefixes');
Configure::write('Routing.admin', 'admin'); Configure::write('Routing.prefixes', array('admin'));
$params = array( $params = array(
'plugin' => null, 'plugin' => null,
'pass' => array(), 'pass' => array(),
@ -594,8 +595,54 @@ class ScaffoldViewTest extends CakeTestCase {
$this->assertPattern('#admin/scaffold_mock/edit/1#', $result); $this->assertPattern('#admin/scaffold_mock/edit/1#', $result);
$this->assertPattern('#Scaffold Mock#', $result); $this->assertPattern('#Scaffold Mock#', $result);
Configure::write('Routing.admin', $_backAdmin); Configure::write('Routing.prefixes', $_backAdmin);
} }
/**
* Test Admin Index Scaffolding.
*
* @access public
* @return void
**/
function testMultiplePrefixScaffold() {
$_backAdmin = Configure::read('Routing.prefixes');
Configure::write('Routing.prefixes', array('admin', 'member'));
$params = array(
'plugin' => null,
'pass' => array(),
'form' => array(),
'named' => array(),
'prefix' => 'member',
'url' => array('url' =>'member/scaffold_mock'),
'controller' => 'scaffold_mock',
'action' => 'member_index',
'member' => 1,
);
//reset, and set router.
Router::reload();
Router::setRequestInfo(array($params, array('base' => '/', 'here' => '/member/scaffold_mock', 'webroot' => '/')));
$this->Controller->params = $params;
$this->Controller->controller = 'scaffold_mock';
$this->Controller->base = '/';
$this->Controller->action = 'member_index';
$this->Controller->here = '/tests/member/scaffold_mock';
$this->Controller->webroot = '/';
$this->Controller->scaffold = 'member';
$this->Controller->constructClasses();
ob_start();
$Scaffold = new Scaffold($this->Controller, $params);
$result = ob_get_clean();
$this->assertPattern('/<h2>Scaffold Mock<\/h2>/', $result);
$this->assertPattern('/<table cellpadding="0" cellspacing="0">/', $result);
//TODO: add testing for table generation
$this->assertPattern('/<li><a href="\/member\/scaffold_mock\/add\/">New Scaffold Mock<\/a><\/li>/', $result);
Configure::write('Routing.prefixes', $_backAdmin);
}
} }
/** /**