Bug 169146 - Applying a template doesn't manage start/end time
Summary: Applying a template doesn't manage start/end time
Status: RESOLVED FIXED
Alias: None
Product: korganizer
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-15 00:51 UTC by FiNeX
Modified: 2009-05-25 13:48 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
example patch (1.95 KB, patch)
2008-08-15 00:53 UTC, FiNeX
Details

Note You need to log in before you can comment on or make changes to this bug.
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