D5355: Fix query for available modules

2017-04-10 Thread Albert Astals Cid
This revision was automatically updated to reflect the committed changes. Closed by commit R126:19a6bb179d5f: Fix query for available modules (authored by aacid). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D5355?vs=13238=13290#toc REPOSITORY R126 KDE CLI Utilities CHANGES SINCE

D5355: Fix query for available modules

2017-04-09 Thread Aleix Pol Gonzalez
apol accepted this revision. apol added inline comments. This revision is now accepted and ready to land. INLINE COMMENTS > main.cpp:62 > +// We need the exist calls because otherwise the trader language aborts > if the property doesn't exist and the second part of the or is not evaluated >

D5355: Fix query for available modules

2017-04-08 Thread Albert Astals Cid
aacid created this revision. Restricted Application added a project: Plasma. Restricted Application added a subscriber: plasma-devel. REVISION SUMMARY The old query was bad because two reasons: - it didn't use the same query systemsettings uses fo - it didn't use exist so if the first