Re: [evolution-patches] A patch for calendar based on trunk

[JP Rosevear <jpr ximian com>]

On Tue, 2004-04-27 at 17:03 +0800, Kidd Wang wrote:
> Thanks. I've checked in the patch.
> 
> Rodrigo Moya wrote:
> > On Mon, 2004-04-26 at 18:08 +0800, Kidd Wang wrote:
> >   
> > > Hi,
> > > 
> > > This patch is aimed for trunk. We have found a bug in calendar which can 
> > > be reproduced as follows:
> > > 1. Invoke your evolution
> > > 2. Create a new task
> > > 3. Set the start time of the new task
> > > 
> > > As you can see, the setting of step 3 is not allowed. This is because it 
> > > is checked whether the due time
> > > is before the start time of a task whenever the start time or the due 
> > > time is set. However, the due time is
> > > null when we create a new task. Therefore the check will always fail.
> > > 
> > > Enclosed is a patch to fix the problem. Would you like to spend a little 
> > > time to review it?

I'd actually gotten rid of this check entirely in my local tree (I'd
already yanked the check in the task editor).  There is no start you
can't start a task after its due.

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