[kfile] [Bug 245580] Dolphin advanced ACL editor silently fails if user does not have permission / options not grayed out

[David Edmundson via KDE Bugzilla]

https://bugs.kde.org/show_bug.cgi?id=245580

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|CONFIRMED                   |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/kio/
                   |                            |549fdb643b171952b5101ae0c7b
                   |                            |9f80f361f3845

--- Comment #5 from David Edmundson <k...@davidedmundson.co.uk> ---
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

--- Comment #6 from David Edmundson <k...@davidedmundson.co.uk> ---
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

-- 
You are receiving this mail because:
You are watching all bug changes.