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 248105 - No warning if you create an appointment in the past
No warning if you create an appointment in the past
Status: RESOLVED FIXED
Product: evolution
Classification: Applications
Component: Calendar
2.28.x (obsolete)
Other All
: Normal minor
: ---
Assigned To: evolution-calendar-maintainers
Evolution QA team
: 234182 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2003-09-03 16:53 UTC by Breda McColgan
Modified: 2010-05-27 15:29 UTC
See Also:
GNOME target: ---
GNOME version: 2.27/2.28


Attachments
prposed evo patch (6.14 KB, patch)
2010-03-15 17:16 UTC, Milan Crha
reviewed Details | Review
evo patch (11.10 KB, patch)
2010-05-27 15:23 UTC, Milan Crha
committed Details | Review

Description Breda McColgan 2003-09-03 16:53:40 UTC
Description of Problem:
Calendar does not display a warning if you create
an appointment in the past.

Steps to reproduce the problem:
1. Invoke Calendar.
2. Select a past date in the date pane.
3. Create an appointment. 

Actual Results:
The appointment is created with no warning.


Expected Results:
Should display a warning and require confirmation
to create an appointment in the past.


How often does this happen? 
Every time.

Additional Information:
Evolution 1.4.
Comment 1 André Klapper 2004-12-10 12:22:28 UTC
*** bug 234182 has been marked as a duplicate of this bug. ***
Comment 2 André Klapper 2004-12-10 12:23:01 UTC
adding version number and target milestone.
Comment 3 Karthik.V 2005-07-11 15:45:05 UTC
i ll start working on it
Comment 4 Karthik.V 2005-07-11 15:58:22 UTC
i ll start working on it
Comment 5 anuj dixit 2005-09-15 16:39:19 UTC
i am working on it
Comment 6 Matthew Barnes 2008-03-11 00:28:32 UTC
Bumping version to a stable release.
Comment 7 Milan Crha 2009-06-30 15:51:12 UTC
It seems nobody is working on this. Just to summarize:
a) for events (appointments/meetings)
b) when saving from an editor
c) also when creating inline events (in a view or table)
d) not when drag&drop events

Questions:
e) With a text: "You are going to save an event which starts in the past.
   Are you sure you want to continue?"

f) For new events only or when changing the old event too?

This reminded me of bug #223893, maybe this one is part of it, isn't it?
Comment 8 Akhil Laddha 2009-11-27 04:55:17 UTC
(In reply to comment #7)
> It seems nobody is working on this. Just to summarize:
> a) for events (appointments/meetings)
> b) when saving from an editor
> c) also when creating inline events (in a view or table)
> d) not when drag&drop events
> 
> Questions:
> e) With a text: "You are going to save an event which starts in the past.
>    Are you sure you want to continue?"
> 

This string looks fine to me

> f) For new events only or when changing the old event too?
> 

IMHO, it should be displayed only for the new events.

> This reminded me of bug #223893, maybe this one is part of it, isn't it?
Comment 9 Milan Crha 2010-03-15 17:16:04 UTC
Created attachment 156198 [details] [review]
prposed evo patch

for evolution;

Text changed, because a need for an error definition and breaking it into primary and secondary texts. Otherwise should work.

The other approach I noticed in Outlook is about showing a notification at the top of the window, like with "you are not an organizer" in evo, telling the start time is in the past. Maybe it's less intrusive, but more likely to overlook?
Comment 10 Chenthill P 2010-05-19 11:19:42 UTC
Review of attachment 156198 [details] [review]:

I would like to go with the second approach especially since its less intrusive.
Comment 11 Milan Crha 2010-05-27 15:23:36 UTC
Created attachment 162114 [details] [review]
evo patch

for evolution;

OK, rewritten to update info at the top of the editor page.
Comment 12 Milan Crha 2010-05-27 15:29:04 UTC
Created commit c98ff48 in evo master (2.31.3+)