From 71e27e50cb3c8711206f3ad0058f152ab9a1cb5a Mon Sep 17 00:00:00 2001
From: Florian Schlichting <fsfs@debian.org>
Date: Fri, 8 Jan 2016 14:27:37 +0100
Subject: [PATCH] fix Thunderbird mutilating external attendees

semicolon is a separator in iCal, and even though we send it enclosed in
double quotes TB will choke on it and produce a mess
---
 inc/caldav-PUT-functions.php | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/inc/caldav-PUT-functions.php b/inc/caldav-PUT-functions.php
index 935a9f2d..6ba05909 100644
--- a/inc/caldav-PUT-functions.php
+++ b/inc/caldav-PUT-functions.php
@@ -608,19 +608,19 @@ function do_scheduling_requests( vCalendar $resource, $create, $old_data = null
       $remote = new iSchedule ();
       $answer = $remote->sendRequest ( $email, 'VEVENT/REQUEST', $schedule_request->Render() );
       if ( $answer === false ) {
-        $response = "3.7;Invalid Calendar User" ;
+        $response = '3.7'; // Invalid Calendar User: iSchedule request failed or DKIM not configured
       }
       else {
         foreach ( $answer as $a ) // should only be one element in array
         {
           if ( $a === false ) {
-            $response = "3.7;Invalid Calendar User" ;
+            $response = '3.7'; // Invalid Calendar User: weird reply from remote server
           }
           elseif ( substr( $a, 0, 1 ) >= 1 ) {
-            $response = $a ;
+            $response = $a; // NOTE: this may need to be limited to the reponse code
           }
           else {
-            $response = "2.0;Success" ;
+            $response = '2.0'; // Success
           }
         }
       }