diff options
-rw-r--r-- | calendar/ChangeLog | 85 | ||||
-rw-r--r-- | calendar/cal-util/cal-component.c | 26 | ||||
-rw-r--r-- | calendar/cal-util/cal-component.h | 4 | ||||
-rw-r--r-- | calendar/cal-util/timeutil.h | 2 | ||||
-rw-r--r-- | calendar/gui/calendar-model.c | 10 | ||||
-rw-r--r-- | calendar/gui/dialogs/task-editor.c | 48 | ||||
-rw-r--r-- | calendar/gui/e-itip-control.c | 1 | ||||
-rw-r--r-- | calendar/gui/e-meeting-edit.c | 1 | ||||
-rw-r--r-- | calendar/gui/itip-utils.h | 1 |
9 files changed, 81 insertions, 97 deletions
diff --git a/calendar/ChangeLog b/calendar/ChangeLog index ab05db166c..7d66a7730d 100644 --- a/calendar/ChangeLog +++ b/calendar/ChangeLog @@ -1,3 +1,32 @@ +2000-12-11 Federico Mena Quintero <federico@helixcode.com> + + This is to make things work with libical 0.21helix1 and later. + Warnings remain because at last libical was constified; will take + care of those tomorrow. + + * cal-util/timeutil.h: #include <ical.h> instead of <icaltypes.h> + + * gui/e-itip-control.c: Likewise. + + * gui/e-meeting-edit.c: Likewise. + + * gui/itip-utils.h: Likewise. + + * cal-util/cal-component.c (alarm_uid_from_prop): constify. + (cal_component_get_status): Updated for new libical API. + (cal_component_set_status): Likewise. + + * gui/calendar-model.c (ensure_task_complete): Removed unused + status code. + (ensure_task_not_complete): Update for new status API. + + * gui/dialogs/task-editor.c (status_string_to_value): Removed + function. + (status_value_to_string): Removed function. + (status_string_map): Removed variable. + (fill_widgets): Update for new status API. + (dialog_to_comp_object): Likewise. + 2000-12-11 Damon Chaplin <damon@helixcode.com> * cal-util/cal-recur.c (generate_instances_for_chunk): updated the @@ -17,7 +46,7 @@ Fixed the 'All day event' toggle button. Made the 'Alarm' page sensitive as appropriate when filling widgets. (Though note that the alarm widgets are not being set yet.) - + * gui/dialogs/task-editor.c: added changed flag as above. * gui/event-editor-dialog.glade: used good names for all the @@ -59,8 +88,8 @@ 2000-12-08 JP Rosevear <jpr@helixcode.com> - * gui/e-calendar-table.c (e_calendar_table_init): Attach signal - handlers to the e_scrolled_table's etable rather than to the + * gui/e-calendar-table.c (e_calendar_table_init): Attach signal + handlers to the e_scrolled_table's etable rather than to the e_scrolled_table directly (e_calendar_table_on_double_click): This signal provides more params now @@ -76,7 +105,7 @@ until pcs can be altered (longer term) * conduits/todo/todo-conduit.c (post_sync): ditto - + 2000-12-07 Chris Toshok <toshok@helixcode.com> * cal-client/Makefile.am (client_test_LDADD): add @@ -140,7 +169,7 @@ * pcs/cal-backend.h: update prototype - * pcs/cal-backend.c (cal_backend_compute_changes_foreach_key): Build + * pcs/cal-backend.c (cal_backend_compute_changes_foreach_key): Build the corba struct rather than the old calobjchange thing (cal_backend_compute_changes): ditto. build and return the actual corba sequence rather than the list of calobjchanges @@ -149,17 +178,17 @@ * cal-util/cal-util.h: Remove CalObjChange cruft * cal-util/cal-util.c (cal_obj_change_list_free): Kill - + 2000-12-06 JP Rosevear <jpr@helixcode.com> - * cal-util/cal-util.c: + * cal-util/cal-util.c: * conduits/calendar/calendar-conduit.c (map_name): Update so as not to conflict with calendar (next_changed_item): update to use CalClientChange instead of CalObjChange (compute_status): ditto (pre_sync): ditto - (for_each_modified): since we now have the cal component we can call + (for_each_modified): since we now have the cal component we can call local_record_from_comp directly * conduits/todo/todo-conduit.c: same as above @@ -203,16 +232,16 @@ CalClientChange. * cal-client/Makefile.am: Build new files - + 2000-12-06 JP Rosevear <jpr@helixcode.com> * conduits/todo/Makefile.am: Fix my build stupidty READ THE MACRO - + * conduits/calendar/Makefile.am: ditto 2000-12-04 JP Rosevear <jpr@helixcode.com> - * gui/e-day-view-time-item.c (e_day_view_time_item_get_column_width): + * gui/e-day-view-time-item.c (e_day_view_time_item_get_column_width): Initialize max_large_digit_width to 0 to prevent crazy sizing issues. 2000-12-04 Dan Winship <danw@helixcode.com> @@ -283,7 +312,7 @@ * conduits/calendar/calendar-conduit.c: Debug message cleanups (get_ical_day): Fix off-by-one error which affected weekly occurences. (comp_from_remote_record): Monthly by day and by date were reversed - (nth_weekday): function taken from event-editor.c that encodes BYDAY + (nth_weekday): function taken from event-editor.c that encodes BYDAY values - this needs to be in libical really. (comp_from_remote_record): Don't set the description if the pilot note is null. Rejig so that we don't have to free objects. @@ -329,7 +358,7 @@ 2000-11-28 Damon Chaplin <damon@helixcode.com> - * gui/e-day-view*.[hc]: + * gui/e-day-view*.[hc]: * gui/e-week-view*.[hc]: finished 12-hour support and tried to tidy up & comment the drawing code in places. Also fixed a couple of bugs I spotted. All the options on the 'Calendar' page should now work. @@ -348,7 +377,7 @@ * conduits/todo/todo-conduit.c (free_prepare): Ditto - * conduits/calendar/calendar-conduit.c (free_prepare): Adjust + * conduits/calendar/calendar-conduit.c (free_prepare): Adjust free_prepare to the correct signal parameters. Don't actually do anything - there is a semantic discrepancy that needs to be resolved. @@ -361,7 +390,7 @@ * gui/gnome-cal.c: added more support for config settings. - * gui/e-week-view.[hc]: + * gui/e-week-view.[hc]: * gui/e-day-view.[hc]: added support for setting - show event end times, week start day and 12-hour format (unfinished). @@ -382,7 +411,7 @@ * gui/dialogs/task-editor.c (task_editor_create_date_edit): use config settings. - * gui/dialogs/cal-prefs-dialog.c (cal_prefs_dialog_update_config): + * gui/dialogs/cal-prefs-dialog.c (cal_prefs_dialog_update_config): updated EDateEdit calls. 2000-11-24 Federico Mena Quintero <federico@helixcode.com> @@ -395,7 +424,7 @@ * cal-util/cal-component.c (free_icalcomponent): DOH, fixed reversed test for the presence of the icalcomp's parent. This was causing memory leaks in the Wombat and elsewhere. - + * pcs/cal-backend.c (cal_backend_set_node_timet): Plug leak. 2000-11-24 Federico Mena Quintero <federico@helixcode.com> @@ -413,8 +442,8 @@ * pcs/cal-backend.c (cal_backend_log_entry): Plug leak. (cal_backend_log_sync): Free the entry->uid. - * util/icalendar-save.[ch]: - * util/icalendar-test.c: + * util/icalendar-save.[ch]: + * util/icalendar-test.c: * util/icalendar.[ch]: Removed obsolete files. 2000-11-21 Federico Mena Quintero <federico@helixcode.com> @@ -440,8 +469,8 @@ (time_format_changed): Removed. (colors_changed): Removed. - * gui/calendar-commands.h: - * gui/prop.c (prop_apply): + * gui/calendar-commands.h: + * gui/prop.c (prop_apply): * gui/calendar-commands.c (init_calendar): Removed the old to-do list crap. @@ -468,7 +497,7 @@ thing to do for most meetings. 2000-11-13 Jesse Pavel <jpavel@helixcode.com> - + * gui/e-itip-control.c: made the REPLY code actually work. 2000-11-13 Jesse Pavel <jpavel@helixcode.com> @@ -616,11 +645,11 @@ component. 2000-11-08 Jesse Pavel <jpavel@helixcode.com> - + * gui/e-itip-control.c, gui/e-itip-control.glade: updated the GUI to allow the user to add PUBLISHed events to his calendar, and created unworking buttons for meeting requests. - + 2000-11-08 Federico Mena Quintero <federico@helixcode.com> These changes fix bugzilla bugs #874 and #875. @@ -687,7 +716,7 @@ * gui/e-itip-control.c, gui/e-itip-control.glade: changed to GUI to accomodate dynamically generated buttons, which will be tailored to the type of iTip message that is incoming. - + * gui/e-meeting-dialog.glade gui/e-meeting-edit.c: added a new button to publish events, in addition to requesting meetings. @@ -758,7 +787,7 @@ * gui/main.c: (main): added call to bindtextdomain and textdomain, so all calendar gui shows up localized. - + 2000-10-31 Federico Mena Quintero <federico@helixcode.com> * gui/event-editor.c (count_by_xxx): Hmmm. SHRT_MAX changed to @@ -778,7 +807,7 @@ add changed_hash. * conduits/calendar/calendar-conduit.h: ditto - + * conduits/todo/todo-conduit.c (next_changed_item): Utility function to get the next "really" changed item (changed status can be cleared now) (compute_status): Compute status based on changed_hash @@ -858,7 +887,7 @@ 2000-10-27 <jpr@helixcode.com> - * conduits/calendar/calendar-conduit.c (check_for_slow_setting): + * conduits/calendar/calendar-conduit.c (check_for_slow_setting): Check boundary case of fast sync * conduits/todo/todo-conduit.c (check_for_slow_setting): ditto diff --git a/calendar/cal-util/cal-component.c b/calendar/cal-util/cal-component.c index 4aafdbf13a..cc920c19c6 100644 --- a/calendar/cal-util/cal-component.c +++ b/calendar/cal-util/cal-component.c @@ -607,7 +607,7 @@ scan_property (CalComponent *comp, icalproperty *prop) static const char * alarm_uid_from_prop (icalproperty *prop) { - char *xstr; + const char *xstr; g_assert (icalproperty_isa (prop) == ICAL_X_PROPERTY); @@ -1129,12 +1129,13 @@ cal_component_set_uid (CalComponent *comp, const char *uid) /** * cal_component_get_status: * @comp: A calendar component object. - * @status: Return value for the status string. + * @status: Return value for the status value. It is set to #ICAL_STATUS_NONE + * if the component has no status property. * * Queries the status property of a calendar component object. **/ void -cal_component_get_status (CalComponent *comp, const char **status) +cal_component_get_status (CalComponent *comp, icalproperty_status *status) { CalComponentPrivate *priv; @@ -1146,7 +1147,7 @@ cal_component_get_status (CalComponent *comp, const char **status) g_return_if_fail (priv->icalcomp != NULL); if (!priv->status) { - *status = NULL; + *status = ICAL_STATUS_NONE; return; } @@ -1156,25 +1157,25 @@ cal_component_get_status (CalComponent *comp, const char **status) /** * cal_component_set_status: * @comp: A calendar component object. - * @status: a status string, e.g. "IN-PROCESS", "NEEDS-ACTION". See the RFC. + * @status: Status value. You should use #ICAL_STATUS_NONE if you want to unset + * this property. * - * Sets the status string property of a calendar component object. + * Sets the status property of a calendar component object. **/ void -cal_component_set_status (CalComponent *comp, const char *status) +cal_component_set_status (CalComponent *comp, icalproperty_status status) { CalComponentPrivate *priv; g_return_if_fail (comp != NULL); g_return_if_fail (IS_CAL_COMPONENT (comp)); - g_return_if_fail (status != NULL); priv = comp->priv; g_return_if_fail (priv->icalcomp != NULL); priv->need_sequence_inc = TRUE; - if (status == NULL) { + if (status == ICAL_STATUS_NONE) { if (priv->status) { icalcomponent_remove_property (priv->icalcomp, priv->status); icalproperty_free (priv->status); @@ -1185,11 +1186,10 @@ cal_component_set_status (CalComponent *comp, const char *status) } if (priv->status) { - icalproperty_set_status (priv->status, (char *) status); + icalproperty_set_status (priv->status, status); } else { - priv->status = icalproperty_new_status ((char *) status); - icalcomponent_add_property (priv->icalcomp, - priv->status); + priv->status = icalproperty_new_status (status); + icalcomponent_add_property (priv->icalcomp, priv->status); } } diff --git a/calendar/cal-util/cal-component.h b/calendar/cal-util/cal-component.h index eb4b129f17..f59bb33a7b 100644 --- a/calendar/cal-util/cal-component.h +++ b/calendar/cal-util/cal-component.h @@ -165,8 +165,8 @@ void cal_component_commit_sequence (CalComponent *comp); void cal_component_get_uid (CalComponent *comp, const char **uid); void cal_component_set_uid (CalComponent *comp, const char *uid); -void cal_component_get_status (CalComponent *comp, const char **status); -void cal_component_set_status (CalComponent *comp, const char *status); +void cal_component_get_status (CalComponent *comp, icalproperty_status *status); +void cal_component_set_status (CalComponent *comp, icalproperty_status status); void cal_component_get_categories_list (CalComponent *comp, GSList **categ_list); void cal_component_set_categories_list (CalComponent *comp, GSList *categ_list); diff --git a/calendar/cal-util/timeutil.h b/calendar/cal-util/timeutil.h index b8fa6400d2..11bfcceb05 100644 --- a/calendar/cal-util/timeutil.h +++ b/calendar/cal-util/timeutil.h @@ -11,7 +11,7 @@ #include <time.h> -#include <icaltypes.h> +#include <ical.h> time_t time_from_isodate (char *str); diff --git a/calendar/gui/calendar-model.c b/calendar/gui/calendar-model.c index 8de4ab2e09..20c2bff234 100644 --- a/calendar/gui/calendar-model.c +++ b/calendar/gui/calendar-model.c @@ -1915,7 +1915,6 @@ ensure_task_complete (CalComponent *comp, { struct icaltimetype *old_completed = NULL; struct icaltimetype new_completed; - const char *old_status; int *old_percent, new_percent; gboolean set_completed = TRUE; @@ -1945,9 +1944,6 @@ ensure_task_complete (CalComponent *comp, } if (old_percent) cal_component_free_percent (old_percent); - - /* Status. */ - cal_component_get_status (comp, &old_status); } @@ -1958,7 +1954,7 @@ ensure_task_complete (CalComponent *comp, static void ensure_task_not_complete (CalComponent *comp) { - const char *old_status; + icalproperty_status old_status; int *old_percent; /* Date Completed. */ @@ -1973,8 +1969,8 @@ ensure_task_not_complete (CalComponent *comp) /* Status. */ cal_component_get_status (comp, &old_status); - if (old_status && !strcmp (old_status, "COMPLETED")) - cal_component_set_status (comp, "NEEDS-ACTION"); + if (old_status == ICAL_STATUS_COMPLETED) + cal_component_set_status (comp, ICAL_STATUS_NEEDSACTION); } diff --git a/calendar/gui/dialogs/task-editor.c b/calendar/gui/dialogs/task-editor.c index 211c31321a..6114ffa529 100644 --- a/calendar/gui/dialogs/task-editor.c +++ b/calendar/gui/dialogs/task-editor.c @@ -161,9 +161,6 @@ static void raise_and_focus (GtkWidget *widget); static TaskEditorPriority priority_value_to_index (int priority_value); static int priority_index_to_value (TaskEditorPriority priority); -static int status_string_to_value (const char *status_string); -static const char* status_value_to_string (int status); - static void completed_changed (EDateEdit *dedit, TaskEditor *tedit); static void status_changed (GtkMenu *menu, @@ -744,7 +741,7 @@ fill_widgets (TaskEditor *tedit) int *priority_value, *percent; icalproperty_status status; TaskEditorPriority priority; - const char *url, *status_string; + const char *url; priv = tedit->priv; @@ -810,10 +807,8 @@ fill_widgets (TaskEditor *tedit) } /* Status. */ - cal_component_get_status (priv->comp, &status_string); - if (status_string) { - status = status_string_to_value (status_string); - } else { + cal_component_get_status (priv->comp, &status); + if (status == ICAL_STATUS_NONE) { /* Try to user the percent value. */ if (percent) { if (*percent == 0) @@ -822,9 +817,8 @@ fill_widgets (TaskEditor *tedit) status = ICAL_STATUS_COMPLETED; else status = ICAL_STATUS_INPROCESS; - } else { + } else status = ICAL_STATUS_NEEDSACTION; - } } e_dialog_option_menu_set (priv->status, status, status_map); @@ -892,7 +886,6 @@ dialog_to_comp_object (TaskEditor *tedit) int priority_value, percent; CalComponentClassification classification; char *url; - const char *status_string; char *str; priv = tedit->priv; @@ -975,8 +968,7 @@ dialog_to_comp_object (TaskEditor *tedit) /* Status. */ status = e_dialog_option_menu_get (priv->status, status_map); - status_string = status_value_to_string (status); - cal_component_set_status (comp, status_string); + cal_component_set_status (comp, status); /* Priority. */ priority = e_dialog_option_menu_get (priv->priority, priority_map); @@ -1117,36 +1109,6 @@ priority_index_to_value (TaskEditorPriority priority) } -static int -status_string_to_value (const char *status_string) -{ - int i; - - for (i = 0; status_map[i] != -1; i++) { - if (!strcmp (status_string_map[i], status_string)) - return status_map[i]; - } - - g_warning ("Invalid todo status string"); - return ICAL_STATUS_NEEDSACTION; -} - - -static const char* -status_value_to_string (int status) -{ - int i; - - for (i = 0; status_map[i] != -1; i++) { - if (status_map[i] == status) - return status_string_map[i]; - } - - g_warning ("Invalid todo status value"); - return NULL; -} - - static void completed_changed (EDateEdit *dedit, TaskEditor *tedit) diff --git a/calendar/gui/e-itip-control.c b/calendar/gui/e-itip-control.c index a19763344c..ced1431292 100644 --- a/calendar/gui/e-itip-control.c +++ b/calendar/gui/e-itip-control.c @@ -12,7 +12,6 @@ #include <gnome.h> #include <bonobo.h> #include <glade/glade.h> -#include <icaltypes.h> #include <ical.h> #include <time.h> #include <Evolution-Composer.h> diff --git a/calendar/gui/e-meeting-edit.c b/calendar/gui/e-meeting-edit.c index efe41bbf09..abff825a29 100644 --- a/calendar/gui/e-meeting-edit.c +++ b/calendar/gui/e-meeting-edit.c @@ -22,7 +22,6 @@ #include <config.h> #include <gnome.h> #include <glade/glade.h> -#include <icaltypes.h> #include <ical.h> #include <widgets/meeting-time-sel/e-meeting-time-sel.h> #include <Evolution-Composer.h> diff --git a/calendar/gui/itip-utils.h b/calendar/gui/itip-utils.h index 0d2b073e4f..146cb68154 100644 --- a/calendar/gui/itip-utils.h +++ b/calendar/gui/itip-utils.h @@ -3,7 +3,6 @@ #define ITIP_UTILS_HEADER #include <config.h> -#include <icaltypes.h> #include <ical.h> #include <string.h> #include <gnome.h> |