suggestions by @marcind

pull/18777/head
Hans-Joachim Kliemeck 9 years ago committed by Matt Clay
parent a240b14e09
commit bdf0a888bb

@ -26,7 +26,7 @@ $result = New-Object PSObject;
Set-Attr $result "changed" $false;
$path = Get-Attr $params "path" -failifempty $true
$copy = Get-Attr $params "copy" "no" -validateSet "no","yes" -resultobj $result
$copy = Get-Attr $params "copy" "no" -validateSet "no","yes" -resultobj $result | ConvertTo-Bool
If (-Not (Test-Path -Path $path)) {
Fail-Json $result "$path file or directory does not exist on the host"
@ -36,7 +36,7 @@ Try {
$objACL = Get-ACL $path
$alreadyDisabled = !$objACL.AreAccessRulesProtected
If ($copy -eq "yes") {
If ($copy) {
$objACL.SetAccessRuleProtection($True, $True)
} Else {
$objACL.SetAccessRuleProtection($True, $False)

@ -27,7 +27,7 @@ module: win_acl_inheritance
version_added: "2.0"
short_description: Disable ACL inheritance
description:
- Disable ACL inheritance and optionally converts ACE to dedicated ACE
- Disable ACL (Access Control List) inheritance and optionally converts ACE (Access Control Entry) to dedicated ACE
options:
path:
description:
@ -48,12 +48,12 @@ EXAMPLES = '''
# Playbook example
---
- name: Disable and copy
win_owner:
win_acl_inheritance:
path: 'C:\\apache\\'
copy: yes
- name: Disable
win_owner:
win_acl_inheritance:
path: 'C:\\apache\\'
copy: no
'''

Loading…
Cancel
Save