From bbb0b8da6b8c069c7e3db3ff894afff8ae4317df Mon Sep 17 00:00:00 2001
From: Kevin Cyster <kevinc@knowledgetree.com>
Date: Thu, 19 Feb 2009 12:38:13 +0200
Subject: [PATCH] Added webservice functionality for subscriptions

---
 ktapi/KTAPIDocument.inc.php | 33 +++++++++++++++++++++++++++------
 1 file changed, 27 insertions(+), 6 deletions(-)

diff --git a/ktapi/KTAPIDocument.inc.php b/ktapi/KTAPIDocument.inc.php
index cfa8ac9..92ca8c5 100644
--- a/ktapi/KTAPIDocument.inc.php
+++ b/ktapi/KTAPIDocument.inc.php
@@ -2285,7 +2285,7 @@ class KTAPI_Document extends KTAPI_FolderItem
 	 *
 	 * @author KnowledgeTree Team
 	 * @access public
-	 * @return bool TRUE if subscribed | FALSE if not
+	 * @return bool $result TRUE if subscribed | FALSE if not
 	 */
 	public function isSubscribed()
 	{
@@ -2293,7 +2293,8 @@ class KTAPI_Document extends KTAPI_FolderItem
         $user = $this->ktapi->get_user();
         $document = $this->document;
 
-        return Subscription::exists($user->getId(), $document->getId(), $subscriptionType);
+        $result = Subscription::exists($user->getId(), $document->getId(), $subscriptionType);
+        return $result;
 	}
 
 	/**
@@ -2301,12 +2302,13 @@ class KTAPI_Document extends KTAPI_FolderItem
 	 *
 	 * @author KnowledgeTree Team
 	 * @access public
+	 * @return boolean|object $result SUCCESS Boolean result of operation | FAILURE - a pear error object
 	 */
 	public function unsubscribe()
 	{
         if (!$this->isSubscribed())
         {
-            return;
+            return TRUE;
         }
 
         $subscriptionType = SubscriptionEvent::subTypes('Document');
@@ -2314,7 +2316,16 @@ class KTAPI_Document extends KTAPI_FolderItem
         $document = $this->document;
 
         $subscription = & Subscription::getByIDs($user->getId(), $document->getId(), $subscriptionType);
-        $subscription->delete();
+        $result = $subscription->delete();
+
+        if(PEAR::isError($result)){
+            return $result->getMessage();
+        }
+        if($result){
+            return $result;
+        }
+
+        return $_SESSION['errorMessage'];
 	}
 
 	/**
@@ -2322,12 +2333,13 @@ class KTAPI_Document extends KTAPI_FolderItem
 	 *
 	 * @author KnowledgeTree Team
 	 * @access public
+	 * @return boolean|object $result SUCCESS Boolean result of operation | FAILURE - a pear error object
 	 */
 	public function subscribe()
 	{
         if ($this->isSubscribed())
         {
-            return;
+            return TRUE;
         }
 
         $subscriptionType = SubscriptionEvent::subTypes('Document');
@@ -2335,7 +2347,16 @@ class KTAPI_Document extends KTAPI_FolderItem
         $document = $this->document;
 
         $subscription = new Subscription($user->getId(), $document->getId(), $subscriptionType);
-        $subscription->create();
+        $result = $subscription->create();
+
+        if(PEAR::isError($result)){
+            return $result->getMessage();
+        }
+        if($result){
+            return $result;
+        }
+
+        return $_SESSION['errorMessage'];
 	}
 
 	/**
--
libgit2 0.21.4