Ask for a share factory when we want a share factory.
authorMichael J. Rubinsky <mrubinsk@horde.org>
Sat, 2 Oct 2010 21:20:34 +0000 (17:20 -0400)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Sat, 2 Oct 2010 21:21:06 +0000 (17:21 -0400)
26 files changed:
ansel/lib/Injector/Factory/Storage.php
fima/lib/base.php
folks/lib/Driver.php
folks/lib/Friends/shared.php
folks/perms.php
framework/Core/lib/Horde/Core/Binder/Share.php [deleted file]
framework/Core/lib/Horde/Core/Binder/ShareFactory.php [new file with mode: 0644]
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
skoli/lib/base.php
turba/lib/Application.php
turba/lib/tests/KolabTestBase.php
turba/scripts/import_squirrelmail_sql_abook.php
whups/lib/Query.php

index ecc62c3..743b8ea 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')->getScope($scope, 'Sql_Hierarchical'));
+            $this->_instances[$scope] = new Ansel_Storage($this->_injector->getInstance('Horde_Share_Factory')->getScope($scope, 'Sql_Hierarchical'));
         }
 
        return $this->_instances[$scope];
index 47ad7c5..2acae93 100644 (file)
@@ -48,6 +48,6 @@ Horde::compressOutput();
 $registry->setTimeZone();
 
 // Create a share instance.
-$GLOBALS['fima_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
+$GLOBALS['fima_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
 Fima::initialize();
index 38647f9..00ae4b4 100644 (file)
@@ -463,7 +463,7 @@ class Folks_Driver {
 
         // Delete groups
         if ($GLOBALS['conf']['friends']) {
-            $shares = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
+            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
             $groups = $shares->listShares($GLOBALS['registry']->getAuth(), Horde_Perms::SHOW, true);
             foreach ($groups as $share) {
                 $result = $shares->removeShare($share);
index f7165e3..4fb8f3f 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')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         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')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
         $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')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
         $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')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         $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')->getScope();
+        $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         $share = $GLOBALS['folks_shares']->newShare(strval(new Horde_Support_Uuid()));
 
index d2141fe..4cbfc64 100644 (file)
@@ -11,7 +11,7 @@
 
 require_once dirname(__FILE__) . '/lib/base.php';
 
-$shares = $injector->getInstance('Horde_Share')->getScope();
+$shares = $injector->getInstance('Horde_Share_Factory')->getScope();
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth')->getAuth();
 
diff --git a/framework/Core/lib/Horde/Core/Binder/Share.php b/framework/Core/lib/Horde/Core/Binder/Share.php
deleted file mode 100644 (file)
index 1a99046..0000000
+++ /dev/null
@@ -1,18 +0,0 @@
-<?php
-/**
- * @category Horde
- * @package  Core
- */
-class Horde_Core_Binder_Share 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;
-    }
-
-}
diff --git a/framework/Core/lib/Horde/Core/Binder/ShareFactory.php b/framework/Core/lib/Horde/Core/Binder/ShareFactory.php
new file mode 100644 (file)
index 0000000..36f4cb9
--- /dev/null
@@ -0,0 +1,18 @@
+<?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 c71f2f3..536a593 100644 (file)
@@ -296,7 +296,7 @@ class Horde_Registry
             'Horde_Service_Facebook' => 'Horde_Core_Binder_Facebook',
             'Horde_Service_Twitter' => 'Horde_Core_Binder_Twitter',
             'Horde_SessionHandler' => 'Horde_Core_Binder_SessionHandler',
-            'Horde_Share' => 'Horde_Core_Binder_Share',
+            'Horde_Share_Factory' => 'Horde_Core_Binder_ShareFactory',
             'Horde_Template' => 'Horde_Core_Binder_Template',
             'Horde_Text_Filter' => 'Horde_Core_Binder_TextFilter',
             'Horde_Tree' => 'Horde_Core_Binder_Tree',
index b757f25..2c166f8 100644 (file)
@@ -140,7 +140,7 @@ Application Framework.</description>
        <file name="Prefs.php" role="php" />
        <file name="Secret.php" role="php" />
        <file name="SessionHandler.php" role="php" />
-       <file name="Share.php" role="php" />
+       <file name="ShareFactory.php" role="php" />
        <file name="Template.php" role="php" />
        <file name="TextFilter.php" role="php" />
        <file name="Token.php" role="php" />
@@ -466,7 +466,7 @@ Application Framework.</description>
    <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/Share.php" name="lib/Horde/Core/Binder/Share.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 1b40ba9..f362443 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')->getScope($scope);
+                    $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
                     $this->_contact_shares = $shares->listShares($GLOBALS['registry']->getAuth(), Horde_Perms::SHOW, $GLOBALS['registry']->getAuth());
                 }
                 // Contruct a list of owner ids to use
