Bug 339824

Summary: Creating Event from message, adjusting start time can leave end time before start time
Product: [Applications] kmail2 Reporter: Sander van Grieken <sander>
Component: message listAssignee: kdepim bugs <kdepim-bugs>
Status: RESOLVED FIXED    
Severity: normal CC: aacid, montel
Priority: NOR    
Version: 4.14.0   
Target Milestone: ---   
Platform: Debian testing   
OS: Linux   
Latest Commit: http://commits.kde.org/kdepim/918b91cdf0a2dffffe1348387ffaa3b4e95cc4e9 Version Fixed In: 4.14.2
Sentry Crash Report:

Description Sander van Grieken 2014-10-09 17:46:21 UTC 
When I rightclick a message and select Create Event.. , then adjust the start time forward, the end time is not changed so it not before the start time.

Most convenient behaviour would be to make the end time equal, or one hour after the new start time



Reproducible: Always

Steps to Reproduce:
1. Select message
2. rightclick, click Create Event...
3. Adjust start time forward
4. Observe end time

Actual Results:  
end time stays what it was

Expected Results:  
end time is not adjusted to be at least start time

nice to have : have a warning popup if the user manually puts the end time before the start time
Comment 1 Laurent Montel 2014-10-10 05:51:57 UTC 
Git commit 918b91cdf0a2dffffe1348387ffaa3b4e95cc4e9 by Montel Laurent.
Committed on 10/10/2014 at 05:51.
Pushed by mlaurent into branch 'KDE/4.14'.

Fix Bug 339824 - Creating Event from message, adjusting start time can leave end time before start time

FIXED-IN: 4.14.3

M  +39   -0    messageviewer/tests/eventedittest.cpp
M  +1    -0    messageviewer/tests/eventedittest.h
M  +3    -1    messageviewer/widgets/eventedit.cpp

http://commits.kde.org/kdepim/918b91cdf0a2dffffe1348387ffaa3b4e95cc4e9
Comment 2 Albert Astals Cid 2014-10-10 07:41:08 UTC 
I had to retag early this morning so this will be in 4.14.2
Comment 3 Laurent Montel 2014-10-10 07:54:33 UTC 
Albert ok great.
if you can create after 10h00 am it will good
i fixed 3 others bugs :)
Thanks a lot.
Comment 4 Laurent Montel 2014-10-10 07:55:20 UTC 
"nice to have : have a warning popup if the user manually puts the end time before the start time" I will create it for KF5.
I add to my todo.
Comment 5 Sander van Grieken 2014-10-10 07:58:35 UTC 
Double awesome! Fixed within 12 hours and right before tagging the next point release :)