tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

API CHANGE: Make Object::uninherited_static() have a separate execution path to Object::get_static(), for more reliable operation. The intention is that for any given static, you either use Object::get_static() or you use Object::uninherited_static() - not both. (from r84151, r84155, r84156)

Browse Source 
git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/branches/2.3@84157 467b73ca-7a2a-4603-9d3b-597d59a354a9
This commit is contained in:
Sam Minnee 2009-08-11 05:55:05 +00:00
parent 38846653a2
commit 427cce498d
4 changed files with 70 additions and 34 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

67
core/Object.php
View File

@ -41,10 +41,12 @@ abstract class Object {
*/
private static
$statics = array(),
$cached_statics = array(),
$extra_statics = array(),
$replaced_statics = array(),
$statics = array(),
$cached_statics = array(),
$uninherited_statics = array(),
$cached_uninherited_statics = array(),
$extra_statics = array(),
$replaced_statics = array(),
$_cache_statics_prepared = array();
private static
@ -231,7 +233,7 @@ abstract class Object {
return self::$statics[$class][$name];
}
/**
* Set a static variable
*
@ -244,10 +246,12 @@ abstract class Object {
Object::prepare_statics($class);
}
self::$statics[$class][$name] = $value;
self::$statics[$class][$name] = $value;
self::$uninherited_statics[$class][$name] = $value;
self::$cached_statics[$class][$name] = true;
self::$cached_uninherited_statics[$class][$name] = true;
}
/**
* Get an uninherited static variable - a variable that is explicity set in this class, and not in the parent class.
*
@ -260,19 +264,33 @@ abstract class Object {
* @param string $name
* @return mixed
*/
public static function uninherited_static($class, $name) {
$inherited = self::get_static($class, $name);
$parent = null;
if($parentClass = get_parent_class($class)) {
$parent = self::get_static($parentClass, $name);
public static function uninherited_static($class, $name, $uncached = false) {
if(!isset(self::$_cache_statics_prepared[$class])) {
Object::prepare_statics($class);
}
if(is_array($inherited) && is_array($parent)) {
return array_diff_assoc($inherited, $parent);
if(!isset(self::$cached_uninherited_statics[$class][$name]) || $uncached) {
$classRef = new ReflectionClass($class);
$classProps = $classRef->getStaticProperties();
$parentClass = get_parent_class($class);
if($parentClass) {
$parentRef = new ReflectionClass($parentClass);
$parentProps = $parentRef->getStaticProperties();
$uninheritedBuiltIn = isset($classProps[$name])
&& (!isset($parentProps[$name])
|| $classProps[$name] != $parentProps[$name]) ? $classProps[$name] : null;
} else {
$uninheritedBuiltIn = isset($classProps[$name]) ? $classProps[$name] : null;
}
self::$cached_uninherited_statics[$class][$name] = true;
self::$uninherited_statics[$class][$name] = $uninheritedBuiltIn;
}
return ($inherited != $parent) ? $inherited : null;
return self::$uninherited_statics[$class][$name];
}
/**
@ -356,7 +374,7 @@ abstract class Object {
*/
public static function has_extension($class, $requiredExtension) {
$requiredExtension = strtolower($requiredExtension);
if($extensions = self::get_static($class, 'extensions')) foreach($extensions as $extension) {
if($extensions = self::combined_static($class, 'extensions')) foreach($extensions as $extension) {
$left = strtolower(Extension::get_classname_without_arguments($extension));
$right = strtolower(Extension::get_classname_without_arguments($requiredExtension));
if($left == $right) return true;
@ -399,7 +417,14 @@ abstract class Object {
}
// merge with existing static vars
self::add_static_var($class, 'extensions', array($extension));
$extensions = self::uninherited_static($class, 'extensions');
// We use unshift rather than push so that module extensions are added before built-in ones.
// in particular, this ensures that the Versioned rewriting is done last.
if($extensions) array_unshift($extensions, $extension);
else $extensions = array($extension);
self::set_static($class, 'extensions', $extensions);
// load statics now for DataObject classes
if(is_subclass_of($class, 'DataObject')) {
@ -418,7 +443,7 @@ abstract class Object {
// load statics now for DataObject classes
if(is_subclass_of($class, 'DataObject')) {
$extensions = Object::get_static($class, 'extensions');
$extensions = Object::uninherited_static($class, 'extensions');
if($extensions) foreach($extensions as $extension) {
if(preg_match('/^([^(]*)/', $extension, $matches)) {
$extensionClass = $matches[1];
@ -450,7 +475,7 @@ abstract class Object {
self::set_static(
$class,
'extensions',
array_diff(self::get_static($class, 'extensions'), array($extension))
array_diff(self::uninherited_static($class, 'extensions'), array($extension))
);
}

24
core/model/DataObject.php
View File

@ -723,7 +723,7 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
$classes = array_reverse(ClassInfo::ancestry($this));
foreach($classes as $class) {
$defaults = Object::get_static($class, 'defaults');
$defaults = Object::uninherited_static($class, 'defaults');
if($defaults) foreach($defaults as $fieldName => $fieldValue) {
// SRM 2007-03-06: Stricter check
@ -1336,13 +1336,13 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
if(in_array($class, array('Object', 'ViewableData', 'DataObject'))) continue;
if($component) {
$hasOne = Object::get_static($class, 'has_one');
$hasOne = Object::uninherited_static($class, 'has_one');
if(isset($hasOne[$component])) {
return $hasOne[$component];
}
} else {
$newItems = (array) Object::get_static($class, 'has_one');
$newItems = (array) Object::uninherited_static($class, 'has_one');
// Validate the data
foreach($newItems as $k => $v) {
if(!is_string($k) || is_numeric($k) || !is_string($v)) user_error("$class::\$has_one has a bad entry: "
@ -1375,13 +1375,13 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
}
if($component) {
$db = Object::get_static($class, 'db');
$db = Object::uninherited_static($class, 'db');
if(isset($db[$component])) {
return $db[$component];
}
} else {
$newItems = (array) Object::get_static($class, 'db');
$newItems = (array) Object::uninherited_static($class, 'db');
// Validate the data
foreach($newItems as $k => $v) {
if(!is_string($k) || is_numeric($k) || !is_string($v)) user_error("$class::\$db has a bad entry: "
@ -1409,13 +1409,13 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
if(in_array($class, array('ViewableData', 'Object', 'DataObject'))) continue;
if($component) {
$hasMany = Object::get_static($class, 'has_many');
$hasMany = Object::uninherited_static($class, 'has_many');
if(isset($hasMany[$component])) {
return $hasMany[$component];
}
} else {
$newItems = (array) Object::get_static($class, 'has_many');
$newItems = (array) Object::uninherited_static($class, 'has_many');
// Validate the data
foreach($newItems as $k => $v) {
if(!is_string($k) || is_numeric($k) || !is_string($v)) user_error("$class::\$has_many has a bad entry: "
@ -1529,7 +1529,7 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
if(in_array($class, array('ViewableData', 'Object', 'DataObject'))) continue;
if($component) {
$manyMany = Object::get_static($class, 'many_many');
$manyMany = Object::uninherited_static($class, 'many_many');
// Try many_many
$candidate = (isset($manyMany[$component])) ? $manyMany[$component] : null;
if($candidate) {
@ -1539,13 +1539,13 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
}
// Try belongs_many_many
$belongsManyMany = Object::get_static($class, 'belongs_many_many');
$belongsManyMany = Object::uninherited_static($class, 'belongs_many_many');
$candidate = (isset($belongsManyMany[$component])) ? $belongsManyMany[$component] : null;
if($candidate) {
$childField = $candidate . "ID";
// We need to find the inverse component name
$otherManyMany = Object::get_static($candidate, 'many_many');
$otherManyMany = Object::uninherited_static($candidate, 'many_many');
if(!$otherManyMany) {
user_error("Inverse component of $candidate not found ({$this->class})", E_USER_ERROR);
}
@ -1564,7 +1564,7 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
user_error("Orphaned \$belongs_many_many value for $this->class.$component", E_USER_ERROR);
}
} else {
$newItems = (array) Object::get_static($class, 'many_many');
$newItems = (array) Object::uninherited_static($class, 'many_many');
// Validate the data
foreach($newItems as $k => $v) {
if(!is_string($k) || is_numeric($k) || !is_string($v)) user_error("$class::\$many_many has a bad entry: "
@ -1572,7 +1572,7 @@ class DataObject extends ViewableData implements DataObjectInterface, i18nEntity
}
$items = isset($items) ? array_merge($newItems, $items) : $newItems;
$newItems = (array) Object::get_static($class, 'belongs_many_many');
$newItems = (array) Object::uninherited_static($class, 'belongs_many_many');
// Validate the data
foreach($newItems as $k => $v) {
if(!is_string($k) || is_numeric($k) || !is_string($v)) user_error("$class::\$belongs_many_many has a bad entry: "

2
core/model/DataObjectDecorator.php
View File

@ -69,10 +69,10 @@ abstract class DataObjectDecorator extends Extension {
if($statics) {
foreach($statics as $name => $newVal) {
if(isset(self::$decoratable_statics[$name])) {
$origVal = self::get_static($class, $name);
// Array to be merged
if(self::$decoratable_statics[$name]) {
$origVal = self::uninherited_static($class, $name);
// Can't use add_static_var() here as it would merge the array rather than replacing
self::set_static($class, $name, array_merge((array)$origVal, $newVal));

11
tests/ObjectStaticTest.php
View File

@ -52,6 +52,17 @@ class ObjectStaticTest extends SapphireTest {
$this->assertEquals(Object::uninherited_static('ObjectStaticTest_Fourth', 'third', true), null);
}
/**
* Confirms that Object::add_static_var() doesn't work for uninherited stats
*/
public function testAddStaticVarDoesntWorkFor() {
Object::add_static_var('ObjectStaticTest_First', 'first', array('test_1b'));
Object::add_static_var('ObjectStaticTest_Second', 'first', array('test_2b'));
$this->assertNotContains('test_1b', Object::uninherited_static('ObjectStaticTest_First', 'first'));
$this->assertNotContains('test_2b', Object::uninherited_static('ObjectStaticTest_Second', 'first'));
}
}
/**#@+