Bug 177196

Summary:	advanced reminder editor: reminder repeat interval not saved
Product:	[Applications] korganizer	Reporter:	Szczepan Hołyszewski <mikid>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	normal	CC:	daniel.dumitrache, finex, larsnospam, listes.rusconi
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Attachments:	Proposed patch

Description Szczepan Hołyszewski 2008-12-08 05:02:38 UTC

Version:           4.1.0 (using KDE 4.1.3)
OS:                Linux
Installed from:    Unlisted Binary Package

Reproduce:

Create an event.
Open the advanced reminders editor.
Add a reminder.
Enable reminder repeating.
Edit repeat interval to something other than 1 minute.
Apply.
Close the advanced reminders editor.
Reopen the advanced reminders editor.
Select the reminder you created previously.

Problem: the repeat interval is still at 1 minute.

Expected: should retain the modified value.

Comment 1 FiNeX 2008-12-08 15:31:28 UTC

Confirmed in current trunk too: the repeat interval is not set.

Comment 2 Daniel Dumitrache 2009-01-30 01:04:42 UTC

still present in kde 4.2.0

Comment 3 rusconi 2009-02-04 10:33:38 UTC

I do also suffer from this bug :

Version 4.3.0 pre
Using KDE 4.2.60 (KDE 4.2.60 (KDE 4.3 >= 20090116))

Comment 4 Lars 2009-02-24 16:29:35 UTC

Real nasty bug.

In fact it IS saved, but the value is interpreted as seconds, rather than minutes.

So to help yourself you could put in a value of seconds to the inputbox. Say, eg. "120" will set the repeat interval to 2 minutes - until next editing of that event at least ;)

Comment 5 Lars 2009-02-24 16:31:17 UTC

Created attachment 31610 [details]
Proposed patch

A patch to solve this issue.

Comment 6 Sergio Martins 2009-02-27 13:48:15 UTC

SVN commit 932770 by smartins:

Alarm interval was converted to minutes when read, but not converted back to seconds when written.
Thanks to Lars who spotted this.

BUG:177196
CCBUG:140551
CCBUG:141283
CCBUG:144683
CCBUG:177929


 M  +2 -2      koeditoralarms.cpp  


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