Version: unspecified (using Devel) OS: Linux When a user tries to change permissions on a folder they don't have access to change with the advanced POSIX ACL editor and hit OK, Dolphin silently fails, instead of telling the user that they don't have access to the folder. Reproducible: Always OS: Linux (i686) release 2.6.35-6-generic Compiler: cc
It does not only fail silently, the UI pretends it were possible to change the ACLs on those files/directories. All options inside the "Advanced Permissions" dialog should be grayed out, including the labels for the "Set UID", "Set GID" and "Sticky" checkboxes (the checkboxes themselves are, but not the labels) as well as the ACL list and the add/edit/delete buttons on the right side. I consider this a major usability issue.
(From the usability team): I agree that this is a big usability problem and I agree with the solution suggested by Janek. This definitely has to be fixed.
still present in 4.14
Still present in 15.08.2
Git commit 549fdb643b171952b5101ae0c7b9f80f361f3845 by David Edmundson. Committed on 07/04/2016 at 23:44. Pushed by davidedmundson into branch 'master'. Don't enable acl permissions widget if we don't own the file The man page for setfacl says only the file owner or root can change the permissions, so it would always fail. Better to disable it in the UI. REVIEW: 127414 M +1 -0 src/widgets/kpropertiesdialog.cpp http://commits.kde.org/kio/549fdb643b171952b5101ae0c7b9f80f361f3845
Git commit 36d80144ab9acf078deb0379e6aff3a912f5d3da by David Edmundson. Committed on 07/04/2016 at 23:44. Pushed by davidedmundson into branch 'master'. Set text on QCheckbox widget rather than using a separate label This fixes (amongst other focus changes) the label having the right enabled effect if the permissions are not editable. REVIEW: 127418 M +18 -33 src/widgets/kpropertiesdialog.cpp http://commits.kde.org/kio/36d80144ab9acf078deb0379e6aff3a912f5d3da