From c185a1b39f596d9fb8acb2e1deade43db8f533b0 Mon Sep 17 00:00:00 2001 From: Christopher James Lahey Date: Wed, 11 Apr 2001 04:33:22 +0000 Subject: Fix this to just send the data to the wombat instead of inserting it 2001-04-11 Christopher James Lahey * gui/calendar-model.c (calendar_model_append_row): Fix this to just send the data to the wombat instead of inserting it ourselves. svn path=/trunk/; revision=9223 --- calendar/ChangeLog | 6 ++++ calendar/gui/calendar-model.c | 64 ++++++++++++++----------------------------- 2 files changed, 26 insertions(+), 44 deletions(-) diff --git a/calendar/ChangeLog b/calendar/ChangeLog index afea681130..1c31d992b5 100644 --- a/calendar/ChangeLog +++ b/calendar/ChangeLog @@ -1,3 +1,9 @@ +2001-04-11 Christopher James Lahey + + * gui/calendar-model.c (calendar_model_append_row): Fix this to + just send the data to the wombat instead of inserting it + ourselves. + 2001-04-11 Gediminas Paulauskas Display fixes, thanks to Kjartan for finding these. diff --git a/calendar/gui/calendar-model.c b/calendar/gui/calendar-model.c index 9f4f4b2217..f75499f4da 100644 --- a/calendar/gui/calendar-model.c +++ b/calendar/gui/calendar-model.c @@ -41,11 +41,11 @@ #include #include #include - -#include "e-util/e-time-utils.h" -#include "cal-util/timeutil.h" -#include "calendar-commands.h" +#include +#include #include "calendar-model.h" +#include "calendar-commands.h" + /* Private part of the ECalendarModel structure */ struct _CalendarModelPrivate { @@ -67,9 +67,6 @@ struct _CalendarModelPrivate { /* Whether we display dates in 24-hour format. */ gboolean use_24_hour_format; - /* HACK: so that ETable can do its stupid append_row() thing */ - guint appending_row : 1; - /* The default category to use when creating new tasks, e.g. when the filter is set to a certain category we use that category when creating a new task. */ @@ -1286,12 +1283,6 @@ calendar_model_set_value_at (ETableModel *etm, int col, int row, const void *val break; } - /* FIXME: this is an ugly HACK. ETable needs a better API for the - * "click here to add an element" thingy. - */ - if (priv->appending_row) - return; - if (!cal_client_update_object (priv->client, comp)) g_message ("calendar_model_set_value_at(): Could not update the object!"); } @@ -1340,51 +1331,36 @@ calendar_model_append_row (ETableModel *etm, ETableModel *source, gint row) CalendarModel *model; CalendarModelPrivate *priv; CalComponent *comp; - int *new_idx, col; - const char *uid; model = CALENDAR_MODEL (etm); priv = model->priv; - /* This is a HACK */ - priv->appending_row = TRUE; - /* FIXME: This should support other types of components, but for now it * is only used for the task list. */ comp = cal_component_new (); cal_component_set_new_vtype (comp, priv->new_comp_vtype); - cal_component_get_uid (comp, &uid); - - g_array_append_val (priv->objects, comp); - new_idx = g_new (int, 1); - *new_idx = priv->objects->len - 1; - g_hash_table_insert (priv->uid_index_hash, (char *) uid, new_idx); - - /* Notify the views about the new row. I think we have to do that here, - or the views may become confused when they start getting - "row_changed" or "cell_changed" signals for this new row. */ - e_table_model_row_inserted (etm, *new_idx); - - for (col = 0; col < CAL_COMPONENT_FIELD_NUM_FIELDS; col++) { - const void *val; - - if (!e_table_model_is_cell_editable (etm, col, *new_idx)) - continue; - - val = e_table_model_value_at(source, col, row); - e_table_model_set_value_at (etm, col, *new_idx, val); - } - - /* This is the end of the HACK */ - priv->appending_row = FALSE; + set_categories (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_CATEGORIES, row)); + set_classification (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_CLASSIFICATION, row)); + set_completed (model, comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_COMPLETED, row)); + /* FIXME: Need to reset dtstart if dtend happens before it */ + set_datetime (model, comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_DTEND, row), cal_component_set_dtend); + /* FIXME: Need to reset dtend if dtstart happens after it */ + set_datetime (model, comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_DTSTART, row), cal_component_set_dtstart); + set_datetime (model, comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_DUE, row), cal_component_set_due); + set_geo (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_GEO, row)); + set_percent (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_PERCENT, row)); + set_priority (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_PRIORITY, row)); + set_summary (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_SUMMARY, row)); + set_transparency (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_TRANSPARENCY, row)); + set_url (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_URL, row)); + set_complete (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_COMPLETE, row)); + set_status (comp, e_table_model_value_at(source, CAL_COMPONENT_FIELD_STATUS, row)); if (!cal_client_update_object (priv->client, comp)) { /* FIXME: Show error dialog. */ g_message ("calendar_model_append_row(): Could not add new object!"); - remove_object (model, uid); - e_table_model_row_deleted (etm, *new_idx); } } -- cgit v1.2.3