No need for this binder, call the factory directly. Standardize on using create(...

diff --git a/turba/add.php b/turba/add.php
index ee42629..734200e 100644 (file)
--- a/turba/add.php
+++ b/turba/add.php
@@ -34,7 +34,7 @@ if (!$addSources) {
 /* A source has been selected, connect and set up the fields. */
 if ($source) {
     try {
-        $driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+        $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
     } catch (Turba_Exception $e) {
         $notification->push($e, 'horde.error');
         $driver = null;

diff --git a/turba/contact.php b/turba/contact.php
index 68102fb..56c747a 100644 (file)
--- a/turba/contact.php
+++ b/turba/contact.php
@@ -22,7 +22,7 @@ if (!isset($GLOBALS['cfgSources'][$source])) {
 
 /* Set the contact from the key requested. */
 try {
-    $driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+    $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 } catch (Turba_Exception $e) {
     $notification->push($e, 'horde.error');
     Horde::url($prefs->getValue('initial_page'), true)->redirect();

diff --git a/turba/data.php b/turba/data.php
index 7810402..e635a30 100644 (file)
--- a/turba/data.php
+++ b/turba/data.php
@@ -242,7 +242,7 @@ case 'export':
     foreach ($sources as $source => $objectkeys) {
         /* Create a Turba storage instance. */
         try {
-            $driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
         } catch (Turba_Exception $e) {
             $notification->push($e, 'horde.error');
             $error = true;
@@ -335,7 +335,7 @@ case 'export':
 case Horde_Data::IMPORT_FILE:
     $dest = Horde_Util::getFormData('dest');
     try {
-        $driver = $injector->getInstance('Turba_Driver')->getDriver($dest);
+        $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($dest);
     } catch (Turba_Exception $e) {
         $notification->push($e, 'horde.error');
         $error = true;
@@ -423,7 +423,7 @@ if (is_array($next_step)) {
     /* Create a Turba storage instance. */
     $dest = $session->get('horde', 'import_data/target');
     try {
-        $driver = $injector->getInstance('Turba_Driver')->getDriver($dest);
+        $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($dest);
     } catch (Turba_Exception $e) {
         $notification->push($e, 'horde.error');
         $driver = null;

diff --git a/turba/delete.php b/turba/delete.php
index 9de84c8..8f37d86 100644 (file)
--- a/turba/delete.php
+++ b/turba/delete.php
@@ -18,7 +18,7 @@ Horde_Registry::appInit('turba');
 
 $source = Horde_Util::getFormData('source');
 $key = Horde_Util::getFormData('key');
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
 if ($conf['documents']['type'] != 'none') {
     try {

diff --git a/turba/deletefile.php b/turba/deletefile.php
index a67f296..778499e 100644 (file)
--- a/turba/deletefile.php
+++ b/turba/deletefile.php
@@ -23,7 +23,7 @@ if ($source === null || !isset($cfgSources[$source])) {
     Horde::url($prefs->getValue('initial_page'), true)->redirect();
 }
 
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
 try {
     $contact = $driver->getObject(Horde_Util::getPost('key'));

diff --git a/turba/edit.php b/turba/edit.php
index de49e45..b60ccfb 100644 (file)
--- a/turba/edit.php
+++ b/turba/edit.php
@@ -45,7 +45,7 @@ if ($source === null || !isset($cfgSources[$source])) {
     $url->redirect();
 }
 
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
 /* Set the contact from the requested key. */
 try {

diff --git a/turba/lib/Api.php b/turba/lib/Api.php
index 461bc91..2ed619e 100644 (file)
--- a/turba/lib/Api.php
+++ b/turba/lib/Api.php
@@ -51,7 +51,7 @@ class Turba_Api extends Horde_Registry_Api
         @list($source, $key) = explode('.', $id, 2);
         if (isset($GLOBALS['cfgSources'][$source]) && $key) {
             try {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
                 $object = $driver->getObject($key)->getValue('name');
             } catch (Turba_Exception $e) {}
         }
@@ -322,7 +322,7 @@ class Turba_Api extends Horde_Registry_Api
             }
 
             // Load the Turba driver.
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($parts[1]);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($parts[1]);
 
             $contacts = $driver->search(array());
 
@@ -371,7 +371,7 @@ class Turba_Api extends Horde_Registry_Api
             }
 
             // Load the Turba driver.
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($parts[1]);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($parts[1]);
 
             $contact = $driver->getObject($parts[2]);
 
@@ -418,7 +418,7 @@ class Turba_Api extends Horde_Registry_Api
         }
 
         // Load the Turba driver.
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($parts[1]);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($parts[1]);
 
         return $driver->delete($parts[2]);
     }
@@ -458,7 +458,7 @@ class Turba_Api extends Horde_Registry_Api
                 throw new Turba_Exception(sprintf(_("Invalid address book: %s"), $source));
             }
 
-            $storage = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $storage = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             try {
                 $results = $storage->search(array());
@@ -518,7 +518,7 @@ class Turba_Api extends Horde_Registry_Api
                 throw new Turba_Exception(sprintf(_("Invalid address book: %s"), $source));
             }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             $histories = $history->getByTimestamp(
                 '>', $timestamp, $filter,
@@ -589,7 +589,7 @@ class Turba_Api extends Horde_Registry_Api
                 throw new Turba_Exception(sprintf(_("Invalid address book: %s"), $source));
             }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             $ts = $history->getActionTimestamp('turba:' . $driver->getName()
                 . ':' . $uid,
@@ -635,7 +635,7 @@ class Turba_Api extends Horde_Registry_Api
             throw new Turba_Exception(sprintf(_("Invalid address book: %s"), $import_source));
         }
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($import_source);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($import_source);
 
         if (!$driver->hasPermission(Horde_Perms::EDIT)) {
             throw new Turba_Exception(_("Permission denied"));
@@ -763,7 +763,7 @@ class Turba_Api extends Horde_Registry_Api
                 throw new Turba_Exception(_("Invalid ID"));
             }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             if (!$driver->hasPermission(Horde_Perms::READ)) {
                 continue;
@@ -827,7 +827,7 @@ class Turba_Api extends Horde_Registry_Api
     public function ownVCard()
     {
         $contact = $this->getOwnContactObject();
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($contact['source']);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($contact['source']);
 
         $vcard = $driver->tovCard($contact['contact'], '3.0', null, true);
         $vcard->setAttribute('VERSION', '3.0');
@@ -868,7 +868,7 @@ class Turba_Api extends Horde_Registry_Api
             throw new Turba_Exception(_("The address book with your own contact doesn't exist anymore."));
         }
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
         if (!$driver->hasPermission(Horde_Perms::READ)) {
             throw new Turba_Exception(_("You don't have sufficient permissions to read the address book that contains your own contact."));
@@ -933,7 +933,7 @@ class Turba_Api extends Horde_Registry_Api
                 throw new Turba_Exception(_("Invalid ID"));
             }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             if (!$GLOBALS['registry']->isAdmin() &&
                 !$driver->hasPermission(Horde_Perms::DELETE)) {
@@ -996,7 +996,7 @@ class Turba_Api extends Horde_Registry_Api
             }
 
             // Check permissions.
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
             if (!$driver->hasPermission(Horde_Perms::EDIT)) {
                 continue;
             }
@@ -1121,7 +1121,7 @@ class Turba_Api extends Horde_Registry_Api
                     continue;
                 }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             // Determine the name of the column to sort by.
             $columns = isset($sort_columns[$source])
@@ -1274,7 +1274,7 @@ class Turba_Api extends Horde_Registry_Api
         }
 
         if (isset($cfgSources[$source])) {
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             $object = $driver->getObject($objectId);
 
@@ -1310,7 +1310,7 @@ class Turba_Api extends Horde_Registry_Api
         }
 
         if (isset($cfgSources[$source])) {
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
             $objects = $driver->getObjects($objectIds);
 
@@ -1351,7 +1351,7 @@ class Turba_Api extends Horde_Registry_Api
         $results = array();
         foreach ($sources as $source) {
             if (isset($cfgSources[$source])) {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
                 $res = $driver->search(array());
                 if (!($res instanceof Turba_List)) {
@@ -1419,7 +1419,7 @@ class Turba_Api extends Horde_Registry_Api
         $objects = array();
         foreach ($time_categories as $category) {
             list($category, $source) = explode('/', $category, 2);
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
             $objects = array_merge($objects, $driver->listTimeObjects($start, $end, $category));
         }
 
@@ -1563,7 +1563,7 @@ class Turba_Api extends Horde_Registry_Api
             throw new Turba_Exception(_("Invalid entry"));
         }
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
         if (!$driver->hasPermission(Horde_Perms::EDIT)) {
             throw new Turba_Exception(_("Permission denied"));
@@ -1654,7 +1654,7 @@ class Turba_Api extends Horde_Registry_Api
                 continue;
             }
 
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
             $criterium = array('email' => $address);
             if (!isset($driver->map['email'])) {
                 if (isset($driver->map['emails'])) {
@@ -1719,7 +1719,7 @@ class Turba_Api extends Horde_Registry_Api
 
         foreach ($sources as $source) {
             if (isset($cfgSources[$source])) {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
                 if (!$driver->hasPermission(Horde_Perms::EDIT)) {
                     continue;
                 }

diff --git a/turba/lib/Application.php b/turba/lib/Application.php
index 39dda94..5d831b2 100644 (file)
--- a/turba/lib/Application.php
+++ b/turba/lib/Application.php
@@ -59,15 +59,6 @@ class Turba_Application extends Horde_Registry_Application
      */
     protected function _init()
     {
-        /* Add Turba-specific binders. */
-        $binders = array(
-            'Turba_Driver' => new Turba_Injector_Binder_Driver()
-        );
-
-        foreach ($binders as $key => $val) {
-            $GLOBALS['injector']->addBinder($key, $val);
-        }
-
         // Turba source and attribute configuration.
         $attributes = Horde::loadConfiguration('attributes.php', 'attributes', 'turba');
         include TURBA_BASE . '/config/backends.php';
@@ -399,7 +390,7 @@ class Turba_Application extends Horde_Registry_Application
             if (empty($source['use_shares'])) {
                 // Shares not enabled for this source
                 try {
-                    $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                    $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
                 } catch (Turba_Exception $e) {
                     Horde::logMessage($e, 'ERR');
                     throw new Turba_Exception(sprintf(_("There was an error removing an address book for %s"), $user));
@@ -432,7 +423,7 @@ class Turba_Application extends Horde_Registry_Application
             if (!empty($params['default'])) {
                 $config = Turba::getSourceFromShare($share);
                 try {
-                    $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($config);
+                    $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($config);
                 } catch (Turba_Exception $e) {
                     continue;
                 }

diff --git a/turba/lib/Driver/Vbook.php b/turba/lib/Driver/Vbook.php
index 5b679a2..4e7e740 100644 (file)
--- a/turba/lib/Driver/Vbook.php
+++ b/turba/lib/Driver/Vbook.php
@@ -48,7 +48,7 @@ class Turba_Driver_Vbook extends Turba_Driver
         $this->_share = $this->_params['share'];
 
         /* Load the underlying driver. */
-        $this->_driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($this->_params['source']);
+        $this->_driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($this->_params['source']);
 
         $this->searchCriteria = empty($this->_params['criteria'])
             ? array()

diff --git a/turba/lib/Form/CreateAddressBook.php b/turba/lib/Form/CreateAddressBook.php
index a64568e..6c39f22 100644 (file)
--- a/turba/lib/Form/CreateAddressBook.php
+++ b/turba/lib/Form/CreateAddressBook.php
@@ -35,7 +35,7 @@ class Turba_Form_CreateAddressBook extends Horde_Form
         // Need a clean cfgSources array
         include TURBA_BASE . '/config/backends.php';
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($cfgSources[$GLOBALS['conf']['shares']['source']]);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($cfgSources[$GLOBALS['conf']['shares']['source']]);
 
         $params = array(
             'params' => array('source' => $GLOBALS['conf']['shares']['source']),

diff --git a/turba/lib/Form/DeleteAddressBook.php b/turba/lib/Form/DeleteAddressBook.php
index bb69134..0b09708 100644 (file)
--- a/turba/lib/Form/DeleteAddressBook.php
+++ b/turba/lib/Form/DeleteAddressBook.php
@@ -50,7 +50,7 @@ class Turba_Form_DeleteAddressBook extends Horde_Form
             throw new Turba_Exception(_("You do not have permissions to delete this address book."));
         }
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($this->_addressbook->getName());
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($this->_addressbook->getName());
 
         // We have a Turba_Driver, try to delete the address book.
         $driver->deleteAll();

diff --git a/turba/lib/Injector/Binder/Driver.php b/turba/lib/Injector/Binder/Driver.php
deleted file mode 100644 (file)
index 51e1f3f..0000000
--- a/turba/lib/Injector/Binder/Driver.php
+++ /dev/null
@@ -1,31 +0,0 @@
-<?php
-/**
- * Binder for Turba_Driver::.
- *
- * Copyright 2010 The Horde Project (http://www.horde.org/)
- *
- * See the enclosed file COPYING for license information (ASL). If you
- * did not receive this file, see http://www.horde.org/licenses/asl.html.
- *
- * @author   Michael Slusarz <slusarz@horde.org>
- * @category Horde
- * @license  http://www.horde.org/licenses/asl.html ASL
- * @package  Turba
- */
-class Turba_Injector_Binder_Driver implements Horde_Injector_Binder
-{
-    /**
-     */
-    public function create(Horde_Injector $injector)
-    {
-        return new Turba_Injector_Factory_Driver($injector);
-    }
-
-    /**
-     */
-    public function equals(Horde_Injector_Binder $binder)
-    {
-        return false;
-    }
-
-}

diff --git a/turba/lib/Injector/Factory/Driver.php b/turba/lib/Injector/Factory/Driver.php
index 43dcf66..658e8b2 100644 (file)
--- a/turba/lib/Injector/Factory/Driver.php
+++ b/turba/lib/Injector/Factory/Driver.php
@@ -60,7 +60,7 @@ class Turba_Injector_Factory_Driver
      * @return Turba_Driver  The singleton instance.
      * @throws Turba_Exception
      */
-    public function getDriver($name)
+    public function create($name)
     {
         if (is_array($name)) {
             $key = md5(serialize($name));

diff --git a/turba/lib/List.php b/turba/lib/List.php
index 39e35bc..a8c231c 100644 (file)
--- a/turba/lib/List.php
+++ b/turba/lib/List.php
@@ -38,7 +38,7 @@ class Turba_List implements Countable
         foreach ($ids as $value) {
             list($source, $key) = explode(':', $value);
             try {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
                 $this->insert($driver->getObject($key));
             } catch (Turba_Exception $e) {}
         }

diff --git a/turba/lib/LoginTasks/SystemTask/UpgradeLists.php b/turba/lib/LoginTasks/SystemTask/UpgradeLists.php
index bba646d..6c4adeb 100644 (file)
--- a/turba/lib/LoginTasks/SystemTask/UpgradeLists.php
+++ b/turba/lib/LoginTasks/SystemTask/UpgradeLists.php
@@ -40,7 +40,7 @@ class Turba_LoginTasks_SystemTask_UpgradeLists extends Horde_LoginTasks_SystemTa
             $sources = array_keys($GLOBALS['cfgSources']);
             foreach ($sources as $sourcekey) {
                 try {
-                    $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($sourcekey);
+                    $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($sourcekey);
                     $lists = $driver->search($criteria);
                 } catch (Turba_Exception $e) {
                     return false;

diff --git a/turba/lib/Object/Group.php b/turba/lib/Object/Group.php
index b627317..156670b 100644 (file)
--- a/turba/lib/Object/Group.php
+++ b/turba/lib/Object/Group.php
@@ -67,7 +67,7 @@ class Turba_Object_Group extends Turba_Object {
         if ($sourceId == $this->getSource()) {
             $contact = $this->driver->getObject($contactId);
         } else {
-            $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($sourceId);
+            $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($sourceId);
             $contact = $driver->getObject($contactId);
         }
 
@@ -170,7 +170,7 @@ class Turba_Object_Group extends Turba_Object {
                 }
 
                 try {
-                    $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($sourceId);
+                    $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($sourceId);
                 } catch (Turba_Exception $e) {
                     continue;
                 }

diff --git a/turba/lib/Turba.php b/turba/lib/Turba.php
index 38b9ca9..5ad3b50 100644 (file)
--- a/turba/lib/Turba.php
+++ b/turba/lib/Turba.php
@@ -353,7 +353,7 @@ class Turba {
 
         foreach ($in as $sourceId => $source) {
             try {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($sourceId);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($sourceId);
             } catch (Turba_Exception $e) {
                 Horde::logMessage($e, 'ERR');
                 continue;
@@ -440,7 +440,7 @@ class Turba {
             if ($GLOBALS['registry']->getAuth() && !$personal) {
                 // User's default share is missing.
                 try {
-                    $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                    $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
                 } catch (Turba_Exception $e) {
                     $GLOBALS['notification']->push($driver, 'horde.error');
                     continue;

diff --git a/turba/lib/View/Browse.php b/turba/lib/View/Browse.php
index b265e89..7aaea16 100644 (file)
--- a/turba/lib/View/Browse.php
+++ b/turba/lib/View/Browse.php
@@ -69,7 +69,7 @@ class Turba_View_Browse {
             $notification->push(_("There are no browseable address books."), 'horde.warning');
         } else {
             try {
-                $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($source);
+                $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($source);
             } catch (Turba_Exception $e) {
                 $notification->push($e, 'horde.error');
                 unset($driver);
@@ -141,7 +141,7 @@ class Turba_View_Browse {
                 $targetSource = $vars->get('targetAddressbook');
 
                 try {
-                    $targetDriver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($targetSource);
+                    $targetDriver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($targetSource);
                 } catch (Turba_Exception $e) {
                     $notification->push($e, 'horde.error');
                     break;
@@ -170,7 +170,7 @@ class Turba_View_Browse {
 
                     // Try and load the driver for the source.
                     try {
-                        $sourceDriver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($objectSource);
+                        $sourceDriver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($objectSource);
                     } catch (Turba_Exception $e) {
                         $notification->push($e, 'horde.error');
                         continue;
@@ -256,7 +256,7 @@ class Turba_View_Browse {
                     }
 
                     try {
-                        $targetDriver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($targetSource);
+                        $targetDriver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($targetSource);
                     } catch (Turba_Exception $e) {
                         $notification->push($e, 'horde.error');
                         break;
@@ -271,7 +271,7 @@ class Turba_View_Browse {
                 } else {
                     $targetSource = $vars->get('targetAddressbook');
                     try {
-                        $targetDriver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($targetSource);
+                        $targetDriver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($targetSource);
                     } catch (Turba_Exception $e) {
                         $notification->push($e, 'horde.error');
                         break;

diff --git a/turba/lib/View/List.php b/turba/lib/View/List.php
index 75fc1b4..5e141d7 100644 (file)
--- a/turba/lib/View/List.php
+++ b/turba/lib/View/List.php
@@ -145,7 +145,7 @@ class Turba_View_List implements Countable
     {
         global $prefs, $session, $default_source, $copymove_source_options;
 
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($default_source);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($default_source);
 
         $hasDelete = $driver->hasPermission(Horde_Perms::DELETE);
         $hasEdit = $driver->hasPermission(Horde_Perms::EDIT);
@@ -394,7 +394,7 @@ class Turba_View_List implements Countable
                                             'name' => '  ' . htmlspecialchars($srcConfig['title']),
                                             'source' => htmlspecialchars($src));
 
-                $srcDriver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($src);
+                $srcDriver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($src);
                 try {
                     $listList = $srcDriver->search(
                         array('__type' => 'Group'),

diff --git a/turba/lib/tests/KolabTest.php b/turba/lib/tests/KolabTest.php
index 0635f48..38188c3 100644 (file)
--- a/turba/lib/tests/KolabTest.php
+++ b/turba/lib/tests/KolabTest.php
@@ -47,14 +47,14 @@ class Turba_KolabTest extends Turba_KolabTestBase {
         $this->_kolab->_storage->save($object);
 
         // Check that the driver can be created
-        $turba = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver('wrobel@example.org');
+        $turba = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create('wrobel@example.org');
         //$this->assertNoError($turba);
 
         $result = $turba->search(array(), array('last-name'));
         $this->assertNoError($result);
         $this->assertEquals(2, count($result));
 
-        $turba = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver('INBOX%2Ftest2');
+        $turba = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create('INBOX%2Ftest2');
         $result = $turba->search(array(), array('last-name'));
 
         $this->assertEquals(0, count($result));
@@ -75,7 +75,7 @@ class Turba_KolabTest extends Turba_KolabTestBase {
         );
 
         // Save the contact
-        $turba = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver('wrobel@example.org');
+        $turba = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create('wrobel@example.org');
         //$this->assertNoError($turba);
 
         $this->assertNoError($turba->_add($object));
@@ -102,7 +102,7 @@ class Turba_KolabTest extends Turba_KolabTestBase {
         );
 
         // Save the contact
-        $turba = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver('wrobel@example.org');
+        $turba = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create('wrobel@example.org');
         //$this->assertNoError($turba);
 
         $this->assertNoError($turba->_add($object));

diff --git a/turba/merge.php b/turba/merge.php
index a2be0b7..b288a17 100644 (file)
--- a/turba/merge.php
+++ b/turba/merge.php
@@ -16,7 +16,7 @@ Horde_Registry::appInit('turba');
 $source = Horde_Util::getFormData('source');
 $key = Horde_Util::getFormData('key');
 $mergeInto = Horde_Util::getFormData('merge_into');
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
 if ($url = Horde_Util::getFormData('url')) {
     $url = new Horde_Url($url, true);

diff --git a/turba/minisearch.php b/turba/minisearch.php
index f179f76..66a7e21 100644 (file)
--- a/turba/minisearch.php
+++ b/turba/minisearch.php
@@ -22,7 +22,7 @@ $source = Horde_Util::getFormData('source', Turba::getDefaultAddressBook());
 // Do the search if we have one.
 if (!is_null($search)) {
     try {
-        $driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+        $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
         $criteria['name'] = trim($search);
         $res = $driver->search($criteria);
 

diff --git a/turba/scripts/import_squirrelmail_file_abook.php b/turba/scripts/import_squirrelmail_file_abook.php
index b2cbaed..344dc6c 100755 (executable)
--- a/turba/scripts/import_squirrelmail_file_abook.php
+++ b/turba/scripts/import_squirrelmail_file_abook.php
@@ -83,7 +83,7 @@ foreach($files as $file) {
 
     // Initiate driver
     try {
-        $driver = $GLOBALS['injector']->getInstance('Turba_Driver')->getDriver($import_source);
+        $driver = $GLOBALS['injector']->getInstance('Turba_Injector_Factory_Driver')->create($import_source);
     } catch (Turba_Exception $e) {
         PEAR::raiseError(sprintf(_("Connection failed: %s"), $e->getMessage()), 'horde.error', null, null, $import_source);
         continue;

diff --git a/turba/scripts/import_squirrelmail_sql_abook.php b/turba/scripts/import_squirrelmail_sql_abook.php
index a838676..3aa47f0 100755 (executable)
--- a/turba/scripts/import_squirrelmail_sql_abook.php
+++ b/turba/scripts/import_squirrelmail_sql_abook.php
@@ -92,7 +92,7 @@ while ($row = $handle->fetchRow(DB_FETCHMODE_ASSOC)) {
 
         // Initiate driver
         try {
-            $driver = $injector->getInstance('Turba_Driver')->getDriver($import_source);
+            $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($import_source);
         } catch (Turba_Exception $e) {
             $cli->message('  ' . sprintf(_("Connection failed: %s"), $e->getMessage()), 'cli.error');
             continue;

diff --git a/turba/scripts/upgrades/public_to_horde_share.php b/turba/scripts/upgrades/public_to_horde_share.php
index 0243bce..eb518bb 100755 (executable)
--- a/turba/scripts/upgrades/public_to_horde_share.php
+++ b/turba/scripts/upgrades/public_to_horde_share.php
@@ -68,7 +68,7 @@ $share->save();
 $CLI->message('Created new Horde_Share object for the shared address book.', 'cli.success');
 
 // Share created, now get a Turba_Driver and make the changes.
-$driver = $injector->getInstance('Turba_Driver')->getDriver($sourceKey);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($sourceKey);
 
 $db = &$driver->_db;
 

diff --git a/turba/search.php b/turba/search.php
index be56e21..8238fdb 100644 (file)
--- a/turba/search.php
+++ b/turba/search.php
@@ -85,7 +85,7 @@ $val = Horde_Util::getFormData('val');
 $action = Horde_Util::getFormData('actionID');
 
 try {
-    $driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+    $driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 } catch (Turba_Exception $e) {
     $notification->push($e, 'horde.error');
     $driver = null;

diff --git a/turba/vcard.php b/turba/vcard.php
index d3fbae5..eaa6c15 100644 (file)
--- a/turba/vcard.php
+++ b/turba/vcard.php
@@ -19,7 +19,7 @@ if (!isset($cfgSources[$source])) {
     Horde::url($prefs->getValue('initial_page'), true)->redirect();
 }
 
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 
 /* Set the contact from the key requested. */
 try {

diff --git a/turba/view.php b/turba/view.php
index b4db025..377640a 100644 (file)
--- a/turba/view.php
+++ b/turba/view.php
@@ -28,7 +28,7 @@ if (!isset($cfgSources[$source])) {
     throw new Turba_Exception(_("The contact you requested does not exist."));
 }
 
-$driver = $injector->getInstance('Turba_Driver')->getDriver($source);
+$driver = $injector->getInstance('Turba_Injector_Factory_Driver')->create($source);
 $object = $driver->getObject($key);
 
 /* Check permissions. */