Differential D9060

Fix kdeconnect-cli device list
ClosedPublic
Actions

Authored by mtijink on Nov 29 2017, 8:24 PM.

Details

Reviewers
apol
Group Reviewers
KDE Connect
Commits
R224:6a6b97e7c9e5: Fix kdeconnect-cli device list
Summary

Previously, the .devices() call passed the parameters in the wrong order. I checked the order in the interface and the daemon, and this was the only place it went wrong.

Now running kdeconnect-cli -l correctly lists all paired devices, even if unreachable.

Test Plan

Tested with both a reachable and unreachable device.

Diff Detail

Repository
R224 KDE Connect
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mtijink requested review of this revision.Nov 29 2017, 8:24 PM
mtijink created this revision.
apol accepted this revision.Nov 30 2017, 2:48 PM
This revision is now accepted and ready to land.Nov 30 2017, 2:48 PM
Closed by commit R224:6a6b97e7c9e5: Fix kdeconnect-cli device list (authored by mtijink, committed by nicolasfella). · Explain WhyDec 3 2017, 6:29 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mcli/kdeconnect-cli.cpp (2 lines)

Diff 23395

View Options

cli/kdeconnect-cli.cpp

Loading...