Bug 177196 - advanced reminder editor: reminder repeat interval not saved
Summary: advanced reminder editor: reminder repeat interval not saved
Status: RESOLVED FIXED
Alias: None
Product: korganizer
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-12-08 05:02 UTC by Szczepan Hołyszewski
Modified: 2009-02-27 13:48 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
Proposed patch (514 bytes, patch)
2009-02-24 16:31 UTC, Lars
Details

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