How to change Custom Object Property Values while doing foreach-object loop on Objectarray

Question

function DeleteADUser ()
{
    $AccountObjectCollection | foreach-object {

    if ($_.ADAccountExists -eq $true)
    {
        write-host "User: $($_.ADAccount) found. Deleting..." -ForegroundColor white
        $User = Get-ADUser -server $domain -Identity $_.ADAccount
        Remove-ADUser $User -server $domain -Confirm:$False
        write-host "User: $($_.ADAccount) deleted. Check again..." -ForegroundColor white
            try {
                Get-ADUser  -server $domain -Identity $_.ADAccount | Out-Null
                write-host "User: $($_.ADAccount) exists. Deletion failed!." -ForegroundColor red
        }
                catch [Microsoft.ActiveDirectory.Management.ADIdentityResolutionException] {
                $_.ADAccountExists = $false
                write-host "User: $($_.ADAccount) deleted." -ForegroundColor green
        }
    
    }else {write-host "User: $($_.ADAccount) not found." -ForegroundColor green}

    }
}

I have some custom objects in an object array $AccountObjectCollection and try to changes values in my foreach loop on the fly but the line $_.ADAccountExists = $false gets the following exception: ´The property 'ADAccountExists' cannot be found on this object. Verify that the property exists and can be set.´ Thats odd because I'm literally in an IF Statement that only shoots when this specific Value exists and is set to true so I'm propably doing something wrong here. How can I change the value inside the foreach loop?

Try to use the .keys method; loop through the keys and then access the object via $AccountObjectCollection[$key] — Farbkreis
– Farbkreis, Commented Apr 14, 2021 at 7:49
Inside your catch block, $_ refers to an exception object, not the current AD account object piped into your foreach-object loop. Thus, it might be best if you used foreach ($ADObject in $AccountObjectCollection) { ... } and replaced your instances of $_ with $ADObject. — leeharvey1
– leeharvey1, Commented Apr 16, 2021 at 1:14
@leeharvey1 Thats exactly what I needed to know. Now I understand why it didn't work and fixed it by using your method! Please post your comment as answer, so I can approve it. — ddl
– ddl, Commented Apr 16, 2021 at 9:21

leeharvey1 · Accepted Answer · 2021-04-16 13:43:15Z

2

Inside your catch block, $_ refers to an exception object, not the current AD account object piped into your foreach-object loop.

Thus, it might be best if you used:

foreach ($ADObject in $AccountObjectCollection) {
    # ...
}

...and replaced your instances of $_ inside this foreach block with $ADObject

answered Apr 16, 2021 at 13:43

leeharvey1

1,45613 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

How to change Custom Object Property Values while doing foreach-object loop on Objectarray

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related