Fixing Scaffold in 1.2.x.x when setting Controller::persistModel = true;

Corrected error message with correct method name in scaffold_error templates.
Removed unneeded check for Controller::persistModel in 1.2.x.x.


git-svn-id: https://svn.cakephp.org/repo/branches/1.2.x.x@4038 3807eeeb-6ff5-0310-8944-8be069107fe0
This commit is contained in:
phpnut 2006-12-01 00:48:00 +00:00
parent 83afee9a52
commit 7b6e81c1e2
3 changed files with 158 additions and 207 deletions

View file

@ -308,9 +308,6 @@ class Controller extends Object {
$cached = false;
$object = null;
if ($this->persistModel === true){
uses('neat_array');
}
if($this->uses === false) {
if(!class_exists($this->modelClass)){
loadModel($this->modelClass);

View file

@ -1,6 +1,5 @@
<?php
/* SVN FILE: $Id$ */
/**
* Scaffold.
*
@ -27,7 +26,6 @@
* @lastmodified $Date$
* @license http://www.opensource.org/licenses/mit-license.php The MIT License
*/
/**
* Scaffolding is a set of automatic views, forms and controllers for starting web development work faster.
*
@ -40,14 +38,12 @@
* @subpackage cake.cake.libs.controller
*/
class Scaffold extends Object {
/**
* Controller object
*
* @var Controller
*/
var $controller = null;
/**
* Name of controller to scaffold
*
@ -60,35 +56,30 @@ class Scaffold extends Object {
* @var string
*/
var $action = null;
/**
* Class name of model
*
* @var unknown_type
*/
var $modelClass = null;
/**
* Registry key of model
*
* @var string
*/
var $modelKey = null;
/**
* View path for scaffolded controller
*
* @var string
*/
var $viewPath = null;
/**
* Title HTML element for current scaffolded view
*
* @var string
*/
var $scaffoldTitle = null;
/**
* Construct and set up given controller with given parameters.
*
@ -99,9 +90,11 @@ class Scaffold extends Object {
$this->controller = &$controller;
$this->name = $controller->name;
$this->action = $controller->action;
if (!in_array('Form', $this->controller->helpers)) {
$this->controller->helpers[] = 'Form';
}
if($this->controller->constructClasses() === false) {
return $this->cakeError('missingModel', array(array('className' => $this->modelKey, 'webroot' => '', 'base' => $this->controller->base)));
}
@ -110,7 +103,6 @@ class Scaffold extends Object {
$controller->modelClass = $controller->uses[0];
$controller->modelKey = Inflector::underscore($controller->modelClass);
}
$this->modelClass = $controller->modelClass;
$this->modelKey = $controller->modelKey;
@ -118,17 +110,11 @@ class Scaffold extends Object {
return $this->cakeError('missingModel', array(array('className' => $this->modelClass, 'webroot' => '', 'base' => $controller->base)));
}
$this->ScaffoldModel =& $this->controller->{$this->modelClass};
$this->viewPath = Inflector::underscore($this->name);
$this->scaffoldTitle = Inflector::humanize($this->viewPath);
$this->scaffoldActions = $controller->scaffold;
$this->controller->pageTitle= 'Scaffold :: ' . Inflector::humanize($this->action) . ' :: ' . $this->scaffoldTitle;
$path = '/';
/*if(is_null($controller->plugin)) {
$path = '/';
} else {
$path = '/'.$controller->plugin.'/';
}*/
$this->controller->set('path', $path);
$this->controller->set('controllerName', $this->name);
$this->controller->set('controllerAction', $this->action);
@ -137,19 +123,16 @@ class Scaffold extends Object {
$this->controller->set('viewPath', $this->viewPath);
$this->controller->set('humanSingularName', Inflector::humanize($this->modelKey));
$this->controller->set('humanPluralName', Inflector::humanize($this->viewPath));
$alias = null;
if(!empty($this->ScaffoldModel->alias)) {
$alias = array_combine(array_keys($this->ScaffoldModel->alias), array_keys($this->ScaffoldModel->alias));
}
$this->controller->set('alias', $alias);
$this->controller->set('primaryKey', $this->ScaffoldModel->primaryKey);
$this->controller->set('displayField', $this->ScaffoldModel->getDisplayfield());
$this->__scaffold($params);
}
/**
* Renders a view view of scaffolded Model.
*
@ -159,38 +142,35 @@ class Scaffold extends Object {
*/
function __scaffoldView($params) {
if ($this->controller->_beforeScaffold('view')) {
$this->controller->data = $this->ScaffoldModel->read();
if(isset($params['pass'][0])){
$this->controller->{$this->modelKey}->id = $params['pass'][0];
$this->ScaffoldModel->id = $params['pass'][0];
} elseif (isset($this->controller->Session) && $this->controller->Session->valid != false) {
$this->controller->Session->setFlash('No id set for ' . Inflector::humanize($this->modelKey) . '::view().');
$this->controller->redirect('/' . Inflector::underscore($this->controller->viewPath));
} else {
return $this->controller->flash('No id set for ' . Inflector::humanize($this->modelKey) . '::view().',
'/' . Inflector::underscore($this->controller->viewPath));
}
$this->controller->data = $this->ScaffoldModel->read();
$this->controller->set('data', $this->controller->data);
$this->controller->set('fieldNames',
$this->controller->generateFieldNames($this->controller->data, false));
$this->controller->set('fieldNames', $this->controller->generateFieldNames($this->controller->data, false));
if (file_exists(APP . 'views' . DS . $this->viewPath . DS . 'scaffold.view.thtml')) {
return $this->controller->render($this->action, '', APP . 'views' . DS . $this->viewPath
. DS . 'scaffold.view.thtml');
return $this->controller->render($this->action, '',
APP . 'views' . DS . $this->viewPath . DS . 'scaffold.view.thtml');
} elseif (file_exists(APP . 'views' . DS . 'scaffold' . DS . 'scaffold.view.thtml')) {
return $this->controller->render($this->action, '', APP . 'views' . DS . 'scaffold'
. DS . 'scaffold.view.thtml');
return $this->controller->render($this->action, '',
APP . 'views' . DS . 'scaffold' . DS . 'scaffold.view.thtml');
} else {
return $this->controller->render($this->action, '', LIBS . 'view' . DS . 'templates'
. DS . 'scaffolds' . DS . 'view.thtml');
return $this->controller->render($this->action, '',
LIBS . 'view' . DS . 'templates' . DS . 'scaffolds' . DS . 'view.thtml');
}
} elseif ($this->controller->_scaffoldError('view') === false) {
return $this->__scaffoldError();
}
}
/**
* Renders List view of scaffolded Model.
*
@ -199,7 +179,6 @@ class Scaffold extends Object {
* @access private
*/
function __scaffoldIndex($params) {
if ($this->controller->_beforeScaffold('index')) {
$this->controller->set('fieldNames', $this->controller->generateFieldNames(null, false));
@ -213,9 +192,10 @@ class Scaffold extends Object {
return $this->controller->render($this->action, '',
APP . 'views' . DS . 'scaffold' . DS . 'scaffold.index.thtml');
} else {
return $this->controller->render($this->action, '', LIBS . 'view' . DS . 'templates'
. DS . 'scaffolds' . DS . 'index.thtml');
return $this->controller->render($this->action, '',
LIBS . 'view' . DS . 'templates' . DS . 'scaffolds' . DS . 'index.thtml');
}
} elseif ($this->controller->_scaffoldError('index') === false) {
return $this->__scaffoldError();
}
@ -237,23 +217,21 @@ class Scaffold extends Object {
if ($action == 'edit') {
if(isset($params['pass'][0])){
$this->controller->{$this->modelKey}->id = $params['pass'][0];
$this->ScaffoldModel->id = $params['pass'][0];
} elseif (isset($this->controller->Session) && $this->controller->Session->valid != false) {
$this->controller->Session->setFlash('No id set for ' . Inflector::humanize($this->modelKey) . '::edit().');
$this->controller->redirect('/' . Inflector::underscore($this->controller->viewPath));
} else {
return $this->controller->flash('No id set for ' . Inflector::humanize($this->modelKey) . '::edit().',
'/' . Inflector::underscore($this->controller->viewPath));
}
$this->controller->data = $this->ScaffoldModel->read();
$this->controller->set('fieldNames',
$this->controller->generateFieldNames($this->controller->data));
$this->controller->set('fieldNames', $this->controller->generateFieldNames($this->controller->data));
$this->controller->set('data', $this->controller->data);
} else {
$this->controller->set('fieldNames', $this->controller->generateFieldNames());
}
if (file_exists(APP . 'views' . DS . $this->viewPath . DS . 'scaffold.' . $action . '.thtml')) {
return $this->controller->render($action, '', APP . 'views' . DS . $this->viewPath . DS . 'scaffold.' . $action . '.thtml');
} elseif(file_exists(APP . 'views' . DS . 'scaffold' . DS . 'scaffold.' . $action . '.thtml')) {
@ -261,13 +239,12 @@ class Scaffold extends Object {
} else {
return $this->controller->render($action, '', LIBS . 'view' . DS . 'templates' . DS . 'scaffolds' . DS . $action . '.thtml');
}
} else if($this->controller->_scaffoldError($action) === false) {
return $this->__scaffoldError();
}
}
/**
*
* Saves or updates a model.
*
* @param array $params
@ -335,7 +312,6 @@ class Scaffold extends Object {
return $this->__scaffoldError();
}
}
/**
* Performs a delete on given scaffolded Model.
*
@ -345,54 +321,54 @@ class Scaffold extends Object {
*/
function __scaffoldDelete($params = array()) {
if ($this->controller->_beforeScaffold('delete')) {
if(isset($params['pass'][0])){
$id = $params['pass'][0];
} elseif (isset($this->controller->Session) && $this->controller->Session->valid != false) {
$this->controller->Session->setFlash('No id set for ' . Inflector::humanize($this->modelKey) . '::delete().');
$this->controller->redirect('/' . Inflector::underscore($this->controller->viewPath));
} else {
return $this->controller->flash('No id set for ' . Inflector::humanize($this->modelKey) . '::delete().',
'/' . Inflector::underscore($this->controller->viewPath));
}
if ($this->ScaffoldModel->del($id)) {
if (isset($this->controller->Session) && $this->controller->Session->valid != false) {
$this->controller->Session->setFlash(
'The ' . Inflector::humanize($this->modelClass) . ' with id: ' . $id
. ' has been deleted.');
$this->controller->Session->setFlash('The ' . Inflector::humanize($this->modelClass) . ' with id: ' . $id . ' has been deleted.');
$this->controller->redirect('/' . $this->viewPath);
} else {
return $this->controller->flash(
'The ' . Inflector::humanize($this->modelClass) . ' with id: ' . $id
. ' has been deleted.', '/' . $this->viewPath);
return $this->controller->flash('The ' . Inflector::humanize($this->modelClass) . ' with id: ' . $id . ' has been deleted.', '/' . $this->viewPath);
}
} else {
if (isset($this->controller->Session) && $this->controller->Session->valid != false) {
$this->controller->Session->setFlash(
'There was an error deleting the ' . Inflector::humanize($this->modelClass)
. ' with the id ' . $id);
$this->controller->Session->setFlash('There was an error deleting the ' . Inflector::humanize($this->modelClass) . ' with the id ' . $id);
$this->controller->redirect('/' . $this->viewPath);
} else {
return $this->controller->flash(
'There was an error deleting the ' . Inflector::humanize($this->modelClass)
. ' with the id ' . $id, '/' . $this->viewPath);
return $this->controller->flash('There was an error deleting the ' . Inflector::humanize($this->modelClass) . ' with the id ' . $id, '/' . $this->viewPath);
}
}
} elseif ($this->controller->_scaffoldError('delete') === false) {
return $this->__scaffoldError();
}
}
/**
* Enter description here...
*
* @return unknown
* @return error
*/
function __scaffoldError() {
if (file_exists(APP . 'views' . DS . $this->viewPath . DS . 'scaffolds' . DS. 'scaffold.error.thtml')) {
return $this->controller->render($this->action, '', APP . 'views' . DS . $this->viewPath
. DS . 'scaffolds' . DS . 'scaffold.error.thtml');
return $this->controller->render($this->action, '',
APP . 'views' . DS . $this->viewPath . DS . 'scaffolds' . DS . 'scaffold.error.thtml');
} elseif (file_exists(APP . 'views' . DS . 'scaffold' . DS . 'scaffold.error.thtml')) {
return $this->controller->render($this->action, '', APP . 'views' . DS . 'scaffold'
. DS . 'scaffold.error.thtml');
return $this->controller->render($this->action, '',
APP . 'views' . DS . 'scaffold' . DS . 'scaffold.error.thtml');
} else {
return $this->controller->render($this->action, '', LIBS . 'view' . DS . 'templates' . DS
. 'errors' . DS . 'scaffold_error.thtml');
return $this->controller->render($this->action, '',
LIBS . 'view' . DS . 'templates' . DS . 'errors' . DS . 'scaffold_error.thtml');
}
}
/**
* When forms are submited the arrays need to be rebuilt if
* an error occured, here the arrays are rebuilt to structure needed
@ -412,15 +388,12 @@ class Scaffold extends Object {
$params[$model][$count][$this->ScaffoldModel->primaryKey] = $value;
$count++;
}
unset ($params[$model][$model]);
}
}
}
return $params;
}
/**
* When methods are now present in a controller
* scaffoldView is used to call default Scaffold methods if:
@ -436,64 +409,45 @@ class Scaffold extends Object {
* @access private
*/
function __scaffold($params) {
$db = &ConnectionManager::getDataSource($this->ScaffoldModel->useDbConfig);
if (isset($db)) {
if(empty($this->scaffoldActions)) {
$this->scaffoldActions = array('index', 'list', 'view', 'add', 'create', 'edit', 'update', 'delete');
}
if (in_array($params['action'], $this->scaffoldActions)) {
switch($params['action'])
{
switch($params['action']) {
case 'index':
$this->__scaffoldIndex($params);
break;
case 'view':
$this->__scaffoldView($params);
break;
case 'list':
$this->__scaffoldIndex($params);
break;
case 'add':
$this->__scaffoldForm($params, 'add');
break;
case 'edit':
$this->__scaffoldForm($params, 'edit');
break;
case 'create':
$this->__scaffoldSave($params, 'create');
break;
case 'update':
$this->__scaffoldSave($params, 'update');
break;
case 'delete':
$this->__scaffoldDelete($params);
break;
}
} else {
return $this->cakeError('missingAction', array(
array('className' => $this->controller->name . "Controller",
return $this->cakeError('missingAction', array(array('className' => $this->controller->name . "Controller",
'base' => $this->controller->base,
'action' => $this->action,
'webroot' => $this->controller->webroot)
)
);
'webroot' => $this->controller->webroot)));
}
} else {
return $this->cakeError('missingDatabase', array(array('webroot' => $this->controller->webroot)));

View file

@ -28,6 +28,6 @@
<p class="error">Your must implement the following method in your controller</p>
<p><span class="notice"><strong>Notice:</strong> If you want to customize this error message, create <?php echo APP_DIR.DS."views/errors/scaffold_error.thtml"; ?>.</span></p>
<p>
&nbsp;&nbsp;&nbsp;function scaffoldError() {<br />
&nbsp;&nbsp;&nbsp;function _scaffoldError() {<br />
&nbsp;&nbsp;&nbsp;}<br />
</p>