Skip to content

Commit

Permalink
Removing Overloadable and Overloadable2 they are not needed in PHP5.
Browse files Browse the repository at this point in the history
Removing inheritance and test cases related to Overloadable.
  • Loading branch information
markstory committed Apr 18, 2010
1 parent 92289e5 commit 9046083
Show file tree
Hide file tree
Showing 5 changed files with 3 additions and 159 deletions.
4 changes: 1 addition & 3 deletions cake/libs/configure.php
Expand Up @@ -1060,9 +1060,7 @@ private function __mapped($name, $type, $plugin) {
* @access private
*/
private function __overload($type, $name, $parent) {
if (($type === 'Model' || $type === 'Helper') && $parent !== false) {
Overloadable::overload($name);
}

}

/**
Expand Down
10 changes: 2 additions & 8 deletions cake/libs/model/model.php
Expand Up @@ -27,10 +27,6 @@
App::import('Model', 'ModelBehavior', false);
App::import('Model', 'ConnectionManager', false);

if (!class_exists('Overloadable')) {
require LIBS . 'overloadable.php';
}

/**
* Object-relational mapper.
*
Expand All @@ -43,7 +39,7 @@
* @subpackage cake.cake.libs.model
* @link http://book.cakephp.org/view/1000/Models
*/
class Model extends Overloadable {
class Model extends Object {

/**
* The name of the DataSource connection that this Model uses
Expand Down Expand Up @@ -2997,7 +2993,5 @@ function __sleep() {
function __wakeup() {
}
}
if (!defined('CAKEPHP_UNIT_TEST_EXECUTION')) {
Overloadable::overload('Model');
}

?>
107 changes: 0 additions & 107 deletions cake/libs/overloadable.php

This file was deleted.

40 changes: 0 additions & 40 deletions cake/tests/cases/libs/overloadable.test.php

This file was deleted.

1 change: 0 additions & 1 deletion cake/tests/groups/lib.group.php
Expand Up @@ -53,7 +53,6 @@ function LibGroupTest() {
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'cake_log');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'log' . DS . 'file_log');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'class_registry');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'overloadable');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'sanitize');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'security');
TestManager::addTestFile($this, CORE_TEST_CASES . DS . 'libs' . DS . 'set');
Expand Down

0 comments on commit 9046083

Please sign in to comment.