Merge pull request #500 from nyeholt/injector_strong_create

RECOMMIT This is a recommit of a previously merged, but reverted, commit...
This commit is contained in:
Sam Minnée 2012-06-04 15:32:19 -07:00
commit f8921f7c9f
4 changed files with 4 additions and 7 deletions

View File

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

View File

@ -356,8 +356,6 @@ function getClassFile($className) {
* way to access instance methods which don't rely on instance
* data (e.g. the custom SilverStripe static handling).
*
* @uses Object::strong_create()
*
* @param string $className
* @return Object
*/
@ -367,7 +365,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] = Object::strong_create($className,null, true);
$_SINGLETONS[$className] = Injector::inst()->get($className);
if(!$_SINGLETONS[$className]) user_error("singleton() Unknown class '$className'", E_USER_ERROR);
}
return $_SINGLETONS[$className];

View File

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

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) {
function __construct($stages=array('Stage','Live')) {
parent::__construct();
if(!is_array($stages)) {