Add privilegeExecution field to file protocol description
AcceptedPublic

Authored by chinmoyr on Aug 26 2017, 5:45 PM.

Details

Reviewers
dfaure
Group Reviewers
Frameworks
Maniphest Tasks
T6561: Polkit support in KIO
Summary

"privilegeExecution" field will specify if protocol can perform file operation with elevated privileges.

Dolphin relies on KFileItemListProperties for enabling some of its context menu actions. Internally KFileItemListProperties uses KProtocolManager to determine if a location supports reading, writing,
deleting and moving. With these changes KProtocolManager will now check for the said field. Consequently, dolphin will be aware of our new file protocol and will show its context menu without any
action being disabled.

Diff Detail

Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
chinmoyr created this revision.Aug 26 2017, 5:45 PM
dfaure accepted this revision.Sep 3 2017, 8:09 AM
dfaure added inline comments.
src/core/kprotocolmanager.h
479

Missing @since 5.39

This revision is now accepted and ready to land.Sep 3 2017, 8:09 AM

What's the status of this patch?

What's the status of this patch?

I will push this patch after fixing all security issues with kauth support in KIO. (T8075)

mati865 added a subscriber: mati865.Jun 3 2018, 1:52 PM
mreeves added a subscriber: mreeves.Jun 4 2018, 3:47 AM