Import-Module ActiveDirectory

$users = get-aduser -f {physicalDeliveryOfficeName -eq '07-129'} 
$i = 0
foreach($users in $users){
$i++
Write-Host $users

The code above is what I am currently working on.

What I am trying to do is find any entry in AD where "physicalDeliveryOfficeName" is equal to a number and NOT a letter. Then I am trying to take the "description" attribute and replace the "physicalDeliveryOfficeName" with the "description" and possibly store the previous "physicalDeliveryOfficeName" in another variable.

Any help would be greatly appreciated!

The attribute "physicalDeliveryOfficeName " is a string it is not a number.  --eq requires an exact match so the attribute must have those exact charcters to work.

Start by reviewing how this works.

get-aduser -f {physicalDeliveryOfficeName -eq '07-129'}-Properties description,physicalDeliveryOfficeName  |
     select name,description,physicalDeliveryOfficeName 

Well see that is my problem, while it is a string some of the strings that have been entered are numbers similar to what is shown above and I am trying to locate those to replace.

Like when I run the code above I get a user but his physicalDeliveryOfficeName = "07-129" instead of a location like "Florida", that is why I need to return the results of those that are not letters but numbers (0-9).

• Edited by Chuck_1987 12 hours 7 minutes ago

Try this, I suck at regular expressions so not sure if it will work

$users = Get-ADUSer -Filter * -Prop PhysicalDeliveryOfficeName | Where {
  $_.PhysicalDeliveryOfficeName -match "[0-9]{2}-[0-9]{3}"
}

I saw that, that is what helped me to initially create the above code. But that requires a specific value, I still need to be able to tell whether or not the location has an integer or string and replace them.

That code runs without an error but it does not display any results when I try to sift through it using:

foreach($users in $users){
$i++
Write-Host $users
}

You need to learn how to use PowerShell first then you can start adding the AD bits.

get-aduser -filter * -Properties description,physicalDeliveryOfficeName  |
    Where{ $_,physicalDeliveryOfficeName -match '\d+-\d+'}
     select name,description,physicalDeliveryOfficeName 

Once you see how this works it can be easily turned into a fix tool.

That code runs without an error but it does not display any results when I try to sift through it using:

foreach($users in $users){
$i++
Write-Host $users
}

help foreach - read all of the help to learn how to use the syntax.

$users = get-aduser -Properties * -Filter *
foreach($user in $users)
{
$CheckNum = $user.Description
if($CheckNum -match "[0-9]")
{
Set-ADUser $user.SamAccountName.ToString() -Office $CheckNum.ToString()
}
else
{
}
}

You need to learn how to use PowerShell first then you can start adding the AD bits.

get-aduser -filter * -Properties description,physicalDeliveryOfficeName  |
    Where{ $_,physicalDeliveryOfficeName -match '\d+-\d+'}
     select name,description,physicalDeliveryOfficeName 

Once you see how this works it can be easily turned into a fix tool.

Thanks clayman2 that lets me locate all of the users who do have a number in the office listing! Now I just need to figure out how to take their description and place it in their physicalDeliveryOfficeName.

This is up to you, as jrv mentions you should learn the basic powershell stuff then add the AD stuff once you have some of the basics down. Here is a sample

foreach($user in $user)
{
  $desc = $user.Description
  $params = @{physicalOfficeDeliveryName=$desc}
  Set-ADUser $user -Replace $params
}

I built the following for you because I had some time. It will go through every account in AD and check the description field for a number. If one is found it places the value of the description field into the physicalDeliveryOfficeName attribute field.

$users = get-aduser -Properties * -Filter *
foreach($user in $users)
{
$CheckNum = $user.Description
if($CheckNum -match "[0-9]")
{
Set-ADUser $user.SamAccountName.ToString() -Office $CheckNum.ToString()
}
else
{
}
}

Be careful.... this will alter all us

Thanks for all the help! I will dive further into this and see where I come out but I feel as though this is a good start. 

Also I cannot user the replace function for some reason but I can research that :)

Because this will be faster and match more cases:

get-aduser -filter * -Properties description,physicalDeliveryOfficeName  |
     Where{ $_,physicalDeliveryOfficeName -match '\d+-\d+'}
     ForEach-Object{
          $_,physicalDeliveryOfficeName=$_.description
          Set-AdUser -Instance $_ -PassThru
     } |
     Select *

Hey!  If you have PowerShell why not use it.

Well see that is my problem, while it is a string some of the strings that have been entered are numbers similar to what is shown above and I am trying to locate those to replace.

Like when I run the code above I get a user but his physicalDeliveryOfficeName = "07-129" instead of a location like "Florida", that is why I need to return the results of those that are not letters but numbers (0-9).

• Edited by Chuck_1987 Wednesday, July 01, 2015 7:01 PM

Well see that is my problem, while it is a string some of the strings that have been entered are numbers similar to what is shown above and I am trying to locate those to replace.

Like when I run the code above I get a user but his physicalDeliveryOfficeName = "07-129" instead of a location like "Florida", that is why I need to return the results of those that are not letters but numbers (0-9).

• Edited by Chuck_1987 Wednesday, July 01, 2015 7:01 PM