suggestions by @marcind

This commit is contained in:
Hans-Joachim Kliemeck 2015-10-21 21:20:44 +02:00
parent 95862793d0
commit 8de49a5dea
2 changed files with 5 additions and 5 deletions

View file

@ -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)

View file

@ -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
'''