only call set-acl if necessary

9 years ago · 45249fb042
parent 3a5d4576c6
commit 45249fb042
1 changed files with 6 additions and 6 deletions
--- a/windows/win_owner.ps1
+++ b/windows/win_owner.ps1
@ -111,23 +111,23 @@ Try {
    $acl = Get-Acl $file.FullName

    If ($acl.getOwner([System.Security.Principal.SecurityIdentifier]) -ne $objUser) {
+        $acl.setOwner($objUser)
+        Set-Acl $file.FullName $acl
+
        Set-Attr $result "changed" $true;
    }

-    $acl.setOwner($objUser)
-    Set-Acl $file.FullName $acl
-
    If ($recurse) {
        $files = Get-ChildItem -Path $path -Force -Recurse
        ForEach($file in $files){
            $acl = Get-Acl $file.FullName

            If ($acl.getOwner([System.Security.Principal.SecurityIdentifier]) -ne $objUser) {
+                $acl.setOwner($objUser)
+                Set-Acl $file.FullName $acl
+
                Set-Attr $result "changed" $true;
            }
-
-            $acl.setOwner($objUser)
-            Set-Acl $file.FullName $acl
        }
    }
 }