Bug 169146

Summary:	Applying a template doesn't manage start/end time
Product:	[Applications] korganizer	Reporter:	FiNeX <finex>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	wstephenson
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	example patch

Description FiNeX 2008-08-15 00:51:36 UTC

Version:            (using Devel)
Installed from:    Compiled sources

1) create new event
2) enable "time associated"
3) set the start and end time
4) make it a new template by going to "manage templates" and pressing "new"
5) make another event and apply this template

it will enable "time associated" for you, but it won't set the times.

Thanks to Doviende 
http://techbase.kde.org/Contribute/Bugsquad/BugDays/KDEPIMKrush1/KOrganizer#Bugs

Comment 1 FiNeX 2008-08-15 00:53:54 UTC

Created attachment 26854 [details]
example patch

This patch allow to apply the start/end time from the saved template (if set).

If you (korganizer devs) agree, I can commit the patch too.

Goodbye :-)

Comment 2 Allen Winter 2008-08-15 02:01:35 UTC

Thanks for the patch FiNeX.  Please commit.

Comment 3 FiNeX 2008-08-15 09:52:44 UTC

SVN commit 847268 by finex:

* When a template is applied, it will use the start/end date 
(if present).
* Apply a template with double click on template list.

BUG: 169146
BUG: 123331



 M  +26 -0     koeditorgeneralevent.cpp  
 M  +2 -0      koeditorgeneralevent.h  
 M  +2 -0      templatemanagementdialog.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=847268

Comment 4 Will Stephenson 2008-12-07 09:16:45 UTC

I have a report at https://bugzilla.novell.com/show_bug.cgi?id=443130 that misses the original behaviour.  What do you think of adding another Apply button, so we have "Apply template, keeping times" and "Apply template (move to STARTTIME)"?

Comment 5 Allen Winter 2009-05-25 13:48:06 UTC

SVN commit 972592 by winterz:

Do not apply start or end times when applying an Event template unless we have no other choice.

Fixes 190545, reverts the original fix for 169146, and should also fix novell bug443130

BUGS: 169146,190545


 M  +4 -2      koeditorgeneralevent.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=972592