Re: [evolution-patches] Patch for #59335

[Rodrigo Moya <rodrigo novell com>]

On Tue, 2004-07-27 at 17:24 +0530, Harish Krishnaswamy wrote:
> hi,
> The following is a patch for the #59335 - the issue was that evolution
> requires a refresh (by switching b/w the dates) before the events show
> up
> on the view. This was happening because the signal handlers in the e-
> cal-model were disconnected even before the calendar is fully loaded.
> This one-line patch ensures that they are removed only after the
> calendar is loaded.

> Index: calendar/gui/e-cal-model.c
> ===================================================================
> RCS file: /cvs/gnome/evolution/calendar/gui/e-cal-model.c,v
> retrieving revision 1.40
> diff -u -p -r1.40 e-cal-model.c
> --- calendar/gui/e-cal-model.c 24 Jul 2004 14:28:59 -0000 1.40
> +++ calendar/gui/e-cal-model.c 27 Jul 2004 11:44:16 -0000
> @@ -1445,6 +1445,8 @@ cal_opened_cb (ECal *client, ECalendarSt
> return;
> }
> 
> + if (e_cal_get_load_state (client)!= E_CAL_LOAD_LOADED)
> + return;
> /* Stop listening for this calendar to be opened */
> g_signal_handlers_disconnect_matched (client, G_SIGNAL_MATCH_FUNC |
> G_SIGNAL_MATCH_DATA, 0, 0, NULL, cal_opened_cb, model);
> 
why would cal_opened_cb be called if the calendar has not been opened
yet?

cheers