adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Remove persistModel.

Browse source 
The benefits of persistModel are no longer needed. Because of lazy model associations,
the performance benefits persistModel are no longer realized.
Fixes #1782
This commit is contained in:
Mark Story 2011-06-19 22:08:03 -07:00
parent 1c7d54eacc
commit ed8ccc2903
2 changed files with 7 additions and 61 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

45
lib/Cake/Controller/Controller.php
View file

@ -236,15 +236,6 @@ class Controller extends Object {
*/ */
public $cacheAction = false; public $cacheAction = false;
/**
* Used to create cached instances of models a controller uses.
* When set to true, all models related to the controller will be cached.
* This can increase performance in many cases.
*
* @var boolean
*/
public $persistModel = false;
/** /**
* Holds all params passed and named. * Holds all params passed and named.
* *
@ -593,8 +584,6 @@ class Controller extends Object {
/** /**
* Loads and instantiates models required by this controller. * Loads and instantiates models required by this controller.
* If Controller::$persistModel; is true, controller will cache model instances on first request,
* additional request will used cached models.
* If the model is non existent, it will throw a missing database table error, as Cake generates * If the model is non existent, it will throw a missing database table error, as Cake generates
* dynamic models for the time being. * dynamic models for the time being.
* *
@ -607,36 +596,16 @@ class Controller extends Object {
if ($modelClass === null) { if ($modelClass === null) {
$modelClass = $this->modelClass; $modelClass = $this->modelClass;
} }
$cached = false;
$object = null;
list($plugin, $modelClass) = pluginSplit($modelClass, true); list($plugin, $modelClass) = pluginSplit($modelClass, true);
if ($this->persistModel === true) { $this->modelNames[] = $modelClass;
$cached = $this->_persist($modelClass, null, $object);
$this->{$modelClass} = ClassRegistry::init(array(
'class' => $plugin . $modelClass, 'alias' => $modelClass, 'id' => $id
));
if (!$this->{$modelClass}) {
throw new MissingModelException($modelClass);
} }
if (($cached === false)) {
$this->modelNames[] = $modelClass;
$this->{$modelClass} = ClassRegistry::init(array(
'class' => $plugin . $modelClass, 'alias' => $modelClass, 'id' => $id
));
if (!$this->{$modelClass}) {
throw new MissingModelException($modelClass);
}
if ($this->persistModel === true) {
$this->_persist($modelClass, true, $this->{$modelClass});
$registry = ClassRegistry::getInstance();
$this->_persist($modelClass . 'registry', true, $registry->__objects, 'registry');
}
} else {
$this->_persist($modelClass . 'registry', true, $object, 'registry');
$this->_persist($modelClass, true, $object);
$this->modelNames[] = $modelClass;
}
return true; return true;
} }

23
lib/Cake/Test/Case/Controller/ControllerTest.php
View file

@ -513,29 +513,6 @@ class ControllerTest extends CakeTestCase {
unset($Controller); unset($Controller);
} }
/**
* testPersistent method
*
* @access public
* @return void
*/
public function testPersistent() {
$this->markTestIncomplete('persistModel is totally broken right now.');
Configure::write('Cache.disable', false);
$Controller = new Controller();
$Controller->modelClass = 'ControllerPost';
$Controller->persistModel = true;
$Controller->constructClasses();
$this->assertTrue(file_exists(CACHE . 'persistent' . DS .'controllerpost.php'));
$this->assertTrue(is_a($Controller->ControllerPost, 'ControllerPost'));
@unlink(CACHE . 'persistent' . DS . 'controllerpost.php');
@unlink(CACHE . 'persistent' . DS . 'controllerpostregistry.php');
unset($Controller);
Configure::write('Cache.disable', true);
}
/** /**
* testFlash method * testFlash method
* *