Get username from custom computer attribute and add location from user object

Question

I need a single powershell script combining ad-computer + get-user to get the missing user attribute. I dont now how to combine the two classed into one script.
The Computer object attribute: "extensionattribute1" already has the Username in it, and i need to add the "physicalDeliveryOfficeName" to the computer attrbiute:
extensionattribute2.

Something like:

get-adcomputer COMPUTER1 -Properties Name,extensionattribute1 | select Name,extensionattribute1,(IN HERE I WOULD NEED THE QUERY FOR USER ATTRIBUTE "physicalDeliveryOfficeName")

set-ADComputer -Replace @{extensionAttribute2 = "$($_.physicalDeliveryOfficeName)"}

I have seen this for the Manager attribute. I would need the "physicalDeliveryOfficeName":
@{n='Manager';e={(Get-ADUser $_.Manager).Name}}

I need to combine these parts together into one script so i can write the location into the empty computer attribute extensionattribute2.

Computer Object: NOW:

name,extensionattribute1,extensionattribute2
PC1,John Doe,"NOT SET"

NEEDED:

name,extensionattribute1,extensionattribute2
PC1,John Doe,Office location xyz

Thanks in advance

Answer 1

A bit longer than one line, but it does have error checking:
EDIT: Moved the $n = $_.Name inside the ForEach-Object block.

Get-ADComputer COMPUTER1 -Properties Name, extensionattribute1 |
    ForEach-Object{
        $n = $_.Name  # needed to report computer name in Catch block
        $u = Get-ADUser -Filter {Name -eq $_.extensionattribute1} -Properties physicalDeliveryOfficeName
        if ($u){
            Try{
                $p = $u.physicalDeliveryOfficeName
                Set-ADComputer $_ -Replace @{extensionattribute2 = $p} -ErrorAction Stop
            }
            Catch{
                Write-Host "Failed to update computer object '$n'"
            }
        }
        else{
            Write-Host "Failed to find user '$($_.extensionattribute1)"
        }
    }

Answer 2

Hi,

You can try something like below

$computers = 'computer1','computer2'  
$computers | Get-ADComputer -Properties extensionattribute1 | ForEach-Object{  
    $office = (Get-ADUser -Filter {Name -eq $_.extensionattribute1} -Properties physicalDeliveryOfficeName).physicalDeliveryOfficeName  
    Set-ADComputer -Identity $_ -Replace @{extensionattribute2 = $office}  
}  

Best Regards,
Ian Xue

============================================

If the Answer is helpful, please click "Accept Answer" and upvote it.
Note: Please follow the steps in our documentation to enable e-mail notifications if you want to receive the related email notification for this thread.