aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--calendar/ChangeLog20
-rw-r--r--calendar/conduits/calendar/Makefile.am2
-rw-r--r--calendar/conduits/todo/Makefile.am2
-rw-r--r--calendar/gui/dialogs/Makefile.am2
-rw-r--r--calendar/gui/dialogs/delete-error.c107
-rw-r--r--calendar/gui/dialogs/delete-error.h30
-rw-r--r--calendar/gui/e-calendar-table.c8
-rw-r--r--calendar/gui/e-day-view.c16
-rw-r--r--calendar/gui/e-itip-control.c11
-rw-r--r--calendar/gui/e-tasks.c3
-rw-r--r--calendar/gui/e-week-view.c15
11 files changed, 187 insertions, 29 deletions
diff --git a/calendar/ChangeLog b/calendar/ChangeLog
index dd0925ecef..372b131dff 100644
--- a/calendar/ChangeLog
+++ b/calendar/ChangeLog
@@ -1,3 +1,23 @@
+2003-04-24 JP Rosevear <jpr@ximian.com>
+
+ Fixes #37552
+
+ * gui/dialogs/Makefile.am: build delete-error.[hc]
+
+ * gui/e-tasks.c: wrap calls to cal_client_remove_object with
+ delete_error_dialog
+
+ * gui/e-itip-control.c: ditto
+
+ * gui/e-calendar-table.c: ditto
+
+ * gui/e-week-view.c: ditto
+
+ * gui/e-day-view.c: ditto
+
+ * gui/dialogs/delete-error.[hc]: gemerate an error message based
+ on result
+
2003-04-23 Rodrigo Moya <rodrigo@ximian.com>
* gui/e-day-view.c (e_day_view_on_delete_occurrence): removed
diff --git a/calendar/conduits/calendar/Makefile.am b/calendar/conduits/calendar/Makefile.am
index ef68e17fcf..84efaaa23e 100644
--- a/calendar/conduits/calendar/Makefile.am
+++ b/calendar/conduits/calendar/Makefile.am
@@ -23,7 +23,7 @@ libecalendar_conduit_la_LIBADD = \
$(top_builddir)/libversit/libversit.la \
$(top_builddir)/libical/src/libical/libical-static.la \
$(top_builddir)/libwombat/libwombat-static.la \
- $(top_builddir)/e-util/libeconduit-static.la \
+ $(top_builddir)/e-util/libeconduit.la \
$(EVOLUTION_CALENDAR_CONDUIT_LIBS)
e-calendar.conduit: e-calendar.conduit.in Makefile
diff --git a/calendar/conduits/todo/Makefile.am b/calendar/conduits/todo/Makefile.am
index 5ddc2b7c4e..43e3c5940b 100644
--- a/calendar/conduits/todo/Makefile.am
+++ b/calendar/conduits/todo/Makefile.am
@@ -23,7 +23,7 @@ libetodo_conduit_la_LIBADD = \
$(top_builddir)/libversit/libversit.la \
$(top_builddir)/libical/src/libical/libical-static.la \
$(top_builddir)/libwombat/libwombat-static.la \
- $(top_builddir)/e-util/libeconduit-static.la \
+ $(top_builddir)/e-util/libeconduit.la \
$(EVOLUTION_CALENDAR_CONDUIT_LIBS)
e-todo.conduit: e-todo.conduit.in Makefile
diff --git a/calendar/gui/dialogs/Makefile.am b/calendar/gui/dialogs/Makefile.am
index 8dc79e7fe7..4bbdd90ca5 100644
--- a/calendar/gui/dialogs/Makefile.am
+++ b/calendar/gui/dialogs/Makefile.am
@@ -53,6 +53,8 @@ libcal_dialogs_la_SOURCES = \
comp-editor-util.h \
delete-comp.c \
delete-comp.h \
+ delete-error.c \
+ delete-error.h \
e-delegate-dialog.c \
e-delegate-dialog.h \
event-editor.c \
diff --git a/calendar/gui/dialogs/delete-error.c b/calendar/gui/dialogs/delete-error.c
new file mode 100644
index 0000000000..947aab1e97
--- /dev/null
+++ b/calendar/gui/dialogs/delete-error.c
@@ -0,0 +1,107 @@
+/* Evolution calendar - Send calendar component dialog
+ *
+ * Copyright (C) 2001 Ximian, Inc.
+ *
+ * Author: JP Rosevear <jpr@ximian.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of version 2 of the GNU General Public
+ * License as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307, USA.
+ */
+
+#ifdef HAVE_CONFIG_H
+#include <config.h>
+#endif
+
+#include <glib.h>
+#include <gtk/gtkmessagedialog.h>
+#include <libgnome/gnome-i18n.h>
+#include <libgnomeui/gnome-uidefs.h>
+#include <gal/widgets/e-unicode.h>
+#include "delete-error.h"
+
+
+
+/**
+ * delete_error_dialog:
+ *
+ * Shows any applicable error messages as the result of deleting and object
+ *
+ **/
+void
+delete_error_dialog (CalClientResult result, CalComponentVType vtype)
+{
+ GtkWidget *dialog;
+ const char *str;
+
+ switch (result) {
+ case CAL_CLIENT_RESULT_CORBA_ERROR:
+ switch (vtype) {
+ case CAL_COMPONENT_EVENT:
+ str = _("The event could not be deleted due to a corba error");
+ break;
+ case CAL_COMPONENT_TODO:
+ str = _("The task could not be deleted due to a corba error");
+ break;
+ case CAL_COMPONENT_JOURNAL:
+ str = _("The journal entry could not be deleted due to a corba error");
+ break;
+ default:
+ str = _("The item could not be deleted due to a corba error");
+ break;
+ }
+ break;
+ case CAL_CLIENT_RESULT_PERMISSION_DENIED:
+ switch (vtype) {
+ case CAL_COMPONENT_EVENT:
+ str = _("The event could not be deleted because permission was denied");
+ break;
+ case CAL_COMPONENT_TODO:
+ str = _("The task could not be deleted because permission was denied");
+ break;
+ case CAL_COMPONENT_JOURNAL:
+ str = _("The journal entry could not be deleted because permission was denied");
+ break;
+ default:
+ str = _("The item could not be deleted because permission was denied");
+ break;
+ }
+ break;
+ case CAL_CLIENT_RESULT_INVALID_OBJECT:
+ switch (vtype) {
+ case CAL_COMPONENT_EVENT:
+ str = _("The event could not be deleted because it was invalid");
+ break;
+ case CAL_COMPONENT_TODO:
+ str = _("The task could not be deleted because it was invalid");
+ break;
+ case CAL_COMPONENT_JOURNAL:
+ str = _("The journal entry could not be deleted because it was invalid");
+ break;
+ default:
+ str = _("The item could not be deleted because it was invalid");
+ break;
+ }
+ break;
+ case CAL_CLIENT_RESULT_SUCCESS:
+ case CAL_CLIENT_RESULT_NOT_FOUND:
+ default:
+ /* If not found, we don't care - its gone anyhow */
+ return;
+ }
+
+ dialog = gtk_message_dialog_new (NULL, GTK_DIALOG_MODAL,
+ GTK_MESSAGE_ERROR,
+ GTK_BUTTONS_OK, str);
+ gtk_dialog_run (GTK_DIALOG (dialog));
+ gtk_widget_destroy (dialog);
+}
diff --git a/calendar/gui/dialogs/delete-error.h b/calendar/gui/dialogs/delete-error.h
new file mode 100644
index 0000000000..dcef2fa3be
--- /dev/null
+++ b/calendar/gui/dialogs/delete-error.h
@@ -0,0 +1,30 @@
+/* Evolution calendar - Send calendar component dialog
+ *
+ * Copyright (C) 2001 Ximian, Inc.
+ *
+ * Author: JP Rosevear <jpr@ximian.com>
+ *
+ * This program is free software; you can redistribute it and/or
+ * modify it under the terms of version 2 of the GNU General Public
+ * License as published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307, USA.
+ */
+
+#ifndef DELETE_ERROR_H
+#define DELETE_ERROR_H
+
+#include <glib.h>
+#include <cal-client/cal-client.h>
+#include <cal-util/cal-component.h>
+
+void delete_error_dialog (CalClientResult result, CalComponentVType vtype);
+
+#endif
diff --git a/calendar/gui/e-calendar-table.c b/calendar/gui/e-calendar-table.c
index 41d12685b8..b3ac35e045 100644
--- a/calendar/gui/e-calendar-table.c
+++ b/calendar/gui/e-calendar-table.c
@@ -48,6 +48,7 @@
#include "calendar-model.h"
#include "print.h"
#include "dialogs/delete-comp.h"
+#include "dialogs/delete-error.h"
#include "dialogs/task-editor.h"
/* Pixmaps. */
@@ -716,13 +717,10 @@ delete_selected_components (ECalendarTable *cal_table)
for (l = uids; l; l = l->next) {
const char *uid;
-
+
uid = l->data;
- /* We don't check the return value; FALSE can mean the object
- * was not in the server anyways.
- */
- cal_client_remove_object (client, uid);
+ delete_error_dialog (cal_client_remove_object (client, uid), CAL_COMPONENT_TODO);
}
calendar_model_set_status_message (e_calendar_table_get_model (cal_table), NULL);
diff --git a/calendar/gui/e-day-view.c b/calendar/gui/e-day-view.c
index 2fae66475c..ed6cc4f5cb 100644
--- a/calendar/gui/e-day-view.c
+++ b/calendar/gui/e-day-view.c
@@ -54,6 +54,7 @@
#include "cal-util/timeutil.h"
#include "dialogs/delete-comp.h"
+#include "dialogs/delete-error.h"
#include "dialogs/send-comp.h"
#include "dialogs/cancel-comp.h"
#include "dialogs/recur-comp.h"
@@ -2821,7 +2822,7 @@ e_day_view_cut_clipboard (EDayView *day_view)
e_day_view_copy_clipboard (day_view);
cal_component_get_uid (event->comp, &uid);
- cal_client_remove_object (day_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (day_view->client, uid), CAL_COMPONENT_EVENT);
e_day_view_set_status_message (day_view, NULL);
}
@@ -4077,10 +4078,7 @@ e_day_view_delete_event_internal (EDayView *day_view, EDayViewEvent *event)
cal_component_get_uid (event->comp, &uid);
- /* We don't check the return value; FALSE can mean the object
- * was not in the server anyways.
- */
- cal_client_remove_object (day_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (day_view->client, uid), CAL_COMPONENT_EVENT);
}
}
@@ -4141,9 +4139,9 @@ e_day_view_delete_occurrence_internal (EDayView *day_view, EDayViewEvent *event)
const char *uid;
cal_component_get_uid (event->comp, &uid);
- if (cal_client_remove_object_with_mod (day_view->client, uid, CALOBJ_MOD_THIS) != CAL_CLIENT_RESULT_SUCCESS)
- g_message ("e_day_view_on_delete_occurrence(): Could not update the object!");
-
+
+ delete_error_dialog (cal_client_remove_object_with_mod (day_view->client, uid, CALOBJ_MOD_THIS),
+ CAL_COMPONENT_EVENT);
return;
}
@@ -4215,7 +4213,7 @@ e_day_view_on_cut (GtkWidget *widget, gpointer data)
itip_send_comp (CAL_COMPONENT_METHOD_CANCEL, event->comp, day_view->client, NULL);
cal_component_get_uid (event->comp, &uid);
- cal_client_remove_object (day_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (day_view->client, uid), CAL_COMPONENT_EVENT);
}
static void
diff --git a/calendar/gui/e-itip-control.c b/calendar/gui/e-itip-control.c
index b751015d6e..89434d02c8 100644
--- a/calendar/gui/e-itip-control.c
+++ b/calendar/gui/e-itip-control.c
@@ -51,6 +51,7 @@
#include <evolution-shell-client.h>
#include <evolution-folder-selector-button.h>
#include <camel/camel-mime-filter-tohtml.h>
+#include "dialogs/delete-error.h"
#include "calendar-config.h"
#include "itip-utils.h"
#include "e-itip-control.h"
@@ -2036,7 +2037,8 @@ remove_item (EItipControl *itip)
CalComponentVType type;
const char *uid;
GtkWidget *dialog;
-
+ CalClientResult result;
+
priv = itip->priv;
type = cal_component_get_vtype (priv->comp);
@@ -2049,10 +2051,13 @@ remove_item (EItipControl *itip)
return;
cal_component_get_uid (priv->comp, &uid);
- if (cal_client_remove_object (client, uid) == CAL_CLIENT_RESULT_SUCCESS) {
+ result = cal_client_remove_object (client, uid);
+ if (result == CAL_CLIENT_RESULT_SUCCESS || result == CAL_CLIENT_RESULT_NOT_FOUND) {
dialog = gnome_ok_dialog (_("Removal Complete"));
gnome_dialog_run_and_close (GNOME_DIALOG (dialog));
- }
+ } else {
+ delete_error_dialog (result, type);
+ }
}
static void
diff --git a/calendar/gui/e-tasks.c b/calendar/gui/e-tasks.c
index cc247c2d85..38035400da 100644
--- a/calendar/gui/e-tasks.c
+++ b/calendar/gui/e-tasks.c
@@ -32,6 +32,7 @@
#include "e-util/e-url.h"
#include "widgets/menus/gal-view-menus.h"
+#include "dialogs/delete-error.h"
#include "dialogs/task-editor.h"
#include "cal-search-bar.h"
#include "calendar-config.h"
@@ -604,7 +605,7 @@ query_obj_updated_cb (CalQuery *query, const char *uid,
tasks = E_TASKS (data);
priv = tasks->priv;
- cal_client_remove_object (priv->client, uid);
+ delete_error_dialog (cal_client_remove_object (priv->client, uid), CAL_COMPONENT_TODO);
}
/* Callback used when an evaluation error occurs when running a query */
diff --git a/calendar/gui/e-week-view.c b/calendar/gui/e-week-view.c
index 8ab0b44b1c..406a3876d1 100644
--- a/calendar/gui/e-week-view.c
+++ b/calendar/gui/e-week-view.c
@@ -51,6 +51,7 @@
#include <e-util/e-categories-config.h>
#include <e-util/e-dialog-utils.h>
#include "dialogs/delete-comp.h"
+#include "dialogs/delete-error.h"
#include "dialogs/send-comp.h"
#include "dialogs/cancel-comp.h"
#include "dialogs/recur-comp.h"
@@ -1914,7 +1915,7 @@ e_week_view_cut_clipboard (EWeekView *week_view)
e_week_view_copy_clipboard (week_view);
cal_component_get_uid (event->comp, &uid);
- cal_client_remove_object (week_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (week_view->client, uid), CAL_COMPONENT_EVENT);
e_week_view_set_status_message (week_view, NULL);
}
@@ -3924,10 +3925,7 @@ e_week_view_delete_event_internal (EWeekView *week_view, gint event_num)
cal_component_get_uid (event->comp, &uid);
- /* We don't check the return value; FALSE can mean the object
- * was not in the server anyways.
- */
- cal_client_remove_object (week_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (week_view->client, uid), CAL_COMPONENT_EVENT);
}
}
@@ -3973,9 +3971,8 @@ e_week_view_delete_occurrence_internal (EWeekView *week_view, gint event_num)
const char *uid;
cal_component_get_uid (event->comp, &uid);
- if (cal_client_remove_object_with_mod (week_view->client, uid, CALOBJ_MOD_THIS) != CAL_CLIENT_RESULT_SUCCESS)
- g_message ("e_week_view_on_delete_occurrence(): Could not update the object!");
-
+ delete_error_dialog (cal_client_remove_object_with_mod (week_view->client, uid, CALOBJ_MOD_THIS),
+ CAL_COMPONENT_EVENT);
return;
}
@@ -4037,7 +4034,7 @@ e_week_view_on_cut (GtkWidget *widget, gpointer data)
itip_send_comp (CAL_COMPONENT_METHOD_CANCEL, event->comp, week_view->client, NULL);
cal_component_get_uid (event->comp, &uid);
- cal_client_remove_object (week_view->client, uid);
+ delete_error_dialog (cal_client_remove_object (week_view->client, uid), CAL_COMPONENT_EVENT);
}
static void