some php5 stuff for folks, typo in turba
authorMichael J. Rubinsky <mrubinsk@horde.org>
Thu, 20 May 2010 04:42:24 +0000 (00:42 -0400)
committerMichael J. Rubinsky <mrubinsk@horde.org>
Thu, 20 May 2010 04:43:44 +0000 (00:43 -0400)
mend

folks/lib/Friends/shared.php
turba/lib/Driver/Imsp.php

index 27590ad..181f962 100644 (file)
@@ -44,9 +44,9 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
     {
         $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
 
-        $share = $GLOBALS['folks_shares']->getShareById($group);
-        if ($share instanceof PEAR_Error) {
-            return $share;
+        try {
+            $share = $GLOBALS['folks_shares']->getShareById($group);
+        } catch (Horde_Share_Exception $e) {
         }
 
         return $share->get('owner');
@@ -58,11 +58,7 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
     protected function _getGroups()
     {
         $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
-
         $groups = $GLOBALS['folks_shares']->listShares($this->_user, Horde_Perms::READ);
-        if ($groups instanceof PEAR_Error) {
-            return $groups;
-        }
 
         $list = array();
         foreach ($groups as $group) {
@@ -84,12 +80,8 @@ class Folks_Friends_shared extends  Folks_Friends_sql {
         }
 
         $GLOBALS['folks_shares'] = $GLOBALS['injector']->getInstance('Horde_Share')->getScope();
-
         $share = $GLOBALS['folks_shares']->getShareById($group);
-        if ($share instanceof PEAR_Error) {
-            return $share;
-        }
-
+        
         // Only owners of a group can delete them
         if (!Horde_Auth::getAuth() ||
             (Horde_Auth::getAuth() != $share->get('owner') &&
index a37dda5..7d2651e 100644 (file)
@@ -634,7 +634,7 @@ class Turba_Driver_Imsp extends Turba_Driver
         } else {
             $params['params']['name'] = $this->params['username'] . '.' . $params['name'];
         }
-        $result = &Turba::createShare($share_id, $params);
+        $result = Turba::createShare($share_id, $params);
         if (is_a($result, 'PEAR_Error')) {
             return $result;
         }