Calendar formatter: port to PimUniqueApplication::activateApplication
ClosedPublic

Authored by dfaure on Jul 21 2019, 6:06 PM.

Details

Summary

The whole "if korganizer or kontact is running, bring it to the front, otherwise
start korganizer" logic becomes a single line of code.
I'm not 100% sure about the Windows code using QProcess; clearly it
relies on DBus anyway so the new way should work, but it mentions
korganizer-mobile.sh (which could be called by the DBus service file then...)

Test Plan

Click "Check My Calendar" in an invitation email.
In KMail, it starts standalone korganizer.
In Kontact, it switches to the calendar plugin.

Diff Detail

Repository
R81 KDE PIM Addons
Branch
Applications/19.08
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 14223
Build 14241: arc lint + arc unit
dfaure created this revision.Jul 21 2019, 6:06 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptJul 21 2019, 6:06 PM
dfaure requested review of this revision.Jul 21 2019, 6:06 PM
vkrause accepted this revision.Jul 21 2019, 9:43 PM
This revision is now accepted and ready to land.Jul 21 2019, 9:43 PM
This revision was automatically updated to reflect the committed changes.