Bug 237822

Summary:	Using "edit" button on text popup reminder window to change message or date doesn't save.
Product:	[Applications] kalarm	Reporter:	Cameron D. McIntosh <yateendraji>
Component:	general	Assignee:	David Jarvie <djarvie>
Status:	CLOSED FIXED
Severity:	normal	CC:	yateendraji
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:	KDE 4.4.4, KAlarm 2.4.6
Attachments:	Test calendar file (created using the version of KAlarm in question)

Description Cameron D. McIntosh 2010-05-16 14:29:18 UTC

Version:           2.4.4 (using 4.4.2 (KDE 4.4.2), Kubuntu packages)
Compiler:          cc
OS:                Linux (x86_64) release 2.6.32-22-generic

To reproduce bug (instructions here are specifically as used by author):
(0) Create new calendar (bug originally discovered with upgraded-converted calendar)
(1) Create new reminder text popup specifying clock time (or using "time difference" feature to make alarm pop up in 1 minute). Most convenient to add recurrence interval (I used "daily").
(2) Wait for popup window
(3) Press "Edit" button
(4) Modify date (with or without modifying popup text). Easiest to use "time difference" to defer popup by one minute.
(5) Notice that neither text nor time is changed in the main KAlarm window.
(6) In contrast, edits performed from the main KAlarm window itself *are* correctly saved.
Additional notes:
(1) This is a clean operating system install (Kubuntu Lucid amd64). KAlarm version used was as called by KPackageKit, from normal repositories. OS seems to be working fine in all other respects.
(2) Permissions on the "calendar.ics" file seem correct (refer to user, not "root").

Comment 1 Cameron D. McIntosh 2010-05-16 14:34:22 UTC

Created attachment 43632 [details]
Test calendar file (created using the version of KAlarm in question)

Comment 2 David Jarvie 2010-05-17 11:26:46 UTC

It turns out that the fix for bug 213461 has stopped the edit function working - the code to process the OK button is now never reached. I hope to have a fix for this very soon.

Comment 3 David Jarvie 2010-05-20 00:15:52 UTC

This is now fixed for KDE 4.4.4 (SVN commits 1128669, 1128670).