After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 489437 - start date of recurrent events can be later than end date
start date of recurrent events can be later than end date
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Calendar
unspecified
Other All
: High major
: ---
Assigned To: Milan Crha
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2007-10-23 15:00 UTC by Wang Xin
Modified: 2009-01-13 12:25 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
proposed evo patch (1.77 KB, patch)
2008-09-25 13:28 UTC, Milan Crha 		reviewed Details | Review

Description Wang Xin 2007-10-23 15:00:32 UTC 
Steps to Reproduce:
1,create an appointment with such info: "start time:09/30/2004 03pm
  end   time:09/30/2004 04pm".
2,Select "Recurrence" button
3,Check "This appointment recurs"
4,Let "Every"=(1 days),"until"= "08/01/2004"

Actural Result:
    The appointment will be added to the calendar without warning.
Expected Result:
    It is expected to have system warning to reminder user the end date is earlier than start date.
Comment 1 Milan Crha 2008-09-25 13:28:34 UTC 
Created attachment 119367 [details] [review]
proposed evo patch

for evolution;

The error is considered critical, user should fix his/her input (aka it's not just a warning). Feel free to suggest better error text.

There is one thing I'm not sure about, the timezones. Any idea?
Comment 2 Wang Xin 2009-01-13 05:19:36 UTC 
Can anyone review this patch?
Comment 3 Chenthill P 2009-01-13 07:12:17 UTC 
Tested the patch and found a couple of issues. The validation error keeps on appearing multiple times even after unchecking the "The appointment recurs" check box. I dont know the exact difference between GTK_WIDGET_SENSITIVE() and GTK_WIDGET_IS_SENSITIVE(). But have the latter seems to fix the issue. 
   
It would be better to reset the recur. ending date to start date of the event while displaying the validation error.

Please make these changes and commit the patch.
Comment 4 Milan Crha 2009-01-13 12:25:56 UTC 
Slightly modified committed to trunk. Committed revision 37066.

I changed both above mentioned things.