Split DN Reporting Blank

michaelcoyle · 2018-03-02T16:45:45.000Z

Another rookie mistake I expect. I’m running the following and it works without error and I’ve a longer script that runs perfectly as well. However, on some occasions I just want to check up on someones line manager so I wrote this:

Get-ADUser x | Select SamAccountName,Enabled,@{N="Manager";E= {%{(Get-AdUser $_.Manager).Name}}}

Which returns, a blank manager. Thoughts?

SamAccountName Enabled Manager
-------------- ------- -------
x                True

Neally · 2018-03-02T17:14:20.000Z

You are missing the property for the manager

Get-ADUser $username -Properties Manager  | 
Select SamAccountName,Enabled,@{N="Manager";E= {(Get-AdUser $_.Manager).Name}}

Neally · 2018-03-02T17:18:20.000Z

babayagaa:

@{N="Manager";E= {%{(Get-AdUser $_.Manager).Name}}}

Thoughts?

Why do you use % rather than foreach ? ( da rulz )

You can not have multiple managers AFAIK, so that part seems redundant.

jitensh · 2018-03-02T17:53:35.000Z

Neally:

babayagaa:
@{N="Manager";E= {%{(Get-AdUser $_.Manager).Name}}}
Thoughts?
Why do you use % rather than foreach ? ( da rulz )

You can not have multiple managers AFAIK, so that part seems redundant.

This is my piece of Code, I posted last time an answer, and most of the time I post alias blame me:p

If you frequently need to query users manager better try this Not tested, just wrote pain in code

Function Get-Manager {
Param($user)
If(-not($user))
{
Write-Warning "Please supply user Name"
}
Get-ADUser -user $user -Properties Manager  | 
Select SamAccountName,Enabled,@{N="Manager";E={(Get-AdUser $_.Manager).Name}}
}

Just run the above script and you can run

Get-Manager -user jack

you just need to run get-manager -user and username next time, Hopefully it works

Neally · 2018-03-02T18:29:35.000Z

JitenSh:

I post alias

michaelcoyle · 2018-03-02T22:10:00.000Z

JitenSh:

Neally:
babayagaa:
@{N="Manager";E= {%{(Get-AdUser $_.Manager).Name}}}
Thoughts?
Why do you use % rather than foreach ? ( da rulz )

You can not have multiple managers AFAIK, so that part seems redundant.
This is my piece of Code, I posted last time an answer, and most of the time I post alias blame me:p

If you frequently need to query users manager better try this Not tested, just wrote pain in code
Function Get-Manager {
Param($user)
If(-not($user))
{
Write-Warning "Please supply user Name"
}
Get-ADUser -user $user -Properties Manager  | 
Select SamAccountName,Enabled,@{N="Manager";E={(Get-AdUser $_.Manager).Name}}
}
Just run the above script and you can run
Get-Manager -user jack
you just need to run get-manager -user and username next time, Hopefully it works

I’ve bastardised so much of your code JitenSh. Can’t thank you enough! I swear I’m making process, the last few weeks have been rough (new job) but you’ve been incredibly helpful. Thank you again!

And thank you all. Appreciate all of the assistance!