From: Ben Klang <ben@alkaloid.net>
Date: Wed, 3 Nov 2010 18:17:29 +0000 (-0400)
Subject: Begin working on converting PEAR_Error to Exceptions
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=892cffda6088a328b91098134e5283499956f575;p=horde.git

Begin working on converting PEAR_Error to Exceptions

More work needs to be done here.
---

diff --git a/whups/admin/index.php b/whups/admin/index.php
index 4b8476597..59b88a0eb 100644
--- a/whups/admin/index.php
+++ b/whups/admin/index.php
@@ -385,11 +385,11 @@ case 'deletequeueform':
     $form = new DeleteQueueForm($vars);
     if ($form->validate($vars)) {
         if ($vars->get('yesno') == 1) {
-            $result = $whups_driver->deleteQueue($vars->get('queue'));
-            if (!is_a($result, 'PEAR_Error')) {
+            try {
+                $result = $whups_driver->deleteQueue($vars->get('queue'));
                 $notification->push( _("The queue has been deleted."),
                                      'horde.success');
-            } else {
+            } catch (Horde_Exception $e) {
                 $notification->push(_("There was an error deleting the queue:")
                                     . ' ' . $result->getMessage(),
                                     'horde.error');
diff --git a/whups/lib/Driver.php b/whups/lib/Driver.php
index 5b3320722..038693ce8 100644
--- a/whups/lib/Driver.php
+++ b/whups/lib/Driver.php
@@ -286,12 +286,12 @@ class Whups_Driver {
     function deleteQueue($queueId)
     {
         $perms = $GLOBALS['injector']->getInstance('Horde_Perms');
-        $perm = $perms->getPermission("whups:queues:$queueId");
-        if (is_a($perm, 'PEAR_Error')) {
-            return;
-        }
+        try {
+            $perm = $perms->getPermission("whups:queues:$queueId");
+            return $perms->removePermission($perm, true);
+        } catch (Horde_Perms_Exception $e) {}
 
-        return $perms->removePermission($perm, true);
+        return true;
     }
 
     /**
@@ -305,12 +305,12 @@ class Whups_Driver {
     function deleteReply($reply)
     {
         $perms = $GLOBALS['injector']->getInstance('Horde_Perms');
-        $perm = $perms->getPermission("whups:replies:$reply");
-        if (is_a($perm, 'PEAR_Error')) {
-            return;
-        }
+        try {
+            $perm = $perms->getPermission("whups:replies:$reply");
+            return $perms->removePermission($perm, true);
+        } catch (Horde_Perms_Exception $e) {}
 
-        return $perms->removePermission($perm, true);
+        return true;
     }
 
     /**