SCHEDULE-STATUS must only contain a status code.

Fixes #177.
sabre-io · Jan 6, 2015 · 317eac6 · 317eac6
1 parent d563f55
commit 317eac6
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/ChangeLog.md b/ChangeLog.md
@@ -6,6 +6,8 @@ ChangeLog
 
 * #168: Expanding calendars now removes objects with recurrence rules that
   don't have a valid recurrence instance.
+* #177: SCHEDULE-STATUS should not contain a reason phrase, only a status
+  code.
 
 
 3.3.4 (2014-11-19)

diff --git a/lib/ITip/Broker.php b/lib/ITip/Broker.php
@@ -341,7 +341,7 @@ protected function processMessageReply(Message $itipMessage, VCalendar $existing
             return null;
         }
         $instances = array();
-        $requestStatus = '2.0;Success';
+        $requestStatus = '2.0';
 
         // Finding all the instances the attendee replied to.
         foreach($itipMessage->message->VEVENT as $vevent) {
@@ -350,6 +350,7 @@ protected function processMessageReply(Message $itipMessage, VCalendar $existing
             $instances[$recurId] = $attendee['PARTSTAT']->getValue();
             if (isset($vevent->{'REQUEST-STATUS'})) {
                 $requestStatus = $vevent->{'REQUEST-STATUS'}->getValue();
+                list($requestStatus) = explode(';', $requestStatus);
             }
         }
 

diff --git a/tests/VObject/ITip/BrokerProcessReplyTest.php b/tests/VObject/ITip/BrokerProcessReplyTest.php
@@ -59,8 +59,7 @@ function testReplyAccept() {
 BEGIN:VEVENT
 SEQUENCE:2
 UID:foobar
-ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS="2.0;Success":mailto:foo@example
- .org
+ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS=2.0:mailto:foo@example.org
 ORGANIZER:mailto:bar@example.org
 END:VEVENT
 END:VCALENDAR
@@ -105,8 +104,7 @@ function testReplyRequestStatus() {
 BEGIN:VEVENT
 UID:foobar
 SEQUENCE:2
-ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS="2.3;foo-bar!":mailto:foo@exampl
- e.org
+ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS=2.3:mailto:foo@example.org
 ORGANIZER:mailto:bar@example.org
 END:VEVENT
 END:VCALENDAR
@@ -414,8 +412,7 @@ function testReplyAcceptUpdateRSVP() {
 BEGIN:VEVENT
 SEQUENCE:2
 UID:foobar
-ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS="2.0;Success":mailto:foo@example
- .org
+ATTENDEE;PARTSTAT=ACCEPTED;SCHEDULE-STATUS=2.0:mailto:foo@example.org
 ORGANIZER:mailto:bar@example.org
 END:VEVENT
 END:VCALENDAR