Remove unnesessary binder, ask for the Horde_Core_Factory_Share object directly
authorMichael J. Rubinsky <mrubinsk@horde.org>
Thu, 7 Oct 2010 21:14:22 +0000 (17:14 -0400)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Thu, 7 Oct 2010 21:46:17 +0000 (17:46 -0400)
24 files changed:
ansel/lib/Injector/Factory/Storage.php
folks/lib/Driver.php
folks/lib/Friends/shared.php
folks/perms.php
framework/Core/lib/Horde/Core/Binder/ShareFactory.php [deleted file]
framework/Core/lib/Horde/Core/Factory/Share.php
framework/Core/lib/Horde/Registry.php
framework/Core/package.xml
framework/Group/lib/Horde/Group/Contactlists.php
framework/Kolab_Storage/test/Horde/Kolab/Storage/Scenario.php
framework/Prefs/lib/Horde/Prefs/KolabImap.php
horde/lib/Api.php
horde/services/shares/edit.php
ingo/lib/Application.php
kronolith/lib/Application.php
kronolith/perms.php
mnemo/lib/Block/summary.php
mnemo/lib/Mnemo.php
nag/lib/Application.php
nag/lib/Block/summary.php
turba/lib/Application.php
turba/lib/tests/KolabTestBase.php
turba/scripts/import_squirrelmail_sql_abook.php
whups/lib/Query.php

