win_owner: Clean up and check-mode support (#21385)

* win_owner: Clean up and check-mode support

Changes include:
- Use Get-AnsibleParam with -type/-validateset
- Replace $result PSObject with normal hash
- Add check-mode support

* Implemented -WhatIf:$check_mode support

* Revert to original formatting

lib/ansible/modules/windows/win_owner.ps1

@@ -67,7 +67,7 @@ Function UserSearch
             $Searcher.Filter = "userPrincipalName=$($accountName)"
         }
 
-        $result = $Searcher.FindOne() 
+        $result = $Searcher.FindOne()
         if ($result)
         {
             $user = $result.GetDirectoryEntry()
@@ -80,16 +80,17 @@ Function UserSearch
         }
     }
 }
- 
-$params = Parse-Args $args;
 
-$result = New-Object PSObject;
+$result = @{
-Set-Attr $result "changed" $false;
+    changed = $false
+}
 
-$path = Get-Attr $params "path" -failifempty $true
+$params = Parse-Args $args -supports_check_mode $true
-$user = Get-Attr $params "user" -failifempty $true
+$check_mode = Get-AnsibleParam -obj $params -name "_ansible_check_mode" -type "bool" -default $false
-$recurse = Get-Attr $params "recurse" "no" -validateSet "no","yes" -resultobj $result
+
-$recurse = $recurse | ConvertTo-Bool
+$path = Get-AnsibleParam -obj $params -name "path" -type "path" -failifempty $true
+$user = Get-AnsibleParam -obj $params -name "user" -type "str" -failifempty $true
+$recurse = Get-AnsibleParam -obj $params -name "recurse" -type "bool" -default "no" -validateset "no","yes" -resultobj $result
 
 If (-Not (Test-Path -Path $path)) {
     Fail-Json $result "$path file or directory does not exist on the host"
@@ -97,7 +98,7 @@ If (-Not (Test-Path -Path $path)) {
 
 # Test that the user/group is resolvable on the local machine
 $sid = UserSearch -AccountName ($user)
-if (!$sid)
+if (-not $sid)
 {
     Fail-Json $result "$user is not a valid user or group on the host machine or domain"
 }
@@ -110,9 +111,8 @@ Try {
 
     If ($acl.getOwner([System.Security.Principal.SecurityIdentifier]) -ne $objUser) {
         $acl.setOwner($objUser)
-        Set-Acl $file.FullName $acl
+        Set-Acl -Path $file.FullName -AclObject $acl -WhatIf:$check_mode
-
+        $result.changed = $true
-        Set-Attr $result "changed" $true;
     }
 
     If ($recurse) {
@@ -122,9 +122,8 @@ Try {
 
             If ($acl.getOwner([System.Security.Principal.SecurityIdentifier]) -ne $objUser) {
                 $acl.setOwner($objUser)
-                Set-Acl $file.FullName $acl
+                Set-Acl -Path $file.FullName -AclObject $acl -WhatIf:$check_mode
-
+                $result.changed = $true
-                Set-Attr $result "changed" $true;
             }
         }
     }

lib/ansible/modules/windows/win_owner.py

@@ -44,7 +44,6 @@ options:
   recurse:
     description:
       - Indicates if the owner should be changed recursively
-    required: false
     choices:
       - no
       - yes