aboutsummaryrefslogtreecommitdiffstats
path: root/calendar
diff options
context:
space:
mode:
Diffstat (limited to 'calendar')
-rw-r--r--calendar/ChangeLog47
-rw-r--r--calendar/cal-client/cal-client.c3
-rw-r--r--calendar/cal-util/cal-component.c13
-rw-r--r--calendar/cal-util/cal-recur.c5
-rw-r--r--calendar/conduits/calendar/calendar-conduit.c2
-rw-r--r--calendar/conduits/todo/todo-conduit.c1
-rw-r--r--calendar/gui/calendar-commands.c98
-rw-r--r--calendar/gui/calendar-config.c8
-rw-r--r--calendar/gui/calendar-model.c3
-rw-r--r--calendar/gui/comp-util.c2
-rw-r--r--calendar/gui/dialogs/recurrence-page.c71
-rw-r--r--calendar/gui/e-calendar-table.c59
-rw-r--r--calendar/gui/e-day-view.c42
-rw-r--r--calendar/gui/e-itip-control.c12
-rw-r--r--calendar/gui/e-week-view-layout.c2
-rw-r--r--calendar/gui/e-week-view.c39
-rw-r--r--calendar/gui/gnome-cal.c42
-rw-r--r--calendar/gui/gnome-cal.h3
-rw-r--r--calendar/gui/tag-calendar.c2
-rw-r--r--calendar/gui/tasks-control.c5
20 files changed, 337 insertions, 122 deletions
diff --git a/calendar/ChangeLog b/calendar/ChangeLog
index 40729af584..8a50c4e6c4 100644
--- a/calendar/ChangeLog
+++ b/calendar/ChangeLog
@@ -1,5 +1,51 @@
2001-07-30 Damon Chaplin <damon@ximian.com>
+ * gui/gnome-cal.c:
+ * gui/calendar-commands.c (clear_folder_bar_label): started some code
+ to show the currently displayed dates in the folder title bar.
+ Unfinished.
+
+ * gui/e-itip-control.c (set_date_label):
+ * conduits/todo/todo-conduit.c (local_record_from_comp):
+ * conduits/calendar/calendar-conduit.c (local_record_from_comp): free
+ the CalComponentDateTimes. (Note the iTIP control needs updating for
+ timezone support.)
+
+ * cal-util/cal-component.c: Changed CalComponentDateTime so that the
+ TZID is malloc'ed and freed rather than being a pointer to a static
+ string. This was causing problems as sometimes we were freeing the
+ string that was being pointed to, so we got corrupted TZIDs.
+
+ * gui/comp-util.c (cal_comp_util_add_exdate): set TZID to NULL.
+ DATE values do not have timezones.
+
+ * gui/e-week-view.c:
+ * gui/e-day-view.c: Moved 'Paste' after the New Appointment commands,
+ since I think they are more commonly-used. Also added underlined
+ accelerator keys.
+
+ * gui/e-calendar-table.c: changed 'Edit this task' to 'Open' in the
+ popup menu to be consistent with other folders, and separated from the
+ clipboard commands. Also changed to use EPopupMenu so the accelerators
+ work, and the masks may be useful at some point.
+
+ * gui/dialogs/recurrence-page.c: use DATE values for UNTIL, since
+ that makes it simpler. Fixes bug #5034.
+
+ * gui/calendar-config.c (calendar_config_set_timezone): strdup the
+ location string. Fixes bug #4990.
+
+ * gui/tag-calendar.c (tag_calendar_cb): take 1 off iend as the times
+ don't include the end time.
+
+ * gui/e-week-view-layout.c (e_week_view_layout_event): fixed
+ days_shown. Fixes bug #5709.
+
+ * cal-client/cal-client.c (cal_client_get_timezone): took out some
+ debugging messages.
+
+2001-07-30 Damon Chaplin <damon@ximian.com>
+
* gui/dialogs/cal-prefs-dialog.glade: added Help button. Though of
course it doesn't do anything yet.
@@ -183,7 +229,6 @@
value from cal_component_get_as_string and don't call
GNOME_Evolution_Calendar_Cal_updateObjects if NULL
->>>>>>> 1.1002
2001-07-25 Damon Chaplin <damon@ximian.com>
* gui/dialogs/comp-editor.c (pixmaps): used the new print preview icon.
diff --git a/calendar/cal-client/cal-client.c b/calendar/cal-client/cal-client.c
index ff93470405..007466ddad 100644
--- a/calendar/cal-client/cal-client.c
+++ b/calendar/cal-client/cal-client.c
@@ -936,7 +936,6 @@ cal_client_get_timezone (CalClient *client,
goto out;
}
- g_print (" parsing zone:\n%s\n", comp_str);
icalcomp = icalparser_parse_string (comp_str);
CORBA_free (comp_str);
@@ -945,7 +944,6 @@ cal_client_get_timezone (CalClient *client,
goto out;
}
- g_print (" creating icaltimezone\n");
tmp_zone = icaltimezone_new ();
if (!tmp_zone) {
/* FIXME: Needs better error code - out of memory. Or just
@@ -963,7 +961,6 @@ cal_client_get_timezone (CalClient *client,
g_hash_table_insert (priv->timezones, icaltimezone_get_tzid (tmp_zone),
tmp_zone);
- g_print (" returning icaltimezone\n");
*zone = tmp_zone;
retval = CAL_CLIENT_GET_SUCCESS;
diff --git a/calendar/cal-util/cal-component.c b/calendar/cal-util/cal-component.c
index 596618733d..b806f8e3e7 100644
--- a/calendar/cal-util/cal-component.c
+++ b/calendar/cal-util/cal-component.c
@@ -1825,9 +1825,9 @@ get_datetime (struct datetime *datetime,
/* If the icaltimetype has is_utc set, we set "UTC" as the TZID.
This makes the timezone code simpler. */
if (datetime->tzid_param)
- dt->tzid = icalparameter_get_tzid (datetime->tzid_param);
+ dt->tzid = g_strdup (icalparameter_get_tzid (datetime->tzid_param));
else if (dt->value && dt->value->is_utc)
- dt->tzid = "UTC";
+ dt->tzid = g_strdup ("UTC");
else
dt->tzid = NULL;
}
@@ -2254,7 +2254,7 @@ cal_component_get_exdate_list (CalComponent *comp, GSList **exdate_list)
*cdt->value = icalproperty_get_exdate (dt->prop);
if (dt->tzid_param)
- cdt->tzid = icalparameter_get_tzid (dt->tzid_param);
+ cdt->tzid = g_strdup (icalparameter_get_tzid (dt->tzid_param));
else
cdt->tzid = NULL;
@@ -2290,6 +2290,8 @@ cal_component_set_exdate_list (CalComponent *comp, GSList *exdate_list)
dt = l->data;
+ /* Removing the DATE or DATE-TIME property will also remove
+ any TZID parameter. */
icalcomponent_remove_property (priv->icalcomp, dt->prop);
icalproperty_free (dt->prop);
g_free (dt);
@@ -3842,8 +3844,8 @@ cal_component_free_datetime (CalComponentDateTime *dt)
{
g_return_if_fail (dt != NULL);
- if (dt->value)
- g_free (dt->value);
+ g_free (dt->value);
+ g_free ((char*)dt->tzid);
}
/**
@@ -3866,6 +3868,7 @@ cal_component_free_exdate_list (GSList *exdate_list)
g_assert (cdt->value != NULL);
g_free (cdt->value);
+ g_free ((char*)cdt->tzid);
g_free (cdt);
}
diff --git a/calendar/cal-util/cal-recur.c b/calendar/cal-util/cal-recur.c
index 13c9f0533f..9afcee9784 100644
--- a/calendar/cal-util/cal-recur.c
+++ b/calendar/cal-util/cal-recur.c
@@ -675,6 +675,11 @@ cal_recur_generate_instances_of_rule (CalComponent *comp,
goto out;
}
+ /* FIXME: All floating times, including those from recurrence rules,
+ should be converted to the current timezone, otherwise the time_t
+ values may be outside the given range. This may also mean that
+ we need to add a timezone argument to the IDL method. */
+
if (dtstart.tzid && tz_cb)
start_zone = (*tz_cb) (dtstart.tzid, tz_cb_data);
if (dtend.tzid && tz_cb)
diff --git a/calendar/conduits/calendar/calendar-conduit.c b/calendar/conduits/calendar/calendar-conduit.c
index 823f1a8021..87f766cb37 100644
--- a/calendar/conduits/calendar/calendar-conduit.c
+++ b/calendar/conduits/calendar/calendar-conduit.c
@@ -452,6 +452,7 @@ local_record_from_comp (ECalLocalRecord *local, CalComponent *comp, ECalConduitC
local->appt->begin = *localtime (&dt_time);
}
+ cal_component_free_datetime (&dt);
cal_component_get_dtend (comp, &dt);
if (dt.value && time_add_day (dt_time, 1) != icaltime_as_timet_with_zone (*dt.value, get_timezone (ctxt->client, dt.tzid))) {
@@ -462,6 +463,7 @@ local_record_from_comp (ECalLocalRecord *local, CalComponent *comp, ECalConduitC
} else {
local->appt->event = 1;
}
+ cal_component_free_datetime (&dt);
/* Recurrence Rules */
local->appt->repeatType = repeatNone;
diff --git a/calendar/conduits/todo/todo-conduit.c b/calendar/conduits/todo/todo-conduit.c
index e0b3d6384d..05451ed754 100644
--- a/calendar/conduits/todo/todo-conduit.c
+++ b/calendar/conduits/todo/todo-conduit.c
@@ -408,6 +408,7 @@ local_record_from_comp (EToDoLocalRecord *local, CalComponent *comp, EToDoCondui
} else {
local->todo->indefinite = 1;
}
+ cal_component_free_datetime (&due);
cal_component_get_completed (comp, &completed);
if (completed) {
diff --git a/calendar/gui/calendar-commands.c b/calendar/gui/calendar-commands.c
index a0366685c9..f51a4142ca 100644
--- a/calendar/gui/calendar-commands.c
+++ b/calendar/gui/calendar-commands.c
@@ -354,18 +354,100 @@ get_shell_view_interface (BonoboControl *control)
return shell_view;
}
-/* Clears the folder bar label on the shell view */
+/* Displays the currently displayed time range in the folder bar label on the
+ shell view, according to which view we are showing. */
static void
-clear_folder_bar_label (BonoboControl *control)
+clear_folder_bar_label (GnomeCalendar *gcal, BonoboControl *control)
{
GNOME_Evolution_ShellView shell_view;
CORBA_Environment ev;
+ icaltimezone *zone;
+ struct icaltimetype start_tt, end_tt;
+ time_t start_time, end_time;
+ struct tm start_tm, end_tm;
+ char buffer[512], end_buffer[256];
+ GnomeCalendarViewType view;
+
+ /* FIXME: This isn't the same as the currently visible time range. */
+ gnome_calendar_get_selected_time_range (gcal, &start_time, &end_time);
+ zone = gnome_calendar_get_timezone (gcal);
+
+ start_tt = icaltime_from_timet_with_zone (start_time, FALSE, zone);
+ start_tm.tm_year = start_tt.year - 1900;
+ start_tm.tm_mon = start_tt.month - 1;
+ start_tm.tm_mday = start_tt.day;
+ start_tm.tm_hour = start_tt.hour;
+ start_tm.tm_min = start_tt.minute;
+ start_tm.tm_sec = start_tt.second;
+ start_tm.tm_isdst = -1;
+ start_tm.tm_wday = time_day_of_week (start_tt.day, start_tt.month - 1,
+ start_tt.year);
+
+ end_tt = icaltime_from_timet_with_zone (end_time, FALSE, zone);
+ end_tm.tm_year = end_tt.year - 1900;
+ end_tm.tm_mon = end_tt.month - 1;
+ end_tm.tm_mday = end_tt.day;
+ end_tm.tm_hour = end_tt.hour;
+ end_tm.tm_min = end_tt.minute;
+ end_tm.tm_sec = end_tt.second;
+ end_tm.tm_isdst = -1;
+ end_tm.tm_wday = time_day_of_week (end_tt.day, end_tt.month - 1,
+ end_tt.year);
+
+ view = gnome_calendar_get_view (gcal);
+
+ switch (view) {
+ case GNOME_CAL_DAY_VIEW:
+ strftime (buffer, sizeof (buffer),
+ _("%A %d %B %Y"), &start_tm);
+ break;
+ case GNOME_CAL_WORK_WEEK_VIEW:
+ case GNOME_CAL_WEEK_VIEW:
+ if (start_tm.tm_year == end_tm.tm_year) {
+ strftime (buffer, sizeof (buffer),
+ _("%a %d %b"), &start_tm);
+ strftime (end_buffer, sizeof (end_buffer),
+ _("%a %d %b %Y"), &end_tm);
+ strcat (buffer, " - ");
+ strcat (buffer, end_buffer);
+ } else {
+ strftime (buffer, sizeof (buffer),
+ _("%a %d %b %Y"), &start_tm);
+ strftime (end_buffer, sizeof (end_buffer),
+ _("%a %d %b %Y"), &end_tm);
+ strcat (buffer, " - ");
+ strcat (buffer, end_buffer);
+ }
+ break;
+ case GNOME_CAL_MONTH_VIEW:
+ if (start_tm.tm_year == end_tm.tm_year) {
+ strftime (buffer, sizeof (buffer),
+ _("%B"), &start_tm);
+ strftime (end_buffer, sizeof (end_buffer),
+ _("%B %Y"), &end_tm);
+ strcat (buffer, " - ");
+ strcat (buffer, end_buffer);
+ } else {
+ strftime (buffer, sizeof (buffer),
+ _("%B %Y"), &start_tm);
+ strftime (end_buffer, sizeof (end_buffer),
+ _("%B %Y"), &end_tm);
+ strcat (buffer, " - ");
+ strcat (buffer, end_buffer);
+ }
+ break;
+ default:
+ g_assert_not_reached ();
+ }
shell_view = get_shell_view_interface (control);
if (shell_view == CORBA_OBJECT_NIL)
return;
CORBA_exception_init (&ev);
+#if 0
+ GNOME_Evolution_ShellView_setFolderBarLabel (shell_view, buffer, &ev);
+#endif
GNOME_Evolution_ShellView_setFolderBarLabel (shell_view, "", &ev);
if (ev._major != CORBA_NO_EXCEPTION)
g_message ("clear_folder_bar_label(): Could not set the folder bar label");
@@ -465,7 +547,7 @@ calendar_control_activate (BonoboControl *control,
a default timezone already. */
calendar_config_check_timezone_set ();
- clear_folder_bar_label (control);
+ clear_folder_bar_label (gcal, control);
}
void
@@ -487,6 +569,13 @@ on_calendar_destroyed (GnomeCalendar *gcal)
all_calendars = g_list_remove (all_calendars, gcal);
}
+static void
+on_calendar_dates_shown_changed (GnomeCalendar *gcal)
+{
+ g_print ("In on_calendar_dates_shown_changed\n");
+}
+
+
GnomeCalendar *
new_calendar (void)
{
@@ -501,6 +590,9 @@ new_calendar (void)
gtk_signal_connect (GTK_OBJECT (gcal), "destroy",
GTK_SIGNAL_FUNC (on_calendar_destroyed), NULL);
+ gtk_signal_connect (GTK_OBJECT (gcal), "dates_shown_changed",
+ GTK_SIGNAL_FUNC (on_calendar_dates_shown_changed),
+ NULL);
all_calendars = g_list_prepend (all_calendars, gcal);
diff --git a/calendar/gui/calendar-config.c b/calendar/gui/calendar-config.c
index 7b133075f4..9b6d005259 100644
--- a/calendar/gui/calendar-config.c
+++ b/calendar/gui/calendar-config.c
@@ -281,12 +281,14 @@ calendar_config_get_timezone (void)
/* Sets the timezone. You shouldn't really set it to the empty string or NULL,
as this means that Evolution will show the timezone-setting dialog to ask
- the user for the timezone. */
+ the user for the timezone. It copies the string. */
void
calendar_config_set_timezone (gchar *timezone)
{
+ g_free (config->timezone);
+
if (timezone && timezone[0])
- config->timezone = timezone;
+ config->timezone = g_strdup (timezone);
else
config->timezone = NULL;
}
@@ -689,6 +691,8 @@ on_timezone_set (GnomeDialog *dialog,
e_timezone_dialog_get_timezone (etd, &display_name);
+ g_print ("Location: %s\n", display_name ? display_name : "");
+
if (display_name && display_name[0]) {
calendar_config_set_timezone (display_name);
diff --git a/calendar/gui/calendar-model.c b/calendar/gui/calendar-model.c
index 7fccf3937a..0cdd97b7d6 100644
--- a/calendar/gui/calendar-model.c
+++ b/calendar/gui/calendar-model.c
@@ -1068,7 +1068,8 @@ set_datetime (CalendarModel *model, CalComponent *comp, const char *value,
itt.second = tmp_tm.tm_sec;
dt.value = &itt;
- /* We assume it is being set to the current timezone. */
+ /* FIXME: We assume it is being set to the current timezone.
+ Is that OK? */
dt.tzid = icaltimezone_get_tzid (model->priv->zone);
(* set_func) (comp, &dt);
diff --git a/calendar/gui/comp-util.c b/calendar/gui/comp-util.c
index 944b5dcdce..5aa8d22a01 100644
--- a/calendar/gui/comp-util.c
+++ b/calendar/gui/comp-util.c
@@ -50,7 +50,7 @@ cal_comp_util_add_exdate (CalComponent *comp, time_t t, icaltimezone *zone)
cdt = g_new (CalComponentDateTime, 1);
cdt->value = g_new (struct icaltimetype, 1);
*cdt->value = icaltime_from_timet_with_zone (t, TRUE, zone);
- cdt->tzid = icaltimezone_get_tzid (zone);
+ cdt->tzid = g_strdup (icaltimezone_get_tzid (zone));
list = g_slist_append (list, cdt);
cal_component_set_exdate_list (comp, list);
diff --git a/calendar/gui/dialogs/recurrence-page.c b/calendar/gui/dialogs/recurrence-page.c
index b60a9f6b48..5c7e7d65cb 100644
--- a/calendar/gui/dialogs/recurrence-page.c
+++ b/calendar/gui/dialogs/recurrence-page.c
@@ -44,10 +44,6 @@
-/* We set this as the TZID on all exceptions added. When we actually fill
- the component, we replace it with the TZID from DTSTART. */
-static char *DUMMY_TZID = "DUMMY";
-
enum month_day_options {
MONTH_DAY_NTH,
MONTH_DAY_MON,
@@ -146,7 +142,7 @@ struct _RecurrencePagePrivate {
/* For ending date, created by hand */
GtkWidget *ending_date_edit;
- time_t ending_date;
+ struct icaltimetype ending_date_tt;
/* For ending count of occurrences, created by hand */
GtkWidget *ending_count_spin;
@@ -298,6 +294,7 @@ free_exception_clist_data (RecurrencePage *rpage)
dt = gtk_clist_get_row_data (clist, i);
g_free (dt->value);
+ g_free ((char*)dt->tzid);
g_free (dt);
gtk_clist_set_row_data (clist, i, NULL);
}
@@ -403,7 +400,7 @@ clear_widgets (RecurrencePage *rpage)
freq_map);
gtk_signal_handler_unblock_by_data (GTK_OBJECT (menu), rpage);
- priv->ending_date = time (NULL);
+ priv->ending_date_tt = icaltime_today ();
priv->ending_count = 1;
menu = gtk_option_menu_get_menu (GTK_OPTION_MENU (priv->ending_menu));
@@ -456,7 +453,7 @@ append_exception (RecurrencePage *rpage, CalComponentDateTime *datetime)
dt = g_new (CalComponentDateTime, 1);
dt->value = g_new (struct icaltimetype, 1);
*dt->value = *datetime->value;
- dt->tzid = datetime->tzid;
+ dt->tzid = g_strdup (datetime->tzid);
clist = GTK_CLIST (priv->exception_list);
@@ -760,15 +757,16 @@ simple_recur_to_comp (RecurrencePage *rpage, CalComponent *comp)
g_assert (priv->ending_date_edit != NULL);
g_assert (E_IS_DATE_EDIT (priv->ending_date_edit));
+ /* We only allow a DATE value to be set for the UNTIL property,
+ since we don't support sub-day recurrences. */
date_set = e_date_edit_get_date (E_DATE_EDIT (priv->ending_date_edit),
&r.until.year,
&r.until.month,
&r.until.day);
- e_date_edit_get_time_of_day (E_DATE_EDIT (priv->ending_date_edit),
- &r.until.hour,
- &r.until.minute);
g_assert (date_set);
+ r.until.is_date = 1;
+
break;
case ENDING_FOREVER:
@@ -794,11 +792,9 @@ static void
fill_component (RecurrencePage *rpage, CalComponent *comp)
{
RecurrencePagePrivate *priv;
- CalComponentDateTime start_date;
enum recur_type recur_type;
GtkCList *exception_list;
GSList *list;
- const char *tzid;
int i;
priv = rpage->priv;
@@ -828,10 +824,6 @@ fill_component (RecurrencePage *rpage, CalComponent *comp)
/* Set exceptions */
- cal_component_get_dtstart (comp, &start_date);
- tzid = start_date.tzid;
- cal_component_free_datetime (&start_date);
-
list = NULL;
exception_list = GTK_CLIST (priv->exception_list);
for (i = 0; i < exception_list->rows; i++) {
@@ -844,15 +836,9 @@ fill_component (RecurrencePage *rpage, CalComponent *comp)
g_assert (dt != NULL);
*cdt->value = *dt->value;
+ cdt->tzid = g_strdup (dt->tzid);
- /* If the dummy TZID was used, we use the TZID from the
- start date. We do this because we don't allow editing
- of timezones for RDATEs, so we try to use the same timezone
- as the DTSTART for all new exceptions added. */
- if (dt->tzid == DUMMY_TZID)
- cdt->tzid = tzid;
- else
- cdt->tzid = dt->tzid;
+ g_print ("Adding exception is_date: %i\n", cdt->value->is_date);
list = g_slist_prepend (list, cdt);
}
@@ -1236,7 +1222,9 @@ make_ending_until_special (RecurrencePage *rpage)
/* Set the value */
- e_date_edit_set_time (de, priv->ending_date);
+ e_date_edit_set_date (de, priv->ending_date_tt.year,
+ priv->ending_date_tt.month,
+ priv->ending_date_tt.day);
gtk_signal_connect (GTK_OBJECT (de), "changed",
GTK_SIGNAL_FUNC (ending_until_changed_cb), rpage);
@@ -1365,8 +1353,7 @@ fill_ending_date (RecurrencePage *rpage, struct icalrecurrencetype *r)
} else {
/* Ending date */
- /* FIXME: UNTIL needs to be checked. */
- priv->ending_date = icaltime_as_timet (r->until);
+ priv->ending_date_tt = r->until;
e_dialog_option_menu_set (priv->ending_menu,
ENDING_UNTIL,
ending_types_map);
@@ -1787,12 +1774,10 @@ recurrence_page_set_dates (CompEditorPage *page, CompEditorPageDates *dates)
/* Copy the TZID from the old property.
FIXME: Should get notified when the TZID changes.*/
cal_component_get_dtstart (priv->comp, &old_dt);
- dt.tzid = NULL;
- if (old_dt.tzid)
- dt.tzid = old_dt.tzid;
- cal_component_free_datetime (&old_dt);
+ dt.tzid = old_dt.tzid;
cal_component_set_dtstart (priv->comp, &dt);
+ cal_component_free_datetime (&old_dt);
}
if (dates->end) {
@@ -1801,12 +1786,10 @@ recurrence_page_set_dates (CompEditorPage *page, CompEditorPageDates *dates)
/* Copy the TZID from the old property.
FIXME: Should get notified when the TZID changes.*/
cal_component_get_dtend (priv->comp, &old_dt);
- dt.tzid = NULL;
- if (old_dt.tzid)
- dt.tzid = old_dt.tzid;
- cal_component_free_datetime (&old_dt);
+ dt.tzid = old_dt.tzid;
cal_component_set_dtend (priv->comp, &dt);
+ cal_component_free_datetime (&old_dt);
}
/* Update the weekday picker if necessary */
@@ -1977,19 +1960,15 @@ exception_add_cb (GtkWidget *widget, gpointer data)
field_changed (rpage);
dt.value = &icaltime;
- /* We set this to the dummy TZID for now. When we fill the component
- we will replace it with the TZID from DTSTART. */
- dt.tzid = DUMMY_TZID;
+ /* We use DATE values for exceptions, so we don't need a TZID. */
+ dt.tzid = NULL;
icaltime.is_date = 1;
date_set = e_date_edit_get_date (E_DATE_EDIT (priv->exception_date),
&icaltime.year,
&icaltime.month,
&icaltime.day);
- e_date_edit_get_time_of_day (E_DATE_EDIT (priv->exception_date),
- &icaltime.hour,
- &icaltime.minute);
g_assert (date_set);
append_exception (rpage, &dt);
@@ -2022,9 +2001,14 @@ exception_modify_cb (GtkWidget *widget, gpointer data)
tt = dt->value;
e_date_edit_get_date (E_DATE_EDIT (priv->exception_date),
&tt->year, &tt->month, &tt->day);
- e_date_edit_get_time_of_day (E_DATE_EDIT (priv->exception_date),
- &tt->hour, &tt->minute);
+ tt->hour = 0;
+ tt->minute = 0;
tt->second = 0;
+ tt->is_date = 1;
+
+ /* We get rid of any old TZID, since we are using a DATE value now. */
+ g_free ((char*)dt->tzid);
+ dt->tzid = NULL;
gtk_clist_set_text (clist, sel, 0, get_exception_string (dt));
@@ -2055,6 +2039,7 @@ exception_delete_cb (GtkWidget *widget, gpointer data)
dt = gtk_clist_get_row_data (clist, sel);
g_assert (dt != NULL);
g_free (dt->value);
+ g_free ((char*)dt->tzid);
g_free (dt);
gtk_clist_remove (clist, sel);
diff --git a/calendar/gui/e-calendar-table.c b/calendar/gui/e-calendar-table.c
index a513dbf6f5..d49a571608 100644
--- a/calendar/gui/e-calendar-table.c
+++ b/calendar/gui/e-calendar-table.c
@@ -38,6 +38,7 @@
#include <gal/e-table/e-cell-toggle.h>
#include <gal/e-table/e-cell-text.h>
#include <gal/e-table/e-cell-combo.h>
+#include <gal/widgets/e-popup-menu.h>
#include <widgets/misc/e-cell-date-edit.h>
#include "e-calendar-table.h"
#include "calendar-config.h"
@@ -836,25 +837,38 @@ delete_cb (GtkWidget *menuitem, gpointer data)
e_calendar_table_delete_selected (cal_table);
}
-static GnomeUIInfo tasks_popup_one[] = {
- GNOMEUIINFO_ITEM_NONE (N_("Edit this task"), NULL, e_calendar_table_on_open_task),
- GNOMEUIINFO_ITEM_NONE (N_("Cut"), NULL, e_calendar_table_on_cut),
- GNOMEUIINFO_ITEM_NONE (N_("Copy"), NULL, e_calendar_table_on_copy),
- GNOMEUIINFO_ITEM_NONE (N_("Paste"), NULL, e_calendar_table_on_paste),
- GNOMEUIINFO_SEPARATOR,
- GNOMEUIINFO_ITEM_NONE (N_("Mark as complete"), NULL, mark_as_complete_cb),
- GNOMEUIINFO_ITEM_NONE (N_("Delete this task"), NULL, delete_cb),
- GNOMEUIINFO_END
+
+enum {
+ MASK_SINGLE = 1 << 0, /* For commands that work on 1 task. */
+ MASK_MULTIPLE = 1 << 1, /* For commands for multiple tasks. */
};
-static GnomeUIInfo tasks_popup_many[] = {
- GNOMEUIINFO_ITEM_NONE (N_("Cut"), NULL, e_calendar_table_on_cut),
- GNOMEUIINFO_ITEM_NONE (N_("Copy"), NULL, e_calendar_table_on_copy),
- GNOMEUIINFO_ITEM_NONE (N_("Paste"), NULL, e_calendar_table_on_paste),
- GNOMEUIINFO_SEPARATOR,
- GNOMEUIINFO_ITEM_NONE (N_("Mark tasks as complete"), NULL, mark_as_complete_cb),
- GNOMEUIINFO_ITEM_NONE (N_("Delete selected tasks"), NULL, delete_cb),
- GNOMEUIINFO_END
+
+static EPopupMenu tasks_popup_menu [] = {
+ { N_("_Open"), NULL,
+ e_calendar_table_on_open_task, NULL, MASK_SINGLE },
+ { "", NULL, NULL, NULL, MASK_SINGLE },
+
+ { N_("C_ut"), NULL,
+ e_calendar_table_on_cut, NULL, 0 },
+ { N_("_Copy"), NULL,
+ e_calendar_table_on_copy, NULL, 0 },
+ { N_("_Paste"), NULL,
+ e_calendar_table_on_paste, NULL, 0 },
+
+ { "", NULL, NULL, NULL, 0 },
+
+ { N_("_Mark as Complete"), NULL,
+ mark_as_complete_cb, NULL, MASK_SINGLE },
+ { N_("_Delete this Task"), NULL,
+ delete_cb, NULL, MASK_SINGLE },
+
+ { N_("_Mark Tasks as Complete"), NULL,
+ mark_as_complete_cb, NULL, MASK_MULTIPLE },
+ { N_("_Delete Selected Tasks"), NULL,
+ delete_cb, NULL, MASK_MULTIPLE },
+
+ { NULL, NULL, NULL, NULL, 0 }
};
static gint
@@ -864,19 +878,20 @@ e_calendar_table_on_right_click (ETable *table,
GdkEventButton *event,
ECalendarTable *cal_table)
{
- GtkWidget *popup_menu;
int n_selected;
+ int hide_mask = 0;
+ int disable_mask = 0;
n_selected = e_table_selected_count (table);
g_assert (n_selected > 0);
if (n_selected == 1)
- popup_menu = gnome_popup_menu_new (tasks_popup_one);
+ hide_mask = MASK_MULTIPLE;
else
- popup_menu = gnome_popup_menu_new (tasks_popup_many);
+ hide_mask = MASK_SINGLE;
- gnome_popup_menu_do_popup_modal (popup_menu, NULL, NULL, event, cal_table);
- gtk_widget_destroy (popup_menu);
+ e_popup_menu_run (tasks_popup_menu, (GdkEvent *) event,
+ disable_mask, hide_mask, cal_table);
return TRUE;
}
diff --git a/calendar/gui/e-day-view.c b/calendar/gui/e-day-view.c
index 7f11e2d7b5..ca12b87176 100644
--- a/calendar/gui/e-day-view.c
+++ b/calendar/gui/e-day-view.c
@@ -3280,7 +3280,7 @@ e_day_view_on_event_double_click (EDayView *day_view,
enum {
/*
- * This is used to "flag" events that can not be editted
+ * This is used to "flag" events that can not be edited
*/
MASK_EDITABLE = 1,
@@ -3303,49 +3303,53 @@ enum {
};
static EPopupMenu main_items [] = {
- { N_("Paste"), NULL,
- e_day_view_on_paste, NULL, 0 },
+ { N_("New _Appointment"), NULL,
+ e_day_view_on_new_appointment, NULL, 0 },
+ { N_("New All Day _Event"), NULL,
+ e_day_view_on_new_event, NULL, 0 },
{ "", NULL, NULL, NULL, 0 },
- { N_("New Appointment"), NULL,
- e_day_view_on_new_appointment, NULL, 0 },
- { N_("New All Day Event"), NULL,
- e_day_view_on_new_event, NULL, 0 },
+ { N_("_Paste"), NULL,
+ e_day_view_on_paste, NULL, 0 },
{ "", NULL, NULL, NULL, 0 },
- { N_("Go to Today"), NULL,
+ { N_("Go to _Today"), NULL,
e_day_view_on_goto_today, NULL, 0 },
- { N_("Go to Date..."), NULL,
+ { N_("_Go to Date..."), NULL,
e_day_view_on_goto_date, NULL, 0 },
{ NULL, NULL, NULL, NULL, 0 }
};
static EPopupMenu child_items [] = {
- { N_("Open"), NULL,
+ { N_("_Open"), NULL,
e_day_view_on_edit_appointment, NULL, MASK_EDITABLE | MASK_EDITING },
- { N_("Delete this Appointment"), NULL,
+ { N_("_Delete this Appointment"), NULL,
e_day_view_on_delete_appointment, NULL, MASK_EDITABLE | MASK_SINGLE | MASK_EDITING },
- { N_("Cut"), NULL,
+
+ /* Only show this separator if one of the above is shown. */
+ { "", NULL, NULL, NULL, MASK_EDITABLE | MASK_EDITING },
+
+
+ { N_("C_ut"), NULL,
e_day_view_on_cut, NULL, MASK_EDITABLE | MASK_EDITING },
- { N_("Copy"), NULL,
+ { N_("_Copy"), NULL,
e_day_view_on_copy, NULL, 0 },
- { N_("Paste"), NULL,
+ { N_("_Paste"), NULL,
e_day_view_on_paste, NULL, 0 },
- { "", NULL, NULL, NULL, MASK_SINGLE},
/*
* The following are only shown if this is a recurring event
*/
- { "", NULL, NULL, NULL, MASK_SINGLE},
- { N_("Make this Occurrence Movable"), NULL,
+ { "", NULL, NULL, NULL, MASK_RECURRING | MASK_EDITING },
+ { N_("Make this Occurrence _Movable"), NULL,
e_day_view_on_unrecur_appointment, NULL, MASK_RECURRING | MASK_EDITING },
- { N_("Delete this Occurrence"), NULL,
+ { N_("Delete this _Occurrence"), NULL,
e_day_view_on_delete_occurrence, NULL, MASK_RECURRING | MASK_EDITING },
- { N_("Delete all Occurrences"), NULL,
+ { N_("Delete _All Occurrences"), NULL,
e_day_view_on_delete_appointment, NULL, MASK_RECURRING | MASK_EDITING },
{ NULL, NULL, NULL, NULL, 0 }
diff --git a/calendar/gui/e-itip-control.c b/calendar/gui/e-itip-control.c
index c22c0a643f..45f49421ae 100644
--- a/calendar/gui/e-itip-control.c
+++ b/calendar/gui/e-itip-control.c
@@ -600,15 +600,23 @@ set_date_label (GtkWidget *lbl, CalComponent *comp)
cal_component_get_dtstart (comp, &datetime);
if (datetime.value)
start = icaltime_as_timet (*datetime.value);
+ cal_component_free_datetime (&datetime);
+
cal_component_get_dtend (comp, &datetime);
if (datetime.value)
end = icaltime_as_timet (*datetime.value);
+ cal_component_free_datetime (&datetime);
+
cal_component_get_due (comp, &datetime);
if (datetime.value)
due = icaltime_as_timet (*datetime.value);
+ cal_component_free_datetime (&datetime);
+
cal_component_get_completed (comp, &datetime.value);
- if (datetime.value)
+ if (datetime.value) {
complete = icaltime_as_timet (*datetime.value);
+ cal_component_free_icaltimetype (datetime.value);
+ }
buffer[0] = '\0';
@@ -1020,7 +1028,7 @@ send_freebusy (EItipControl *itip)
priv = itip->priv;
- /* FIXME: timezones. */
+ /* FIXME: timezones and free these. */
cal_component_get_dtstart (priv->comp, &datetime);
start = icaltime_as_timet (*datetime.value);
cal_component_get_dtend (priv->comp, &datetime);
diff --git a/calendar/gui/e-week-view-layout.c b/calendar/gui/e-week-view-layout.c
index 21552e5f17..5e8331c76c 100644
--- a/calendar/gui/e-week-view-layout.c
+++ b/calendar/gui/e-week-view-layout.c
@@ -131,7 +131,7 @@ e_week_view_layout_event (EWeekViewEvent *event,
gint free_row, row, day, span_num, spans_index, num_spans, days_shown;
EWeekViewEventSpan span, *old_span;
- days_shown = multi_week_view ? weeks_shown * 7 - 1 : 7 - 1;
+ days_shown = multi_week_view ? weeks_shown * 7 : 7;
start_day = e_week_view_find_day (event->start, FALSE, days_shown,
day_starts);
end_day = e_week_view_find_day (event->end, TRUE, days_shown,
diff --git a/calendar/gui/e-week-view.c b/calendar/gui/e-week-view.c
index 0f230c21fe..9fbf7cb660 100644
--- a/calendar/gui/e-week-view.c
+++ b/calendar/gui/e-week-view.c
@@ -3141,40 +3141,45 @@ enum {
};
static EPopupMenu main_items [] = {
- { N_("Paste"), NULL,
- e_week_view_on_paste, NULL, 0 },
+ { N_("New _Appointment..."), NULL,
+ e_week_view_on_new_appointment, NULL, 0 },
+ { N_("New All Day _Event"), NULL,
+ e_week_view_on_new_event, NULL, 0 },
{ "", NULL, NULL, NULL, 0 },
- { N_("New Appointment..."), NULL,
- e_week_view_on_new_appointment, NULL, 0 },
- { N_("New All Day Event"), NULL,
- e_week_view_on_new_event, NULL, 0 },
+ { N_("_Paste"), NULL,
+ e_week_view_on_paste, NULL, 0 },
{ "", NULL, NULL, NULL, 0 },
- { N_("Go to Today"), NULL,
+ { N_("Go to _Today"), NULL,
e_week_view_on_goto_today, NULL, 0 },
- { N_("Go to Date..."), NULL,
+ { N_("_Go to Date..."), NULL,
e_week_view_on_goto_date, NULL, 0 },
{ NULL, NULL, NULL, NULL, 0 }
};
static EPopupMenu child_items [] = {
- { N_("Open"), NULL,
+ { N_("_Open"), NULL,
e_week_view_on_edit_appointment, NULL, MASK_EDITABLE | MASK_EDITING },
- { N_("Delete this Appointment"), NULL,
+ { N_("_Delete this Appointment"), NULL,
e_week_view_on_delete_appointment, NULL, MASK_EDITABLE | MASK_SINGLE | MASK_EDITING },
- { N_("Cut"), NULL,
+
+ /* Only show this separator if one of the above is shown. */
+ { "", NULL, NULL, NULL, MASK_EDITABLE | MASK_EDITING },
+
+
+ { N_("C_ut"), NULL,
e_week_view_on_cut, NULL, MASK_EDITING | MASK_EDITABLE },
- { N_("Copy"), NULL,
+ { N_("_Copy"), NULL,
e_week_view_on_copy, NULL, MASK_EDITING | MASK_EDITABLE },
- { N_("Paste"), NULL,
+ { N_("_Paste"), NULL,
e_week_view_on_paste, NULL, 0 },
{ "", NULL, NULL, NULL, 0},
- { N_("New Appointment..."), NULL,
+ { N_("New _Appointment..."), NULL,
e_week_view_on_new_appointment, NULL, 0 },
{ "", NULL, NULL, NULL, MASK_SINGLE },
@@ -3183,11 +3188,11 @@ static EPopupMenu child_items [] = {
* The following are only shown if this is a recurring event
*/
{ "", NULL, NULL, NULL, MASK_SINGLE},
- { N_("Make this Occurrence Movable"), NULL,
+ { N_("Make this Occurrence _Movable"), NULL,
e_week_view_on_unrecur_appointment, NULL, MASK_RECURRING | MASK_EDITING },
- { N_("Delete this Occurrence"), NULL,
+ { N_("Delete this _Occurrence"), NULL,
e_week_view_on_delete_occurrence, NULL, MASK_RECURRING | MASK_EDITING },
- { N_("Delete All Occurrences"), NULL,
+ { N_("Delete _All Occurrences"), NULL,
e_week_view_on_delete_appointment, NULL, MASK_RECURRING | MASK_EDITING },
{ NULL, NULL, NULL, NULL, 0 }
diff --git a/calendar/gui/gnome-cal.c b/calendar/gui/gnome-cal.c
index 7a50df5389..96f7fbc405 100644
--- a/calendar/gui/gnome-cal.c
+++ b/calendar/gui/gnome-cal.c
@@ -137,6 +137,16 @@ struct _GnomeCalendarPrivate {
icaltimezone *zone;
};
+/* Signal IDs */
+
+enum {
+ DATES_SHOWN_CHANGED,
+ LAST_SIGNAL
+};
+
+static guint gnome_calendar_signals[LAST_SIGNAL];
+
+
static void gnome_calendar_class_init (GnomeCalendarClass *class);
@@ -158,6 +168,8 @@ static void gnome_calendar_on_date_navigator_date_range_changed (ECalendarItem *
GnomeCalendar *gcal);
static void gnome_calendar_on_date_navigator_selection_changed (ECalendarItem *calitem,
GnomeCalendar *gcal);
+static void gnome_calendar_notify_dates_shown_changed (GnomeCalendar *gcal);
+
static GtkVBoxClass *parent_class;
@@ -197,7 +209,22 @@ gnome_calendar_class_init (GnomeCalendarClass *class)
parent_class = gtk_type_class (GTK_TYPE_VBOX);
+ gnome_calendar_signals[DATES_SHOWN_CHANGED] =
+ gtk_signal_new ("dates_shown_changed",
+ GTK_RUN_LAST,
+ object_class->type,
+ GTK_SIGNAL_OFFSET (GnomeCalendarClass,
+ dates_shown_changed),
+ gtk_marshal_NONE__NONE,
+ GTK_TYPE_NONE, 0);
+
+ gtk_object_class_add_signals (object_class,
+ gnome_calendar_signals,
+ LAST_SIGNAL);
+
object_class->destroy = gnome_calendar_destroy;
+
+ class->dates_shown_changed = NULL;
}
/**
@@ -496,6 +523,7 @@ gnome_calendar_goto (GnomeCalendar *gcal, time_t new_time)
gnome_calendar_update_view_times (gcal);
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
}
@@ -588,6 +616,7 @@ gnome_calendar_direction (GnomeCalendar *gcal, int direction)
gnome_calendar_update_view_times (gcal);
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
}
void
@@ -784,6 +813,7 @@ gnome_calendar_set_view (GnomeCalendar *gcal, GnomeCalendarViewType view_type,
set_view (gcal, view_type, range_selected, grab_focus);
gnome_calendar_update_view_times (gcal);
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
}
/* Callback used when the view collection asks us to display a particular view */
@@ -1366,6 +1396,7 @@ gnome_calendar_update_config_settings (GnomeCalendar *gcal,
/* The range of days shown may have changed, so we update the date
navigator if needed. */
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
}
@@ -1713,11 +1744,13 @@ gnome_calendar_on_date_navigator_selection_changed (ECalendarItem *calitem,
set_view (gcal, GNOME_CAL_MONTH_VIEW, TRUE, FALSE);
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
} else if (new_days_shown == 7 && starts_on_week_start_day) {
e_week_view_set_first_day_shown (E_WEEK_VIEW (priv->week_view), &new_start_date);
set_view (gcal, GNOME_CAL_WEEK_VIEW, TRUE, FALSE);
gnome_calendar_update_date_navigator (gcal);
+ gnome_calendar_notify_dates_shown_changed (gcal);
} else {
gint start_year, start_month, start_day;
gint end_year, end_month, end_day;
@@ -2013,3 +2046,12 @@ gnome_calendar_get_timezone (GnomeCalendar *gcal)
return gcal->priv->zone;
}
+
+static void
+gnome_calendar_notify_dates_shown_changed (GnomeCalendar *gcal)
+{
+ g_return_if_fail (GNOME_IS_CALENDAR (gcal));
+
+ gtk_signal_emit (GTK_OBJECT (gcal),
+ gnome_calendar_signals[DATES_SHOWN_CHANGED]);
+}
diff --git a/calendar/gui/gnome-cal.h b/calendar/gui/gnome-cal.h
index 492d6bcfd1..5409c6f55e 100644
--- a/calendar/gui/gnome-cal.h
+++ b/calendar/gui/gnome-cal.h
@@ -65,6 +65,9 @@ struct _GnomeCalendar {
struct _GnomeCalendarClass {
GtkVBoxClass parent_class;
+
+ /* Notification signals */
+ void (* dates_shown_changed) (GnomeCalendar *gcal);
};
diff --git a/calendar/gui/tag-calendar.c b/calendar/gui/tag-calendar.c
index 8cdbac8462..a84f4e965e 100644
--- a/calendar/gui/tag-calendar.c
+++ b/calendar/gui/tag-calendar.c
@@ -91,7 +91,7 @@ tag_calendar_cb (CalComponent *comp,
struct icaltimetype start_tt, end_tt;
start_tt = icaltime_from_timet_with_zone (istart, FALSE, c->zone);
- end_tt = icaltime_from_timet_with_zone (iend, FALSE, c->zone);
+ end_tt = icaltime_from_timet_with_zone (iend - 1, FALSE, c->zone);
e_calendar_item_mark_days (c->calitem,
start_tt.year, start_tt.month - 1,
start_tt.day,
diff --git a/calendar/gui/tasks-control.c b/calendar/gui/tasks-control.c
index df3236fff0..f2d594610a 100644
--- a/calendar/gui/tasks-control.c
+++ b/calendar/gui/tasks-control.c
@@ -240,7 +240,10 @@ static BonoboUIVerb verbs [] = {
static EPixmap pixmaps [] = {
E_PIXMAP ("/menu/File/New/NewFirstItem/NewTask", "evolution-tasks-mini.png"),
- E_PIXMAP ("/menu/File/Print/Print", "print.xpm"),
+ E_PIXMAP ("/menu/File/Print/Print", "print.xpm"),
+ E_PIXMAP ("/menu/File/Print/Print Preview", "print-preview.xpm"),
+ E_PIXMAP ("/Toolbar/New", "buttons/new_appointment.png"),
+ E_PIXMAP ("/Toolbar/Print", "buttons/print.png"),
E_PIXMAP_END
};