Get a factory directly from the injector
authorBen Klang <ben@alkaloid.net>
Sun, 3 Oct 2010 15:08:08 +0000 (11:08 -0400)
committerBen Klang <ben@alkaloid.net>
Sun, 3 Oct 2010 15:08:59 +0000 (11:08 -0400)
24 files changed:
ansel/lib/Ansel.php
ansel/lib/View/List.php
ansel/templates/group/owner.inc
folks/lib/Friends.php
framework/Core/lib/Horde/Core/Binder/Prefs.php [deleted file]
framework/Core/lib/Horde/Core/Factory/Identity.php
framework/Core/lib/Horde/Core/Sidebar.php
framework/Core/lib/Horde/Registry.php
framework/Core/package.xml
horde/lib/Api.php
horde/scripts/import_squirrelmail_file_prefs.php
horde/scripts/import_squirrelmail_sql_prefs.php
horde/services/resetpassword.php
imp/attachment.php
ingo/lib/Storage/Prefs.php
kronolith/fb.php
kronolith/lib/Api.php
kronolith/lib/Kronolith.php
kronolith/scripts/agenda.php
kronolith/scripts/upgrades/convert_to_utc.php
nag/lib/Api.php
nag/lib/Nag.php
turba/scripts/import_squirrelmail_file_abook.php
turba/scripts/import_squirrelmail_sql_abook.php

