From 8093932637eef9fdbb299b76c8d3057536eb8001 Mon Sep 17 00:00:00 2001 From: Ben Klang Date: Mon, 4 Oct 2010 14:39:15 -0400 Subject: [PATCH] Rebuild factory for Horde_Notification Still need to so something different with the singleton method I think. Does it really belong in the class? --- .../Core/lib/Horde/Core/{Binder => Factory}/Notification.php | 8 +------- framework/Core/lib/Horde/Registry.php | 5 ++++- framework/Core/package.xml | 10 +++++----- 3 files changed, 10 insertions(+), 13 deletions(-) rename framework/Core/lib/Horde/Core/{Binder => Factory}/Notification.php (77%) diff --git a/framework/Core/lib/Horde/Core/Binder/Notification.php b/framework/Core/lib/Horde/Core/Factory/Notification.php similarity index 77% rename from framework/Core/lib/Horde/Core/Binder/Notification.php rename to framework/Core/lib/Horde/Core/Factory/Notification.php index 1608431ba..1295f6cbd 100644 --- a/framework/Core/lib/Horde/Core/Binder/Notification.php +++ b/framework/Core/lib/Horde/Core/Factory/Notification.php @@ -3,7 +3,7 @@ * @category Horde * @package Core */ -class Horde_Core_Binder_Notification implements Horde_Injector_Binder +class Horde_Core_Factory_Notification { public function create(Horde_Injector $injector) { @@ -17,10 +17,4 @@ class Horde_Core_Binder_Notification implements Horde_Injector_Binder return $notify; } - - public function equals(Horde_Injector_Binder $binder) - { - return false; - } - } diff --git a/framework/Core/lib/Horde/Registry.php b/framework/Core/lib/Horde/Registry.php index 2a010cff3..baa334f6c 100644 --- a/framework/Core/lib/Horde/Registry.php +++ b/framework/Core/lib/Horde/Registry.php @@ -277,7 +277,6 @@ class Horde_Registry 'Horde_LoginTasks' => new Horde_Core_Binder_LoginTasks(), 'Horde_Mail' => new Horde_Core_Binder_Mail(), 'Horde_Mime_Viewer' => new Horde_Core_Binder_MimeViewer(), - 'Horde_Notification' => new Horde_Core_Binder_Notification(), 'Horde_Prefs_Identity' => new Horde_Core_Binder_Identity(), 'Horde_Share_Factory' => new Horde_Core_Binder_ShareFactory(), 'Horde_Template' => new Horde_Core_Binder_Template(), @@ -333,6 +332,10 @@ class Horde_Registry 'Horde_Core_Factory_Memcache', 'create', ), + 'Horde_Notification' => array( + 'Horde_Core_Factory_Notification', + 'create', + ), 'Horde_Perms' => array( 'Horde_Core_Factory_Perms', 'create', diff --git a/framework/Core/package.xml b/framework/Core/package.xml index 35fd3217b..eb54fc9ad 100644 --- a/framework/Core/package.xml +++ b/framework/Core/package.xml @@ -23,8 +23,8 @@ Application Framework. slusarz@horde.org yes - 2010-10-03 - + 2010-10-04 + 0.1.0 0.1.0 @@ -127,7 +127,6 @@ Application Framework. - @@ -164,6 +163,7 @@ Application Framework. + @@ -446,7 +446,6 @@ Application Framework. - @@ -479,6 +478,7 @@ Application Framework. + @@ -575,7 +575,7 @@ Initial packaging beta beta - 2010-10-03 + 2010-10-04 LGPL * Add Horde::addInlineJsVars(). -- 2.11.0