[evolution-patches] Fix for free/busy issues in GW backend in e-d-s and bug# 55169

[Harish Krishnaswamy <kharish novell com>]

hi,

pl. find attached the fix for the f/b issues in the GW backend. The fix
involves modifying the code to  use icalproperty to populate free/busy
information instead of ECalComponentDateTime.

harish

Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/evolution-data-server/calendar/ChangeLog,v
retrieving revision 1.237
diff -u -p -r1.237 ChangeLog
--- ChangeLog	5 Apr 2004 15:45:39 -0000	1.237
+++ ChangeLog	6 Apr 2004 09:15:07 -0000
@@ -1,3 +1,11 @@
+2004-04-06  Harish K <kharish novell com>
+	
+	Fixes #55169
+
+	* backends/groupwise/e-cal-backend-groupwise-utils.c
+	(e_gw_connection_get_freebusy_info) : Use icalproperty
+	to add free/busy data to ECalComponent.
+	
 2004-04-05  Rodrigo Moya <rodrigo ximian com>
 
 	* backends/groupwise/e-cal-backend-groupwise.c
Index: backends/groupwise/e-cal-backend-groupwise-utils.c
===================================================================
RCS
file: /cvs/gnome/evolution-data-server/calendar/backends/groupwise/e-cal-backend-groupwise-utils.c,v
retrieving revision 1.12
diff -u -p -r1.12 e-cal-backend-groupwise-utils.c
--- backends/groupwise/e-cal-backend-groupwise-utils.c	29 Mar 2004
14:47:25 -0000	1.12
+++ backends/groupwise/e-cal-backend-groupwise-utils.c	6 Apr 2004
09:15:07 -0000
@@ -521,7 +521,11 @@ e_gw_connection_get_freebusy_info (EGwCo
 	     subparam = soup_soap_parameter_get_next_child_by_name (subparam,
"user")) {
 		SoupSoapParameter *param_blocks, *subparam_block, *tmp;
 		const char *uuid = NULL, *email = NULL, *name = NULL;
-
+		ECalComponent *comp;
+		ECalComponentAttendee attendee;
+		GSList *attendee_list = NULL;
+		icalcomponent *icalcomp = NULL;
+		
 		tmp = soup_soap_parameter_get_first_child_by_name (subparam,
"email");
 		if (tmp)
 			email = soup_soap_parameter_get_string_value (tmp);
@@ -532,6 +536,29 @@ e_gw_connection_get_freebusy_info (EGwCo
 		if (tmp)
 			name = soup_soap_parameter_get_string_value (tmp);
 
+		comp = e_cal_component_new ();
+		e_cal_component_set_new_vtype (comp, E_CAL_COMPONENT_FREEBUSY); 
+		e_cal_component_commit_sequence (comp);
+		icalcomp = e_cal_component_get_icalcomponent (comp);
+		
+		memset (&attendee, 0, sizeof (ECalComponentAttendee));
+		if (name)
+			attendee.cn = name;
+		if (email)
+			attendee.value = email;
+		
+		attendee.cutype = ICAL_CUTYPE_INDIVIDUAL;
+		attendee.role = ICAL_ROLE_REQPARTICIPANT;
+		attendee.status = ICAL_PARTSTAT_NEEDSACTION;
+
+		/* XXX the uuid is not currently used. hence it is
+		 * discarded */
+
+		attendee_list = g_slist_append (attendee_list, &attendee);	
+
+		e_cal_component_set_attendee_list (comp, attendee_list);
+
+		
 		param_blocks = soup_soap_parameter_get_first_child_by_name (subparam,
"blocks");
 		if (!param_blocks) {
 			g_object_unref (response);
@@ -545,58 +572,49 @@ e_gw_connection_get_freebusy_info (EGwCo
 
 			/* process each block and create ECal free/busy components.*/ 
 			SoupSoapParameter *tmp;
-			ECalComponent *comp;
-			ECalComponentAttendee attendee;
-			ECalComponentDateTime dt;
+			struct icalperiodtype ipt;
+			icalproperty *icalprop;
 			icaltimetype itt;
 			time_t t;
-			const char *start, *end;
-			GSList *attendee_list = NULL;
-
-			comp = e_cal_component_new ();
-			e_cal_component_set_new_vtype (comp, E_CAL_COMPONENT_FREEBUSY); 
-			/* FIXME  verify the mappings b/w response and ECalComponent */
-			memset (&attendee, 0, sizeof (ECalComponentAttendee));
-			if (name)
-				attendee.cn = name;
-			if (email)
-				attendee.value = email;
+			const char *start, *end, *accept_level;
 			
-			attendee.cutype = ICAL_CUTYPE_INDIVIDUAL;
-			attendee.role = ICAL_ROLE_REQPARTICIPANT;
-			attendee.status = ICAL_PARTSTAT_NEEDSACTION;
-
-			/* XXX the uuid is not currently used. hence it is
-			 * discarded */
-
-			attendee_list = g_slist_append (attendee_list, &attendee);	
-
-			e_cal_component_set_attendee_list (comp, attendee_list);
-
+			memset (&ipt, 0, sizeof (struct icalperiodtype));
 			tmp = soup_soap_parameter_get_first_child_by_name (subparam_block,
"startDate");
 			if (tmp) {
 				start = soup_soap_parameter_get_string_value (tmp);
 				t = e_gw_connection_get_date_from_string (start);
 				itt = icaltime_from_timet (t, 0);
-				dt.value = &itt;
-				dt.tzid = "UTC"; 
-				e_cal_component_set_dtstart (comp, &dt);
+				ipt.start = itt;
 			}        
 
-			tmp = soup_soap_parameter_get_first_child_by_name (subparam,
"endDate");
+			tmp = soup_soap_parameter_get_first_child_by_name (subparam_block,
"endDate");
 			if (tmp) {
 				end = soup_soap_parameter_get_string_value (tmp);
 				t = e_gw_connection_get_date_from_string (end);
 				itt = icaltime_from_timet (t, 0);
-				dt.value = &itt;
-				dt.tzid = "UTC"; 
-				e_cal_component_set_dtend (comp, &dt);
+				ipt.end = itt;
 			}
+			icalprop = icalproperty_new_freebusy (ipt);
+
+			tmp = soup_soap_parameter_get_first_child_by_name (subparam_block,
"acceptLevel");
+			if (tmp) {
+				accept_level = soup_soap_parameter_get_string_value (tmp);
+				if (!strcmp (accept_level, "Busy"))
+					icalproperty_set_parameter_from_string (icalprop, "FBTYPE",
"BUSY");
+				else if (!strcmp (accept_level, "Tentative"))
+					icalproperty_set_parameter_from_string (icalprop, "FBTYPE",
"BUSYTENTATIVE");
+				else if (!strcmp (accept_level, "OutOfOffice"))
+					icalproperty_set_parameter_from_string (icalprop, "FBTYPE",
"BUSYUNAVAILABLE");
+				else if (!strcmp (accept_level, "Free"))
+					icalproperty_set_parameter_from_string (icalprop, "FBTYPE",
"FREE");
+							}
+			icalcomponent_add_property(icalcomp, icalprop);
 
-			e_cal_component_commit_sequence (comp);
-			*freebusy = g_list_append (*freebusy, g_strdup
(e_cal_component_get_as_string (comp)));
-			g_object_unref (comp);
 		}
+
+		e_cal_component_commit_sequence (comp);
+		*freebusy = g_list_append (*freebusy, g_strdup
(e_cal_component_get_as_string (comp)));
+		g_object_unref (comp);
 	}
 
         g_object_unref (msg);