D6198: Add KAuth support to delete operation

[Chinmoy Ranjan Pradhan]

chinmoyr added a comment.


  > If there was a malicous agent hiding in a dolphin plugin, it would just be making a direct call to the helper and skip all that.
  
  Till now I was not aware of what a dolphin plugin can or cannot do. I assumed there would be some kind of restriction. I was wrong , my bad. 
  Now the ioslave can also show a warning but right now it does so only for a particular case, i.e, when only some of the items being deleted are in read-only location but for deletion in a read-only folder we rely on the warning from jobuidelegate. So, right now the only thing I find sensible to do is to make ioslave show warning for both the cases above. By removing persistence the user will have to enter password every time he deletes something which is somewhat annoying. Getting a warning everytime is annoying as well but it is only matter of a click. So to some extent it is like the sudo command. I would also like to point out that my plan, right from the start, was to use the warning from ioslave in 'copy' and 'del' method. As for the rest, mkdir, symlink, rename etc, I was to rely on the warning from the client. I believe I may have to rethink my approach. What will you suggest?

REPOSITORY
  R241 KIO

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

To: chinmoyr, elvisangelaccio, #frameworks
Cc: davidedmundson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170623/e2de9102/attachment.html>