index 715b7df..84323f9 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')->getScope($arguments[0], 'kolab');
+            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($arguments[0], 'kolab');
 
             $world['list'] = $shares->listShares(Auth::getAuth());
             break;
index 795014c..7e4e7e7 100644 (file)
@@ -38,7 +38,7 @@ class Horde_Prefs_KolabImap extends Horde_Prefs_Base
             return;
         }
 
-        $shares = $GLOBALS['injector']->getInstance('Horde_Share')->getScope('h-prefs');
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope('h-prefs');
         $default = $shares->getDefaultShare();
         if ($default instanceof PEAR_Error) {
             if ($this->_opts['logger']) {
index 4d230bb..7a1b6f4 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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($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')->getScope($scope);
+        $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope($scope);
 
         if (is_a($share = &$shares->getShare($shareName), 'PEAR_Error')) {
             throw new Horde_Exception($share);
index 1209a7d..1247ebb 100644 (file)
@@ -24,7 +24,7 @@ $fieldsList = array(
 );
 
 $app = Horde_Util::getFormData('app');
-$shares = $injector->getInstance('Horde_Share')->getScope($app);
+$shares = $injector->getInstance('Horde_Share_Factory')->getScope($app);
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth')->getAuth();
 $help = $registry->hasMethod('shareHelp', $app)
index efccc27..e273576 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')->getScope();
+            $GLOBALS['ingo_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
             $GLOBALS['all_rulesets'] = Ingo::listRulesets();
 
             /* If personal share doesn't exist then create it. */
index 869d8b6..3425e6e 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')->getScope();
+        $GLOBALS['kronolith_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         Kronolith::initialize();
     }
index 66b157c..a2590c5 100644 (file)
@@ -21,7 +21,7 @@ if (!empty($conf['share']['no_sharing'])) {
     throw new Horde_Exception('Permission denied.');
 }
 
-$shares = $injector->getInstance('Horde_Share')->getScope();
+$shares = $injector->getInstance('Horde_Share_Factory')->getScope();
 $groups = $injector->getInstance('Horde_Group');
 $auth = $injector->getInstance('Horde_Auth')->getAuth();
 
index 4decbad..0247ded 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')->getScope();
+            $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
         }
 
         $html = '';
index cee292c..7675f1a 100644 (file)
@@ -399,7 +399,7 @@ class Mnemo {
      */
     public static function initialize()
     {
-        $GLOBALS['mnemo_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
+        $GLOBALS['mnemo_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         // Update the preference for which notepads to display. If the
         // user doesn't have any selected notepads for view then fall
index af4f254..b217256 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')->getScope();
+        $GLOBALS['nag_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         Nag::initialize();
     }
index 0470144..0aee2c7 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')->getScope();
+                $shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
                 $share = $shares->getShare($owner);
                 $owner = $share->get('name');
                 $html .= '<td width="1%" class="nowrap">'
index 4c535b1..0674dae 100644 (file)
@@ -32,6 +32,6 @@ $conf = &$GLOBALS['conf'];
 Horde::compressOutput();
 
 // Create a share instance.
-$GLOBALS['skoli_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
+$GLOBALS['skoli_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
 Skoli::initialize();
index 027c638..7d05cc5 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')->getScope();
+                $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
                 $cfgSources = Turba::getConfigFromShares($cfgSources);
                 break;
             }
index f829764..4500e08 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')->getScope();
+        $GLOBALS['turba_shares'] = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 
         $GLOBALS['cfgSources'] = Turba::getConfigFromShares($cfgSources);
     }
index 87f52c7..18a6710 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')->getScope();
+$turba_shares = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
 $user = null;
 $count = 0;
 while ($row = $handle->fetchRow(DB_FETCHMODE_ASSOC)) {
index 8140bb5..3180839 100644 (file)
@@ -876,7 +876,7 @@ class Whups_QueryManager {
      */
     function Whups_QueryManager()
     {
-        $this->_shareManager = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
+        $this->_shareManager = $GLOBALS['injector']->getInstance('Horde_Share_Factory')->getScope();
     }
 
     /**