index fd678e9..4f9a2e3 100644 (file)
@@ -580,7 +580,7 @@ class Ansel
             } elseif ($owner == $GLOBALS['registry']->getAuth()) {
                 $owner_title = _("My Galleries");
             } elseif (!empty($GLOBALS['conf']['gallery']['customlabel'])) {
-                $uprefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('ansel', array(
+                $uprefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('ansel', array(
                     'cache' => false,
                     'user' => $owner
                 ));
index 389dd4a..cdae4f7 100644 (file)
@@ -149,7 +149,7 @@ class Ansel_View_List extends Ansel_View_Base
             if ($this->_owner == $GLOBALS['registry']->getAuth() && empty($this->_params['api'])) {
                 return  _("My Galleries");
             } elseif (!empty($GLOBALS['conf']['gallery']['customlabel'])) {
-                $uprefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('ansel', array(
+                $uprefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('ansel', array(
                     'cache' => false,
                     'owner' => $this->_owner
                 ));
index 6dc8b88..b0b833b 100644 (file)
@@ -3,7 +3,7 @@ if ($group == $GLOBALS['registry']->getAuth()) {
     $owner_title = _("My Galleries");
     $custom = '';
 } elseif (!empty($GLOBALS['conf']['gallery']['customlabel'])) {
-    $uprefs = $injector->getInstance('Horde_Prefs')->getPrefs('ansel', array(
+    $uprefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('ansel', array(
         'cache' => false,
         'user' => $group
     ));
index a0eb899..6e46b18 100644 (file)
@@ -143,7 +143,7 @@ class Folks_Friends {
             return (boolean)$GLOBALS['prefs']->getValue('friends_approval');
         }
 
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('folks', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('folks', array(
             'cache' => false,
             'user' => $GLOBALS['registry']->convertUsername($user, true)
         ));
diff --git a/framework/Core/lib/Horde/Core/Binder/Prefs.php b/framework/Core/lib/Horde/Core/Binder/Prefs.php
deleted file mode 100644 (file)
index 4c02540..0000000
+++ /dev/null
@@ -1,17 +0,0 @@
-<?php
-/**
- * @category Horde
- * @package  Core
- */
-class Horde_Core_Binder_Prefs implements Horde_Injector_Binder
-{
-    public function create(Horde_Injector $injector)
-    {
-        return new Horde_Core_Factory_Prefs($injector);
-    }
-
-    public function equals(Horde_Injector_Binder $binder)
-    {
-        return false;
-    }
-}
index fd4311e..ac239e3 100644 (file)
@@ -82,7 +82,7 @@ class Horde_Core_Factory_Identity
             if (isset($prefs) && ($params['user'] == $registry->getAuth())) {
                 $params['prefs'] = $prefs;
             } else {
-                $params['prefs'] = $injector->getInstance('Horde_Prefs')->getPrefs($registry->getApp(), array(
+                $params['prefs'] = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs($registry->getApp(), array(
                     'cache' => false,
                     'user' => $user
                 ));
index 97cf7e4..0457b2b 100644 (file)
@@ -73,7 +73,7 @@ class Horde_Core_Sidebar
         }
 
         if (Horde_Menu::showService('prefs') &&
-            !($injector->getInstance('Horde_Prefs')->getPrefs() instanceof Horde_Prefs_Session)) {
+            !($injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs() instanceof Horde_Prefs_Session)) {
             $menu['prefs'] = array(
                 'icon' => Horde_Themes::img('prefs.png'),
                 'name' => _("Preferences"),
index 7542c8f..0d80281 100644 (file)
@@ -282,7 +282,6 @@ class Horde_Registry
             'Horde_Memcache' => 'Horde_Core_Binder_Memcache',
             'Horde_Mime_Viewer' => 'Horde_Core_Binder_MimeViewer',
             'Horde_Notification' => 'Horde_Core_Binder_Notification',
-            'Horde_Prefs' => 'Horde_Core_Binder_Prefs',
             'Horde_Prefs_Identity' => 'Horde_Core_Binder_Identity',
             // 'Horde_Registry' - initialized below
             'Horde_Secret' => 'Horde_Core_Binder_Secret',
@@ -1404,7 +1403,7 @@ class Horde_Registry
             );
         }
 
-        $prefs = $injector->getInstance('Horde_Prefs')->getPrefs($app, $opts);
+        $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs($app, $opts);
     }
 
     /**
@@ -1729,7 +1728,7 @@ class Horde_Registry
         unset($_SESSION['horde_auth']);
 
         /* Remove the user's cached preferences if they are present. */
-        $GLOBALS['injector']->getInstance('Horde_Prefs')->clearCache();
+        $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->clearCache();
 
         if ($destroy) {
             session_destroy();
@@ -2118,7 +2117,7 @@ class Horde_Registry
 
         /* Reload preferences for the new user. */
         unset($GLOBALS['prefs']);
-        $GLOBALS['injector']->getInstance('Horde_Prefs')->clearCache();
+        $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->clearCache();
         $this->loadPrefs();
 
         $this->setLanguageEnvironment($GLOBALS['prefs']->getValue('language'), $app);
index 13e8928..6aa2a80 100644 (file)
@@ -24,7 +24,7 @@ Application Framework.</description>
   <active>yes</active>
  </developer>
  <date>2010-10-02</date>
- <time>22:28:34</time>
+ <time>22:52:58</time>
  <version>
   <release>0.1.0</release>
   <api>0.1.0</api>
@@ -110,6 +110,7 @@ Application Framework.</description>
        </dir> <!-- /lib/Horde/Core/Autoloader/Callback -->
       </dir> <!-- /lib/Horde/Core/Autoloader -->
       <dir name="Binder">
+       <file name="Alarm.php" role="php" />
        <file name="Auth.php" role="php" />
        <file name="AuthFactory.php" role="php" />
        <file name="AuthSignup.php" role="php" />
@@ -131,7 +132,6 @@ Application Framework.</description>
        <file name="Memcache.php" role="php" />
        <file name="MimeViewer.php" role="php" />
        <file name="Notification.php" role="php" />
-       <file name="Prefs.php" role="php" />
        <file name="Secret.php" role="php" />
        <file name="SessionHandler.php" role="php" />
        <file name="ShareFactory.php" role="php" />
@@ -431,6 +431,7 @@ Application Framework.</description>
    <install as="Horde/Core/Auth/Signup/SqlObject.php" name="lib/Horde/Core/Auth/Signup/SqlObject.php" />
    <install as="Horde/Core/Autoloader/Callback/Mime.php" name="lib/Horde/Core/Autoloader/Callback/Mime.php" />
    <install as="Horde/Core/Autoloader/Callback/Nls.php" name="lib/Horde/Core/Autoloader/Callback/Nls.php" />
+   <install as="Horde/Core/Binder/Alarm.php" name="lib/Horde/Core/Binder/Alarm.php" />
    <install as="Horde/Core/Binder/Auth.php" name="lib/Horde/Core/Binder/Auth.php" />
    <install as="Horde/Core/Binder/AuthFactory.php" name="lib/Horde/Core/Binder/AuthFactory.php" />
    <install as="Horde/Core/Binder/AuthSignup.php" name="lib/Horde/Core/Binder/AuthSignup.php" />
@@ -452,7 +453,6 @@ Application Framework.</description>
    <install as="Horde/Core/Binder/Memcache.php" name="lib/Horde/Core/Binder/Memcache.php" />
    <install as="Horde/Core/Binder/MimeViewer.php" name="lib/Horde/Core/Binder/MimeViewer.php" />
    <install as="Horde/Core/Binder/Notification.php" name="lib/Horde/Core/Binder/Notification.php" />
-   <install as="Horde/Core/Binder/Prefs.php" name="lib/Horde/Core/Binder/Prefs.php" />
    <install as="Horde/Core/Binder/Secret.php" name="lib/Horde/Core/Binder/Secret.php" />
    <install as="Horde/Core/Binder/SessionHandler.php" name="lib/Horde/Core/Binder/SessionHandler.php" />
    <install as="Horde/Core/Binder/ShareFactory.php" name="lib/Horde/Core/Binder/ShareFactory.php" />
index 7a1b6f4..4152c4b 100644 (file)
@@ -220,7 +220,7 @@ class Horde_Api extends Horde_Registry_Api
         $haveError = false;
 
         /* Remove user's prefs */
-        $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('horde', array(
+        $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
             'user' => $user
         ));
         try {
index 9ca65c5..db173bc 100755 (executable)
@@ -57,7 +57,7 @@ foreach($files as $file) {
     $cli->message('Importing ' . $user . '\'s preferences');
 
     // Reset user prefs
-    $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('horde', array(
+    $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
         'cache' => false,
         'user' => $user
     ));
index 341691e..9d934f9 100755 (executable)
@@ -64,7 +64,7 @@ function importPrefs()
 
     $GLOBALS['registry']->setAuth($user, array());
     $cli->message('Importing ' . $user . '\'s preferences');
-    $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('horde', array(
+    $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
         'cache' => false,
         'user' => $user
     ));
index beb59eb..ee33ff2 100644 (file)
@@ -34,7 +34,7 @@ $can_validate = false;
 /* If a username has been supplied try fetching the prefs stored info. */
 if ($username = $vars->get('username')) {
     $username = Horde_Auth::addHook($username);
-    $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('horde', array(
+    $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
         'cache' => false,
         'user' => $username
     ));
index b5d3212..2945473 100644 (file)
@@ -70,7 +70,7 @@ if ($conf['compose']['link_attachments_notify']) {
 
             /* Load $mail_user's preferences so that we can use their
              * locale information for the notification message. */
-            $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('horde', array(
+            $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
                 'cache' => false,
                 'user' => $mail_user
             ));
index 0b0b71e..056a236 100644 (file)
@@ -34,7 +34,7 @@ class Ingo_Storage_Prefs extends Ingo_Storage
      */
     protected function _retrieve($field, $readonly = false)
     {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('ingo', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('ingo', array(
             'cache' => false,
             'user' => Ingo::getUser()
         ));
@@ -131,7 +131,7 @@ class Ingo_Storage_Prefs extends Ingo_Storage
      */
     protected function _store($ob)
     {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('ingo', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('ingo', array(
             'cache' => false,
             'user' => Ingo::getUser()
         ));
index d083752..94f4554 100644 (file)
@@ -31,7 +31,7 @@ $key = 'kronolith.fb.' . ($user ? 'u.' . $user : 'c.' . $cal);
 $fb = $cache->get($key, 360);
 if (!$fb) {
     if ($user) {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('kronolith', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('kronolith', array(
             'cache' => false,
             'user' => $user
         ));
index e653d4a..c1bdb78 100644 (file)
@@ -1218,7 +1218,7 @@ class Kronolith_Api extends Horde_Registry_Api
                     if ($alarm_user == $current_user) {
                         $prefs = $GLOBALS['prefs'];
                     } else {
-                        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('kronolith', array(
+                        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('kronolith', array(
                             'cache' => false,
                             'user' => $alarm_user
                         ));
index a2546d5..615bf0b 100644 (file)
@@ -2553,7 +2553,7 @@ class Kronolith
      */
     public static function _notificationPref($user, $mode, $calendar = null)
     {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('kronolith', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('kronolith', array(
             'cache' => false,
             'user' => $user
         ));
index 3cfe4a2..d3c7573 100755 (executable)
@@ -59,7 +59,7 @@ function send_agendas()
 
     // Loop through the users and generate an agenda for them
     foreach ($users as $user) {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('kronolith', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('kronolith', array(
             'user' => $user
         ));
         $agenda_calendars = $prefs->getValue('daily_agenda');
index 0b7bb10..acb3e20 100755 (executable)
@@ -34,7 +34,7 @@ while ($row = $result->fetchRow(DB_FETCHMODE_ASSOC)) {
         if (!is_null($creator)) {
             echo "$count\n";
         }
-        $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('horde', array(
+        $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('horde', array(
             'cache' => false,
             'user' => $row['event_creator_id']
         ));
index 30409e7..4df09c8 100644 (file)
@@ -1454,7 +1454,7 @@ class Nag_Api extends Horde_Registry_Api
                 $users = array($user);
             }
             foreach ($users as $alarm_user) {
-                $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('nag', array(
+                $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('nag', array(
                     'cache' => false,
                     'user' => $alarm_user
                 ));
index 2dcad84..3bf0a8c 100644 (file)
@@ -1068,7 +1068,7 @@ class Nag
      */
     public static function _notificationPref($user, $mode, $tasklist = null)
     {
-        $prefs = $GLOBALS['injector']->getInstance('Horde_Prefs')->getPrefs('nag', array(
+        $prefs = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Prefs')->getPrefs('nag', array(
             'cache' => false,
             'user' => $user
         ));
index 0f60edc..52e3a95 100755 (executable)
@@ -57,7 +57,7 @@ foreach($files as $file) {
 
     // Reset user prefs
     unset($prefs);
-    $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('turba', array(
+    $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('turba', array(
         'cache' => false,
         'user' => $user
     ));
index 18a6710..d54581d 100755 (executable)
@@ -53,7 +53,7 @@ while ($row = $handle->fetchRow(DB_FETCHMODE_ASSOC)) {
         $cli->message('Importing ' . $user . '\'s address book');
 
         // Reset user prefs
-        $prefs = $injector->getInstance('Horde_Prefs')->getPrefs('turba', array(
+        $prefs = $injector->getInstance('Horde_Core_Factory_Prefs')->getPrefs('turba', array(
             'cache' => false,
             'user' => $user
         ));