Bug 308932 - Cannot edit properties of an existing event
Summary: Cannot edit properties of an existing event
Status: RESOLVED FIXED
Alias: None
Product: korganizer
Classification: Applications
Component: general (show other bugs)
Version: 4.9.2
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-24 14:44 UTC by Unknown
Modified: 2012-10-29 23:32 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 4.9.3


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Unknown 2012-10-24 14:44:38 UTC
After I created an event in KOrganizer I cannot edit its properties, eg. its stating time.
Interestingly, the Apply button is disabled but the Ok button is not, so if I don't re-check the modifications it seems that KOrg could modify the resource successfully.

Reproducible: Always

Steps to Reproduce:
1. Create an event on a CalDAV server (I use Radicale)
2. Add an event to KOrganizer and wait for a sync
3. Modify its starting time to an hour later
4. Press Ok and wait for a sync
5. Recheck
Comment 1 Sergio Martins 2012-10-28 02:42:51 UTC
Are you changing the start time with your keyboard.
Can you try with mouse ?
Comment 2 Unknown 2012-10-28 07:37:08 UTC
Wow, that worked (with mouse). But unfortunately just for the first time.
As soon as I wanted to change it again, it says: "Unable to store the incidence in the calendar. Try again? Reason: NO Only resources can modify remote revisions" (what does the "NO" mean here?).
I can reproduce it with other events, too.
Comment 3 Sergio Martins 2012-10-29 22:30:00 UTC
Git commit 7262922919c383b07d2f1a28d7fbba19b5f9b692 by Sergio Martins.
Committed on 29/10/2012 at 23:23.
Pushed by smartins into branch 'KDE/4.9'.

Fix change detection when using keyboard digits.

KTimeComboBox::timeChanged() is only emitted when using mouse
or when using up/down arrow.

When changing the time with other keys than up/down, like the
[0-9] keys, timeEdited() is emitted instead.

Not sure I understand the difference between the changed and edited
concepts. Might be a bug?

CCMAIL: john@layt.net

M  +7    -1    incidenceeditor-ng/incidencedatetime.cpp

http://commits.kde.org/kdepim/7262922919c383b07d2f1a28d7fbba19b5f9b692