How to move multiple AD Users into multiple OUs with powershell script? - powershell

Question

I want to move multiple users from input file to multiple OU's. But the code below keeps giving me error:
$i=0
$tempOUsArray = #{}
$tempOUsArray = Import-CSV -path "c:\temp\OUs.txt" #this file contains Full DN of OUs
$UsersToBeMoved = Import-CSV -path "c:\temp\Users.txt #this file contain SamAccountName field of users
$UsersToBeMoved | ForEach-Object{
$tempUsers = (get-aduser -identity $_.Name).distinguishedName
#assign each OU to temp holder
$TempOU = $tempOUsArray[$i]
Move-ADObject -Identity $tempUsers -TargetPath $TempOU
$i++ #inclement position of array
}

Answer 1

(See edits for previous answer attempts, clearing to reduce answer size)
See the below. I've tested this on a setup I have and this successfully moved the objects without an issue.
Users.txt (CSV format, matching what I believe yours is)
samaccountname,name
tst-user1,user1
tst-user2,user2
tst-user3,user3
tst-user4,user4
# (From OP: My users input file is)
# john.doe
# ann.bill
# jose.love
OUs.txt (no quotation marks in txt file, though may need if there's any spaces in the DN)
OU=test,OU=Accounts,DC=domain,DC=int
OU=test,OU=Accounts,DC=domain,DC=int
OU=test,OU=Accounts,DC=domain,DC=int
OU=test,OU=Accounts,DC=domain,DC=int
PS script
$i = 0
$OUs = #()
$OUs = Get-Content -Path C:\temp\OUs.txt
$Users = Import-Csv -Path C:\temp\Users.txt
$users | ForEach-Object {
$tempUser = (Get-ADUser -Identity $($_.samaccountname)).DistinguishedName
$tempOU = $OUs[$i]
Write-Host "Moving User: [$tempUser]"
Write-Host " to OU: [$tempOU]"
Move-ADObject -Identity $tempUser -TargetPath $tempOU
$i++
}
Output
Moving User: [CN=user1,OU=source,OU=test,OU=Accounts,DC=domain,DC=int]
to OU: [OU=test,OU=Accounts,DC=domain,DC=int]
Moving User: [CN=user2,OU=source,OU=test,OU=Accounts,DC=domain,DC=int]
to OU: [OU=test,OU=Accounts,DC=domain,DC=int]
Moving User: [CN=user3,OU=source,OU=test,OU=Accounts,DC=domain,DC=int]
to OU: [OU=test,OU=Accounts,DC=domain,DC=int]
Moving User: [CN=user4,OU=source,OU=test,OU=Accounts,DC=domain,DC=int]
to OU: [OU=test,OU=Accounts,DC=domain,DC=int]
And all worked properly, no errors. Can you try the above code, and ensure your source files are in a similar format?
Final edit
With the provided edit / update to the "users.txt" file, the below script should work as expected.
New users.txt file (note: no column header)
john.doe
ann.bill
jose.love
PS script
$i = 0
$OUs = #() # May as well clear, in case old data exists
$Users = #() # May as well clear, in case old data exists
$OUs = Get-Content -Path C:\temp\OUs.txt
$Users = Get-Content -Path C:\temp\Users.txt
$users | ForEach-Object {
$tempUser = (Get-ADUser -Identity $($_)).DistinguishedName
$tempOU = $OUs[$i]
Write-Host "Moving User: [$tempUser]"
Write-Host " to OU: [$tempOU]"
Move-ADObject -Identity $tempUser -TargetPath $tempOU
$i++
}

Answer 2

It is with the Move-ADObject. This function will NOT work with moving parts like multiple OUs assignment. My revised code is as below:
$i=0
$tempOUsArray = #{}
$tempOUsArray = get-content -path "c:\temp\OUs.txt" #this file contains Full DN of
OUs
$UsersToBeMoved = get-content -path "c:\temp\Users.txt" #this file contain
SamAccountName field of users
ForEach ($tuser in $UsersToBeMoved){
$tempUser = dsquery user -samid $tuser
#assign each OU to temp holder
$TempOU = $tempOUsArray[$i]
dsmove $tempUser -newparent $TempOU
$i++ #inclement position of array
}
$tempOUArray.Clear()
$TempOU=''
$tempUser=''
$tuser=''
Note: Remove All Header from input files. OUs.txt with full DN in double-quotes. Users.txt file contains only the SamAccountName without quotes.