tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-06-30 16:39:38 +02:00
Code Issues Projects Releases Wiki Activity

Revert "BUGFIX Versioned's constructor doesn't provide suitable defaults. Previously a bug/feature in singleton, where it would pass null,true as params to strong_create, which would then get passed through as params to Versioned's constructor, meant that the code still executed fine (as was set to something that wasn't an array, so the null and true were instead taken as args). The fact that the usage of singleton(Versioned) never really used the classes code, purely for value lookup, meant that this never propagated errors. I've now switched singleton() to use the injector for retrieving values, which means these dud values are no longer passed through"

Browse Source 
This reverts commit 5eaca340b4.
This commit is contained in:
Sean Harvey 2012-05-23 09:52:57 +12:00
parent 627def6be3
commit 29e04a1049
4 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
core/Config.php
View File

@ -118,7 +118,7 @@ class Config {
* @return Config
*/
static public function inst() {
if (!self::$instance) self::$instance = new Config();
if (!self::$instance) self::$instance = singleton('Config');
return self::$instance;
}

2
core/Core.php
View File

@ -367,7 +367,7 @@ function singleton($className) {
if(!is_string($className)) user_error("singleton() passed bad class_name: " . var_export($className,true), E_USER_ERROR);
if(!isset($_SINGLETONS[$className])) {
if(!class_exists($className)) user_error("Bad class to singleton() - $className", E_USER_ERROR);
$_SINGLETONS[$className] = Injector::inst()->get($className);
$_SINGLETONS[$className] = Object::strong_create($className,null, true);
if(!$_SINGLETONS[$className]) user_error("singleton() Unknown class '$className'", E_USER_ERROR);
}
return $_SINGLETONS[$className];

3
core/Object.php
View File

@ -235,7 +235,8 @@ abstract class Object {
$class = self::$strong_classes[$class];
}
return Injector::inst()->createWithArgs($class, $args);
$reflector = new ReflectionClass($class);
return $reflector->newInstanceArgs($args);
}
/**

2
model/Versioned.php
View File

@ -92,7 +92,7 @@ class Versioned extends DataExtension {
* The first stage is consiedered the 'default' stage, the last stage is
* considered the 'live' stage.
*/
function __construct($stages=array('Stage','Live')) {
function __construct($stages) {
parent::__construct();
if(!is_array($stages)) {