[Differential] [Request, 119 lines] D4573: [kioslave/remote] Fix broken kded module

Elvis Angelaccio noreply at phabricator.kde.org
Sat Feb 11 17:36:13 UTC 2017


elvisangelaccio created this revision.
elvisangelaccio added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Commit https://phabricator.kde.org/R120:72e39f083a103e0887624482d6a0ce8b0556dd5e broke the kded module. Turns out that the old code
  just forced a refresh of the `remote:/` QUrl, so we can just do that
  explicitly and remove all the old cruft.
  
  While at it, use a KDirWatcher that just watches the
  `.local/share/remote/` folder, instead of checking *every* unrelated
  QUrl emitted by KDirNotify.

TEST PLAN
  Adding/deleting/editing .desktop files in `.local/share/remoteview` now
  automatically reloads the `remote:/` url in Dolphin or Folder View.

REPOSITORY
  R120 Plasma Workspace

BRANCH
  Plasma/5.9

REVISION DETAIL
  https://phabricator.kde.org/D4573

AFFECTED FILES
  kioslave/remote/kdedmodule/remotedirnotify.cpp
  kioslave/remote/kdedmodule/remotedirnotify.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: elvisangelaccio, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170211/30f715a6/attachment.html>


More information about the Plasma-devel mailing list