mirror of
https://github.com/kamilwylegala/cakephp2-php8.git
synced 2024-11-15 03:18:26 +00:00
Fixing issue where plugin models in $uses, would create
incorrect forms when create() is called with no arguments. Fixes #1841
This commit is contained in:
parent
3589a17321
commit
6fb3c72d49
2 changed files with 3 additions and 5 deletions
|
@ -866,7 +866,7 @@ class Controller extends Object {
|
|||
if (!empty($this->uses)) {
|
||||
foreach ($this->uses as $model) {
|
||||
list($plugin, $className) = pluginSplit($model);
|
||||
$this->request->params['models'][$model] = compact('plugin', 'className');
|
||||
$this->request->params['models'][$className] = compact('plugin', 'className');
|
||||
}
|
||||
} if (!empty($this->modelClass) && ($this->uses === false || $this->uses === array())) {
|
||||
$this->request->params['models'][$this->modelClass] = array('plugin' => $this->plugin, 'className' => $this->modelClass);
|
||||
|
|
|
@ -651,6 +651,7 @@ class ControllerTest extends CakeTestCase {
|
|||
$Controller->view = null;
|
||||
|
||||
$Controller = new TestController($request, new CakeResponse());
|
||||
$Controller->uses = array('ControllerAlias', 'TestPlugin.ControllerComment', 'ControllerPost');
|
||||
$Controller->helpers = array('Html');
|
||||
$Controller->constructClasses();
|
||||
$Controller->ControllerComment->validationErrors = array('title' => 'tooShort');
|
||||
|
@ -664,15 +665,12 @@ class ControllerTest extends CakeTestCase {
|
|||
|
||||
$expectedModels = array(
|
||||
'ControllerAlias' => array('plugin' => null, 'className' => 'ControllerAlias'),
|
||||
'ControllerComment' => array('plugin' => null, 'className' => 'ControllerComment'),
|
||||
'ControllerComment' => array('plugin' => 'TestPlugin', 'className' => 'ControllerComment'),
|
||||
'ControllerPost' => array('plugin' => null, 'className' => 'ControllerPost')
|
||||
);
|
||||
$this->assertEqual($expectedModels, $Controller->request->params['models']);
|
||||
|
||||
|
||||
$Controller->ControllerComment->validationErrors = array();
|
||||
ClassRegistry::flush();
|
||||
|
||||
App::build();
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue