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

fixed and tested object has_extension

Browse Source
This commit is contained in:
John Milmine 2014-10-24 16:58:50 +13:00
parent f0ca37fdea
commit 62658a6cca
2 changed files with 31 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
core/Object.php
View File

@ -476,8 +476,9 @@ abstract class Object {
* @param string $classOrExtension if 1 argument supplied, the class name of the extension to * @param string $classOrExtension if 1 argument supplied, the class name of the extension to
* check for; if 2 supplied, the class name to test * check for; if 2 supplied, the class name to test
* @param string $requiredExtension used only if 2 arguments supplied * @param string $requiredExtension used only if 2 arguments supplied
* @param boolean $strict if the extension has to match the required extension and not be a subclass
*/ */
public static function has_extension($classOrExtension, $requiredExtension = null) { public static function has_extension($classOrExtension, $requiredExtension = null, $strict = false) {
//BC support //BC support
if(func_num_args() > 1){ if(func_num_args() > 1){
$class = $classOrExtension; $class = $classOrExtension;
@ -495,6 +496,7 @@ abstract class Object {
$left = strtolower(Extension::get_classname_without_arguments($extension)); $left = strtolower(Extension::get_classname_without_arguments($extension));
$right = strtolower(Extension::get_classname_without_arguments($requiredExtension)); $right = strtolower(Extension::get_classname_without_arguments($requiredExtension));
if($left == $right) return true; if($left == $right) return true;
if (!$strict && is_subclass_of($left, $right)) return true;
} }
return false; return false;

27
tests/core/ObjectTest.php
View File

@ -233,6 +233,25 @@ class ObjectTest extends SapphireTest {
ObjectTest_ExtensionTest::has_extension('ObjectTest_ExtendTest3'), ObjectTest_ExtensionTest::has_extension('ObjectTest_ExtendTest3'),
"Extensions are detected with static has_extension() when added through add_extension()" "Extensions are detected with static has_extension() when added through add_extension()"
); );
// ObjectTest_ExtendTest4 is added manually
ObjectTest_ExtensionTest3::add_extension('ObjectTest_ExtendTest4("Param")');
// test against ObjectTest_ExtendTest3, not ObjectTest_ExtendTest3
$this->assertTrue(
ObjectTest_ExtensionTest3::has_extension('ObjectTest_ExtendTest4'),
"Extensions are detected with static has_extension() when added through add_extension()"
);
// test against ObjectTest_ExtendTest3, not ObjectTest_ExtendTest4 to test if it picks up
// the sub classes of ObjectTest_ExtendTest3
$this->assertTrue(
ObjectTest_ExtensionTest3::has_extension('ObjectTest_ExtendTest3'),
"Sub-Extensions are detected with static has_extension() when added through add_extension()"
);
// strictly test against ObjectTest_ExtendTest3, not ObjectTest_ExtendTest4 to test if it picks up
// the sub classes of ObjectTest_ExtendTest3
$this->assertFalse(
ObjectTest_ExtensionTest3::has_extension('ObjectTest_ExtendTest3', null, true),
"Sub-Extensions are detected with static has_extension() when added through add_extension()"
);
// a singleton() wouldn't work as its already initialized // a singleton() wouldn't work as its already initialized
$objectTest_ExtensionTest = new ObjectTest_ExtensionTest(); $objectTest_ExtensionTest = new ObjectTest_ExtensionTest();
$this->assertTrue( $this->assertTrue(
@ -518,6 +537,10 @@ class ObjectTest_ExtensionTest2 extends Object {
private static $extensions = array('ObjectTest_Extension'); private static $extensions = array('ObjectTest_Extension');
} }
class ObjectTest_ExtensionTest3 extends Object {
}
class ObjectTest_ExtensionRemoveTest extends Object { class ObjectTest_ExtensionRemoveTest extends Object {
private static $extensions = array ( private static $extensions = array (
@ -563,4 +586,8 @@ class ObjectTest_ExtendTest3 extends Extension {
public function extendableMethod($argument = null) { return "ExtendTest3($argument)"; } public function extendableMethod($argument = null) { return "ExtendTest3($argument)"; }
} }
class ObjectTest_ExtendTest4 extends ObjectTest_ExtendTest3 {
public function extendableMethod($argument = null) { return "ExtendTest4($argument)"; }
}
/**#@-*/ /**#@-*/