PowerShell New-AddUser

Hey there,
I'm trying to add bulk users into Active Directory but for some reason it just won't take... The code is embedded here that I'm trying to run.

Import-Module ActiveDirectory
Import-Csv "C:\Scripts\test.csv" | ForEach-Object {
 $userPrincinpal = $_."samAccountName" + "@domainname.Local"
New-ADUser -Name $_.Name `
 -Path $_."Path" `
 -SamAccountName  $_."SamAccountName" `
 -UserPrincipalName  $userPrincinpal `
 -Surname $_. "Surname" `
 -Discription $_. "Discription" `
 -GivenName $_."GivenName" `
 -AccountPassword (ConvertTo-SecureString $_."AccountPassword" -AsPlainText -Force) `
 -ChangePasswordAtLogon $True `
 -Enabled $true
}

Open in new window


The error I get is:

PS C:\Users\Administrator.DomainName> C:\Scripts\Test.ps1
New-ADUser : A positional parameter cannot be found that accepts argument '.'.
At C:\Scripts\Test.ps1:4 char:1
+ New-ADUser -Name $_.Name `
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidArgument: (:) [New-ADUser], ParameterBindingException
    + FullyQualifiedErrorId : PositionalParameterNotFound,Microsoft.ActiveDirectory.Managemen
   t.Commands.NewADUser


I don't really understand why its giving me this error as the -Name parameter is valid i thought... I got the template for this code at

http://www.morgantechspace.com/2014/04/Create-Bulk-AD-Users-from-CSV-using-Powershell-Script.html 

and it seems to work for others. I tried putting in "-Name $_.Name `" and it gives the exact same error. I have attached the CSV file I'm referencing. It seems to have issues with the other ones that look like "-Name $_.Name `" if i remove the -Name attribute... It also doesn't enable the account even though the -Enable $_."Enable" ` is in there...
I have a huge list of people to add, The CSV is the same only this one doesn't have all the users in it. I can successfully add bulk users using my CSVs if i do this command in the PS window:

Import-Csv .|Test.csv | New ADUser

The problem I ran into there is AD wants the passwords as a secure string. I can add the attribute after the code above:

-AccountPassword (ConvertTo-SecureString "Test12344" -AsPlainText -Force)

And it works but I want the password from the CSV file. I know i could just add an attribute to make the user change the password but I want to be able to use the passwords from the CSV. That is why I want to use the Code that is attached.

Please excuse if some of my terms are wrong here as I am new to PS.
Testexample.csv
LVL 1
Jordan SmithAsked:
Who is Participating?

[Product update] Infrastructure Analysis Tool is now available with Business Accounts.Learn More

x
I wear a lot of hats...

"The solutions and answers provided on Experts Exchange have been extremely helpful to me over the last few years. I wear a lot of hats - Developer, Database Administrator, Help Desk, etc., so I know a lot of things but not a lot about one thing. Experts Exchange gives me answers from people who do know a lot about one thing, in a easy to use platform." -Todd S.

footechCommented:
I think the error is from lines 8 and 9 where you have a space after the dot, also "description" was spelled wrong.  Should be:
 -Surname $_."Surname" `
 -Description $_."Description" `

Open in new window

The error is detected in the New-ADUser command, but doesn't get specific enough about where in the command.
0
Member_2_4839798Commented:
This is interesting as I've never used this method before.
I would normally use the below method as an example, its not complete but you get the idea.

$User is the element within the array of users which is referenced in this example by "$($user", the proceeding ".Attribute)" is the heading of the column heading. So if I wanted to reference the SamAccountName header, I would use $($User.SamAccountName).

I'm not saying that your method is wrong, and to be honest I have never even tried this method, this is just an alternative method that might be a little easier for you :)

$Users = Import-CSV -path "C:\Scripts\Test.csv"
foreach($User in $Users)
    {
        New-ADUser -SamAccountName $($User.SamAccountName) -Name $($User.Name) ETC ETC ETC ETC
    }

Open in new window


All the best.

Michael
0

Experts Exchange Solution brought to you by

Your issues matter to us.

Facing a tech roadblock? Get the help and guidance you need from experienced professionals who care. Ask your question anytime, anywhere, with no hassle.

Start your 7-day free trial
footechCommented:
@MichaelIanClaridge - It's just the difference between using the foreach statement vs. the ForEach-Object cmdlet.
0
Determine the Perfect Price for Your IT Services

Do you wonder if your IT business is truly profitable or if you should raise your prices? Learn how to calculate your overhead burden with our free interactive tool and use it to determine the right price for your IT services. Download your free eBook now!

Member_2_4839798Commented:
@footech - Absolutely, I have just never used that method before to create AD Objects :)
The thing I love about PowerShell is that there is more than 1 way to achieve the same thing.

All the best.

Michael
0
footechCommented:
I agree.  :)
0
Jordan SmithAuthor Commented:
Thanks @MichaelIanClaridge! I ended up changing the script to your suggestion and it works great. This way is much simpler that the other way I was doing it... Thanks for the help!
0
footechCommented:
I'm curious why you would think it's simpler?  About the only difference is that you're using $User instead of $_ for all the parameters.

Though to be honest, when I was first starting out in PowerShell I did find using the foreach statement easier to understand than the ForEach-Object cmdlet, I think mainly because I hadn't fully grasped the concept of the pipeline yet.
0
It's more than this solution.Get answers and train to solve all your tech problems - anytime, anywhere.Try it for free Edge Out The Competitionfor your dream job with proven skills and certifications.Get started today Stand Outas the employee with proven skills.Start learning today for free Move Your Career Forwardwith certification training in the latest technologies.Start your trial today
Powershell

From novice to tech pro — start learning today.