Re: [evolution-patches] [calendar] patch for #57142

[JP Rosevear <jpr novell com>]

On Thu, 2004-07-15 at 19:52 +0200, Rodrigo Moya wrote:
> On Thu, 2004-07-15 at 14:35 +0200, Rodrigo Moya wrote:
> > Updated patch for this, which makes ECalComponent use the same mechanism
> > we use in the event editor to keep track of alarms that need a
> > description. Whenever we change the component's summary, all the alarms'
> > descriptions are updated if needed.
> 
> updated patch after some talk with JP on IRC, now including an evolution
> part.

Looks good except I think the case where the summary changes and we have
a custom alarm description is not covered - set_alarm_description_cb
should make sure the previous summary and the current alarm description
are the same (I believe the event editor code does this) before updating
the description.

-JP
-- 
JP Rosevear <jpr novell com>
Novell, Inc.