[Pkg-owncloud-commits] [php-sabre-vobject] 34/106: Making return values consistent.

[David Prévot]

This is an automated email from the git hooks/post-receive script.

taffit pushed a commit to branch master
in repository php-sabre-vobject.

commit 38b10ab8c7597c7a64a53ee5c3b2c39d3ce15725
Author: Evert Pot <me at evertpot.com>
Date:   Wed Jul 23 18:01:20 2014 -0400

    Making return values consistent.
---
 lib/Sabre/VObject/ITip/Broker.php | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/lib/Sabre/VObject/ITip/Broker.php b/lib/Sabre/VObject/ITip/Broker.php
index 626af1c..23865c9 100644
--- a/lib/Sabre/VObject/ITip/Broker.php
+++ b/lib/Sabre/VObject/ITip/Broker.php
@@ -68,7 +68,7 @@ class Broker {
      * update the organizers event to update the ATTENDEE with its correct
      * PARTSTAT.
      *
-     * The $existingObject is updated in-place. If no existing object exists
+     * The $existingObject is updated in-place. If there is no existing object
      * (because it's a new invite for example) a new object will be created.
      *
      * If an existing object does not exist, and the method was CANCEL or
@@ -81,7 +81,7 @@ class Broker {
      *
      * @param Message $itipMessage
      * @param VCalendar $existingObject
-     * @return VCalendar|bool
+     * @return VCalendar|null
      */
     public function processMessage(Message $itipMessage, VCalendar $existingObject = null) {
 
@@ -103,7 +103,7 @@ class Broker {
 
             default :
                 // Unsupported iTip message
-                return false;
+                return null;
 
         }
 
@@ -189,7 +189,7 @@ class Broker {
      *
      * @param Message $itipMessage
      * @param VCalendar $existingObject
-     * @return VCalendar|bool
+     * @return VCalendar|null
      */
     protected function processMessageRequest(Message $itipMessage, VCalendar $existingObject = null) {
 
@@ -224,7 +224,7 @@ class Broker {
      *
      * @param Message $itipMessage
      * @param VCalendar $existingObject
-     * @return VCalendar|bool
+     * @return VCalendar|null
      */
     protected function processMessageCancel(Message $itipMessage, VCalendar $existingObject = null) {
 
@@ -249,14 +249,14 @@ class Broker {
      *
      * @param Message $itipMessage
      * @param VCalendar $existingObject
-     * @return VCalendar|bool
+     * @return VCalendar|null
      */
     protected function processMessageReply(Message $itipMessage, VCalendar $existingObject = null) {
 
         // A reply can only be processed based on an existing object.
         // If the object is not available, the reply is ignored.
         if (!$existingObject) {
-            return false;
+            return null;
         }
         $instances = array();
         foreach($itipMessage->message->VEVENT as $vevent) {
@@ -294,7 +294,7 @@ class Broker {
         foreach($instances as $recurId=>$partstat) {
             if(!$masterObject) {
                 // No master object, we can't add new instances.
-                return false;
+                return null;
             }
             $newObject = clone $masterObject;
             unset(

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-owncloud/php-sabre-vobject.git