From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Mon, 28 Sep 2009 20:20:56 +0000 (-0400)
Subject: Use exceptions from Horde_Lock
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=505e4010c04486099d61c86bc55aac98c9bc8719;p=horde.git

Use exceptions from Horde_Lock
---

diff --git a/framework/Rpc/lib/Horde/Rpc/Webdav.php b/framework/Rpc/lib/Horde/Rpc/Webdav.php
index ceb07e711..0ed2dd009 100644
--- a/framework/Rpc/lib/Horde/Rpc/Webdav.php
+++ b/framework/Rpc/lib/Horde/Rpc/Webdav.php
@@ -737,9 +737,10 @@ class Horde_Rpc_Webdav extends Horde_Rpc
             $timeout = 600;
         }
 
-        $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
-        if (is_a($locks, 'PEAR_Error')) {
-            Horde::logMessage($locks, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return 500;
         }
 
@@ -748,13 +749,14 @@ class Horde_Rpc_Webdav extends Horde_Rpc
             $locktype = Horde_Lock::TYPE_EXCLUSIVE;
         }
 
-        $lockid = $locks->setLock(Horde_Auth::getAuth(), 'webdav', $params['path'],
-                                  $timeout, $locktype);
-
-        if (is_a($lockid, 'PEAR_Error')) {
-            Horde::logMessage($lockid, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $lockid = $locks->setLock(Horde_Auth::getAuth(), 'webdav', $params['path'],
+                                      $timeout, $locktype);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return 500;
-        } elseif ($lockid === false) {
+        }
+        if ($lockid === false) {
             // Resource is already locked.
             return 423;
         }
@@ -782,17 +784,20 @@ class Horde_Rpc_Webdav extends Horde_Rpc
             return 500;
         }
 
-        $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
-        if (is_a($locks, 'PEAR_Error')) {
-            Horde::logMessage($locks, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return 500;
         }
 
-        $res = $locks->clearLock($params['token']);
-        if (is_a($res, 'PEAR_Error')) {
-            Horde::logMessage($res, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $res = $locks->clearLock($params['token']);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return 500;
-        } elseif ($res === false) {
+        }
+        if ($res === false) {
             Horde::logMessage('clearLock() returned false', __FILE__, __LINE__, PEAR_LOG_ERR);
             // Something else has failed:  424 (Method Failure)
             return 424;
@@ -811,15 +816,17 @@ class Horde_Rpc_Webdav extends Horde_Rpc
             return false;
         }
 
-        $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
-        if (is_a($locks, 'PEAR_Error')) {
-            Horde::logMessage($locks, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $locks = Horde_Lock::singleton($GLOBALS['conf']['lock']['driver']);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return false;
         }
 
-        $res =  $locks->getLocks('webdav', $resource);
-        if (is_a($res, 'PEAR_Error')) {
-            Horde::logMessage($res, __FILE__, __LINE__, PEAR_LOG_ERR);
+        try {
+            $res =  $locks->getLocks('webdav', $resource);
+        } catch (Horde_Lock_Exception $e) {
+            Horde::logMessage($e->getMessage(), __FILE__, __LINE__, PEAR_LOG_ERR);
             return false;
         }