index 743b8ea..a75893f 100644 (file)
@@ -37,7 +37,7 @@ class Ansel_Injector_Factory_Storage
     {
         $scope = $this->_injector->getInstance('Ansel_Config')->get('scope');
         if (empty($this->_instances[$scope])) {
-            $this->_instances[$scope] = new Ansel_Storage($this->_injector->getInstance('Horde_Share_Factory')->getScope($scope, 'Sql_Hierarchical'));
+            $this->_instances[$scope] = new Ansel_Storage($this->_injector->getInstance('Horde_Core_Factory_Share')->create($scope, 'Sql_Hierarchical'));
         }
 
        return $this->_instances[$scope];
index 00ae4b4..5e78b0f 100644 (file)
@@ -463,7 +463,7 @@ class Folks_Driver {
 
         // Delete groups
         if ($GLOBALS['conf']['friends']) {
-            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+            $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
             $groups = $shares->listShares($GLOBALS['registry']->getAuth(), Horde_Perms::SHOW, true);
             foreach ($groups as $share) {
                 $result = $shares->removeShare($share);
index 4fb8f3f..6776358 100644 (file)
@@ -42,7 +42,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
      */
     public function getGroupOwner($group)
     {
-        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         try {
             $share = $GLOBALS['folks_shares']->getShareById($group);
@@ -57,7 +57,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
      */
     protected function _getGroups()
     {
-        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
         $groups = $GLOBALS['folks_shares']->listShares($this->_user, Horde_Perms::READ);
 
         $list = array();
@@ -79,7 +79,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
             return PEAR::raiseError(_("A group names cannot be empty"));
         }
 
-        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
         $share = $GLOBALS['folks_shares']->getShareById($group);
 
         // Only owners of a group can delete them
@@ -107,7 +107,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
      */
     public function removeGroup($group)
     {
-        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         $share = $GLOBALS['folks_shares']->getShareById($group);
         if ($share instanceof PEAR_Error) {
@@ -158,7 +158,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
             return PEAR::raiseError(sprintf(_("You already have a group named \"%s\"."), $name));
         }
 
-        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         $share = $GLOBALS['folks_shares']->newShare(strval(new Horde_Support_Uuid()));
 
index 5d7c7c9..54f4e61 100644 (file)
@@ -11,7 +11,7 @@
 
 require_once dirname(__FILE__) . '/lib/base.php';
 
-$shares = $injector->getInstance('Horde_Share_Factory')->getScope();
+$shares = $injector->getInstance('Horde_Core_Factory_Share')->create();
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth_Factory')->getAuth();
 
diff --git a/framework/Core/lib/Horde/Core/Binder/ShareFactory.php b/framework/Core/lib/Horde/Core/Binder/ShareFactory.php
deleted file mode 100644 (file)
index 36f4cb9..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-<?php
-/**
- * @category Horde
- * @package  Core
- */
-class Horde_Core_Binder_ShareFactory implements Horde_Injector_Binder
-{
-    public function create(Horde_Injector $injector)
-    {
-        return new Horde_Core_Factory_Share($injector);
-    }
-
-    public function equals(Horde_Injector_Binder $binder)
-    {
-        return false;
-    }
-
-}
index 7ba5fb0..0870261 100644 (file)
@@ -56,7 +56,7 @@ class Horde_Core_Factory_Share
      * @return Horde_Share  The Horde_Share instance.
      * @throws Horde_Exception
      */
-    public function getScope($app = null, $driver = null)
+    public function create($app = null, $driver = null)
     {
         if (empty($driver)) {
             $driver = $GLOBALS['conf']['share']['driver'];
index 2b49f83..56c93c3 100644 (file)
@@ -252,7 +252,6 @@ class Horde_Registry
         $binders = array(
             'Horde_Auth_Factory' => new Horde_Core_Binder_AuthFactory(),
             'Horde_Core_Auth_Signup' => new Horde_Core_Binder_AuthSignup(),
-            'Horde_Share_Factory' => new Horde_Core_Binder_ShareFactory(),
             'Horde_Template' => new Horde_Core_Binder_Template(),
             'Horde_Text_Filter' => new Horde_Core_Binder_TextFilter(),
             'Horde_Tree' => new Horde_Core_Binder_Tree(),
index d019ce7..94315af 100644 (file)
@@ -113,7 +113,6 @@ Application Framework.</description>
        <file name="Auth.php" role="php" />
        <file name="AuthFactory.php" role="php" />
        <file name="AuthSignup.php" role="php" />
-       <file name="ShareFactory.php" role="php" />
        <file name="Template.php" role="php" />
        <file name="TextFilter.php" role="php" />
        <file name="Tree.php" role="php" />
@@ -426,7 +425,6 @@ Application Framework.</description>
    <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" />
-   <install as="Horde/Core/Binder/ShareFactory.php" name="lib/Horde/Core/Binder/ShareFactory.php" />
    <install as="Horde/Core/Binder/Template.php" name="lib/Horde/Core/Binder/Template.php" />
    <install as="Horde/Core/Binder/TextFilter.php" name="lib/Horde/Core/Binder/TextFilter.php" />
    <install as="Horde/Core/Binder/Tree.php" name="lib/Horde/Core/Binder/Tree.php" />
index f362443..9f27a00 100644 (file)
@@ -339,7 +339,7 @@ class Horde_Group_Contactlists extends Horde_Group
             if ($source['use_shares']) {
                 if (empty($contact_shares)) {
                     $scope = $GLOBALS['registry']->hasInterface('contacts');
-                    $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+                    $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
                     $this->_contact_shares = $shares->listShares($GLOBALS['registry']->getAuth(), Horde_Perms::SHOW, $GLOBALS['registry']->getAuth());
                 }
                 // Contruct a list of owner ids to use
index 84323f9..2b6c54d 100644 (file)
@@ -88,7 +88,7 @@ class Horde_Kolab_Storage_Scenario extends Horde_Kolab_Server_Integration_Scenar
             $folder->setACL($arguments[0], 'alrid');
             break;
         case 'retrieving the list of shares for the application':
-            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($arguments[0], 'kolab');
+            $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($arguments[0], 'kolab');
 
             $world['list'] = $shares->listShares(Auth::getAuth());
             break;
index 7e4e7e7..109749b 100644 (file)
@@ -38,7 +38,7 @@ class Horde_Prefs_KolabImap extends Horde_Prefs_Base
             return;
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope('h-prefs');
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create('h-prefs');
         $default = $shares->getDefaultShare();
         if ($default instanceof PEAR_Error) {
             if ($this->_opts['logger']) {
index 77dc8cd..a0e2bcb 100644 (file)
@@ -476,7 +476,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to add shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->newShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -505,7 +505,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exceptionr(_("You are not allowed to delete shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -532,7 +532,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to list shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         $share_list = &$shares->listShares($userName, Horde_Perms::SHOW, $userName);
         $myshares = array();
@@ -561,7 +561,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to change shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -598,7 +598,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to change shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -640,7 +640,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to change shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -667,7 +667,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to change shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -707,7 +707,7 @@ class Horde_Api extends Horde_Registry_Api
             Horde_Perms::EDIT => 'edit',
             Horde_Perms::DELETE => 'delete');
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -749,7 +749,7 @@ class Horde_Api extends Horde_Registry_Api
             Horde_Perms::EDIT => 'edit',
             Horde_Perms::DELETE => 'delete');
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -786,7 +786,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to list users of shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
@@ -820,7 +820,7 @@ class Horde_Api extends Horde_Registry_Api
             throw new Horde_Exception(_("You are not allowed to list groups of shares."));
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
index 816ede5..1a6c747 100644 (file)
@@ -24,7 +24,7 @@ $fieldsList = array(
 );
 
 $app = Horde_Util::getFormData('app');
-$shares = $injector->getInstance('Horde_Share_Factory')->getScope($app);
+$shares = $injector->getInstance('Horde_Core_Factory_Share')->create($app);
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth_Factory')->getAuth();
 $help = $registry->hasMethod('shareHelp', $app)
index 47e859c..6314637 100644 (file)
@@ -64,7 +64,7 @@ class Ingo_Application extends Horde_Registry_Application
         // Create shares if necessary.
         $transport = Ingo::getTransport();
         if ($transport->supportShares()) {
-            $GLOBALS['ingo_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+            $GLOBALS['ingo_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
             $GLOBALS['all_rulesets'] = Ingo::listRulesets();
 
             /* If personal share doesn't exist then create it. */
index d26a0bb..da5e464 100644 (file)
@@ -68,7 +68,7 @@ class Kronolith_Application extends Horde_Registry_Application
         $GLOBALS['registry']->setTimeZone();
 
         /* Create a share instance. */
-        $GLOBALS['kronolith_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['kronolith_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         Kronolith::initialize();
     }
index 29c6150..68f96a2 100644 (file)
@@ -21,7 +21,7 @@ if (!empty($conf['share']['no_sharing'])) {
     throw new Horde_Exception('Permission denied.');
 }
 
-$shares = $injector->getInstance('Horde_Share_Factory')->getScope();
+$shares = $injector->getInstance('Horde_Core_Factory_Share')->create();
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth_Factory')->getAuth();
 
index 0247ded..30a5f3b 100644 (file)
@@ -56,7 +56,7 @@ class Horde_Block_Mnemo_summary extends Horde_Block
         $fgcolors = $cManager->fgColors();
 
         if (!empty($this->_params['show_notepad'])) {
-            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+            $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
         }
 
         $html = '';
index 1538f2b..f37dde8 100644 (file)
@@ -399,7 +399,7 @@ class Mnemo {
      */
     public static function initialize()
     {
-        $GLOBALS['mnemo_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['mnemo_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         // Update the preference for which notepads to display. If the
         // user doesn't have any selected notepads for view then fall
index 45429e0..056c0ef 100644 (file)
@@ -51,7 +51,7 @@ class Nag_Application extends Horde_Registry_Application
         $GLOBALS['registry']->setTimeZone();
 
         // Create a share instance.
-        $GLOBALS['nag_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['nag_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         Nag::initialize();
     }
index 0aee2c7..b0ad2ad 100644 (file)
@@ -202,7 +202,7 @@ class Horde_Block_nag_summary extends Horde_Block
 
             if (!empty($this->_params['show_tasklist'])) {
                 $owner = $task->tasklist;
-                $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+                $shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
                 $share = $shares->getShare($owner);
                 $owner = $share->get('name');
                 $html .= '<td width="1%" class="nowrap">'
index 36d17bf..8f45240 100644 (file)
@@ -81,7 +81,7 @@ class Turba_Application extends Horde_Registry_Application
             if (!empty($cfg['use_shares'])) {
                 // Create a share instance.
                 $_SESSION['turba']['has_share'] = true;
-                $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+                $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
                 $cfgSources = Turba::getConfigFromShares($cfgSources);
                 break;
             }
index 4500e08..d53607d 100644 (file)
@@ -156,7 +156,7 @@ class Turba_KolabTestBase extends Horde_Kolab_Test_Storage
         $this->prepareNewFolder($world['storage'], 'test2', 'contact');
 
         $_SESSION['turba']['has_share'] =  true;
-        $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 
         $GLOBALS['cfgSources'] = Turba::getConfigFromShares($cfgSources);
     }
index 1fa4a40..2a9c6c2 100755 (executable)
@@ -38,7 +38,7 @@ $handle = $db->query('SELECT owner, nickname, firstname, lastname, email, label
 if ($handle instanceof PEAR_Error) {
     $cli->fatal($handle->toString());
 }
-$turba_shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+$turba_shares = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
 $user = null;
 $count = 0;
 while ($row = $handle->fetchRow(DB_FETCHMODE_ASSOC)) {
index 3180839..ee63223 100644 (file)
@@ -876,7 +876,7 @@ class Whups_QueryManager {
      */
     function Whups_QueryManager()
     {
-        $this->_shareManager = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
+        $this->_shareManager = $GLOBALS['injector']->getInstance('Horde_Core_Factory_Share')->create();
     }
 
     /**