Page MenuHomePhorge

When editing a recurring incidence also ask what exactly the user wants to do.
ClosedPublic

Authored by knauss on Apr 27 2016, 3:08 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Mar 16, 10:29 AM
Unknown Object (File)
Thu, Mar 14, 7:57 PM
Unknown Object (File)
Sat, Mar 2, 4:47 PM
Unknown Object (File)
Sat, Mar 2, 4:47 PM
Unknown Object (File)
Sat, Mar 2, 4:47 PM
Unknown Object (File)
Sat, Mar 2, 4:47 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Unknown Object (File)
Sat, Mar 2, 4:46 PM
Subscribers
None

Details

Summary

A recurrenting event can be edited differntly (only this occurence, all occurences, thisandfuture).
We need to ask the user, what he wants to edit exactly.

Diff Detail

Repository
rKP kdepim
Branch
kolab/integration/4.13.0
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 3479
Build 3454: arc lint + arc unit

Event Timeline

knauss retitled this revision from to When editing a recurring incidence also ask what exactly the user wants to do..
knauss updated this object.
knauss edited the test plan for this revision. (Show Details)
knauss added a reviewer: mollekopf.
mollekopf edited edge metadata.
mollekopf added inline comments.
korganizer/koeventpopupmenu.cpp
149

This seems to break isLastOccurrence just below? We're trying to figure out if qd is the last occurrence by getting the next one, with your change you just get the same occurrence again?
So while the code looks correct to get an occurrence on the date specified by qd, I think you need to do:

const KDateTime thisDateTime = inc->recurrence->getNextDateTime(qd.addDays(-1), QTime(23,59,59), CalendarSupport::KCalPrefs::instance()->timeSpec() ));

On the first line so you end up calling getNextDateTime twice for isLastOccurrence and isFirstOccurrence.

This revision now requires changes to proceed.Apr 28 2016, 1:02 PM
knauss edited edge metadata.

Fix the issue raised by christian.

korganizer/koeventpopupmenu.cpp
149

Well in this case it does not matter much because we only test !( isFirstOccurrence && isLastOccurrence ), that is only true for an item that is not a recurrenting one. But yes better fix the code, because it will be copied around :D

knauss edited edge metadata.

no need to modify showIncidencePopup, because it was correct before.

mollekopf edited edge metadata.
This revision is now accepted and ready to land.Apr 28 2016, 11:57 PM
This revision was automatically updated to reflect the committed changes.