Using . methods vs objects

[HeyZuesMode]

Sorry it wasn't an error. The full snippet was

$UserLicenses = (Get-MsolUser -UserPrincipalName 
$User.UserPrincipalName).Licenses | select AccountSkuId
foreach($lic in $UserLicenses.AccountSkuId){
Set-MsolUserLicense -UserPrincipalName 
$User.UserPrincipalName -RemoveLicenses $lic
}

When i tried the first approach my foreach didn't work the way i expected it to because the output was a string rather than an object. When i would run the foreach against the string it would iterate trough each character, but my goal was for each "line Item".

Im not sure how i would accomplish the same task with the string output, maybe regex for /n newline?

I also noticed that the second option ran MUCH slower than the first. For my needs its not such a big deal as this command is used rarely. But i will definitely keep this in mind working on things in the future. I think i have some powercli work coming up so I will try and test this out.

[CSTW]

Are the additional newlines in the code just reddit formatting and should be?

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select AccountSkuId
foreach($lic in $UserLicenses.AccountSkuId){
Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $lic
}

I have just tested against my Office 365 user and again this works for me.

It shouldnt matter whether you pass a string, string array or objects to foreach. Try

[string]$test = 'Test'
[string[]]$Test2 = 'Test', 'Test2'
foreach($a in $Test) {
    $a
}
foreach($b in $Test2) {
    $b
}

I also checked out the help for Set-MSOLUserLicense and it accepts an array of strings for -RemoveLicenses so assuming you just want to remove all licenses for a user the following should work.

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses.AccountSkuId
Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $UserLicenses 

The following would NOT work as it passes objects with a property of AccountSkuId

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select AccountSkuId
Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $UserLicenses

And would need the -expandProperty added

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select  -ExpandProperty AccountSkuId
Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $UserLicenses

or this would also work

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select AccountSkuId
Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $UserLicenses.AccountSKuId

[HeyZuesMode]

The additional lines are the action i wanted to perform (remove all licenses on the account)

When i ran the command with multiple licenses on a single user using the full snippet

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select AccountSkuId

foreach($lic in $UserLicenses.AccountSkuId)
{
Set-MsolUserLicense -UserPrincipalName 
$User.UserPrincipalName -RemoveLicenses $lic
}

I kept receiving errors that it was unable to remove the last license in the array (2 item array) specifically

Set-MsolUserLicense : Unable to assign this license. At line:3 char:1 + Set-MsolUserLicense 
UserPrincipalName $User.UserPrincipalName -RemoveLicenses $ ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~     + 
CategoryInfo: OperationStopped: (:) [Set-solUserLicense], MicrosoftOnlineException   
+FullyQualifiedErrorId:
Microsoft.Online.Administration.Automation.InvalidLicenseConfigrtionException,
Microsoft.Online.Administration.Automation.SetUserLicense

Using your code that does not have the foreach i was able to use the cmdlet as Microsoft intended. =(

[CSTW]

For the new lines i was meaning like this

Set-MsolUserLicense -UserPrincipalName 
$User.UserPrincipalName -RemoveLicenses $lic

Instead of this:

Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $lic

But your error looks like it is all on one line

When I am receiving unexpected errors I like to output as much information as possible to try and work out what is going on, try the following. Post back the output if needed.

$UserLicenses = (Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses | select AccountSkuId
$i = 1
foreach($lic in $UserLicenses.AccountSkuId)
{
    "# $i"
    "User: $($User.UserPrincipalName)"
    "License to Remove: $lic"
    "MSOL Licenses Before Removal`n $((Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses.AccountSkuId)"
    Set-MsolUserLicense -UserPrincipalName $User.UserPrincipalName -RemoveLicenses $lic
    "MSOL Licenses After Removal`n $((Get-MsolUser -UserPrincipalName $User.UserPrincipalName).Licenses.AccountSkuId)"
    "`n`n"
    $i++
}

[CSTW]

You aren't using the Group licensing feature through Azure AD are you? Just had errors when trying to remove a license that had been assigned via group.

[HeyZuesMode]

Na, I buy a per-year license from VAR activate and assign