Differential D9811

[EventDataVisitor] Place events without start date at end date
ClosedPublic
Actions

Authored by broulik on Jan 11 2018, 8:54 AM.

Details

Reviewers
mlaurent
dvratil
Commits
R81:af23ae70e47d: [EventDataVisitor] Place events without start date at end date
Summary

This avoids an infinite loop.
Thanks Dmitry Nezhevenko for investigating.

CHANGELOG: Fixed an issue that would cause Plasma to freeze when there was an agenda item with no start date in the calendar

BUG: 367541

Test Plan

Placed an agenda item without a start date into my calendar. It showed up at the end date and didn't freeze Plasma. (I chose this rather than just discarding the item, so it doesn't magically not show up)

Diff Detail

Repository
R81 KDE PIM Addons
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
broulik created this revision.Jan 11 2018, 8:54 AM
Restricted Application added a project: KDE PIM. · View Herald TranscriptJan 11 2018, 8:54 AM
Restricted Application added a subscriber: KDE PIM. · View Herald Transcript
broulik requested review of this revision.Jan 11 2018, 8:54 AM
dvratil accepted this revision.Jan 11 2018, 12:00 PM
This revision is now accepted and ready to land.Jan 11 2018, 12:00 PM
Closed by commit R81:af23ae70e47d: [EventDataVisitor] Place events without start date at end date (authored by broulik). · Explain WhyJan 11 2018, 12:03 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 25153

View Options

plugins/plasma/pimeventsplugin/eventdatavisitor.cpp

Loading...