From: Jan Schneider <jan@horde.org>
Date: Wed, 5 Jan 2011 18:08:34 +0000 (+0100)
Subject: No need to return anything.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=96fa90528912b213ca97723ab4c61e2423c7e5de;p=horde.git

No need to return anything.
---

diff --git a/framework/Share/lib/Horde/Share/Object.php b/framework/Share/lib/Horde/Share/Object.php
index c9c11537c..9c100cff8 100644
--- a/framework/Share/lib/Horde/Share/Object.php
+++ b/framework/Share/lib/Horde/Share/Object.php
@@ -246,8 +246,6 @@ abstract class Horde_Share_Object implements Serializable
      * @param Horde_Perms_Permission $perm  Permission object.
      * @param boolean $update               Should the share be saved
      *                                      after this operation?
-     *
-     * @return boolean
      */
     abstract public function setPermission($perm, $update = true);
 
diff --git a/framework/Share/lib/Horde/Share/Object/Sql.php b/framework/Share/lib/Horde/Share/Object/Sql.php
index b76b9b6e9..c5c75bf7c 100644
--- a/framework/Share/lib/Horde/Share/Object/Sql.php
+++ b/framework/Share/lib/Horde/Share/Object/Sql.php
@@ -236,16 +236,13 @@ class Horde_Share_Object_Sql extends Horde_Share_Object implements Serializable
      * @param Horde_Perms_Permission $perm  Permission object.
      * @param boolean $update               Should the share be saved
      *                                      after this operation?
-     *
-     * @return boolean
      */
     public function setPermission($perm, $update = true)
     {
         $this->data['perm'] = $perm->getData();
         if ($update) {
-            return $this->save();
+            $this->save();
         }
-        return true;
     }
 
     /**