KDesktopFileActions: port from KRun::run to KIO::ApplicationLauncherJob
ClosedPublic

Authored by dfaure on Mar 25 2020, 12:29 AM.

Details

Summary

Remove KService::Ptr parameter from ApplicationLauncherJob, the
KServiceAction is now sufficient.

Test Plan

dolphin, RMB, "Checkout from a repository..." (requires kdesvn) still works

Diff Detail

Repository
R241 KIO
Branch
kdesktopfileactions
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 24362
Build 24380: arc lint + arc unit
dfaure requested review of this revision.Mar 25 2020, 12:29 AM
dfaure created this revision.
apol accepted this revision.Mar 25 2020, 3:42 AM

Change looks good.

The advantage here is that it's not blocking, right?

This revision is now accepted and ready to land.Mar 25 2020, 3:42 AM

Exactly.

The downside is that it's more verbose, but the benefits are that it's non-blocking, and that the job class is in KioGui (no QtWidgets dependency). Doesn't matter for this code here, but it'll be good for QML apps who don't want dialog boxes on error.

(and it's better API because we can add setters, instead of ending up with the 4 overloads in krun.h due to adding parameters one at a time...)

dfaure updated this revision to Diff 78710.Mar 27 2020, 10:42 PM
dfaure edited the summary of this revision. (Show Details)

Rebase on D28367, remove KService::Ptr arg

Restricted Application added a project: Frameworks. · View Herald TranscriptMar 27 2020, 10:42 PM
Restricted Application added a subscriber: kde-frameworks-devel. · View Herald Transcript
broulik accepted this revision.Mar 29 2020, 10:32 AM

Much nicer :)

dfaure closed this revision.Mar 29 2020, 11:10 AM