foreach not executing on button click

[AKoval]

I have a little script i'm putting together to help a team take members of a security group and explicitly give them permission to a mailbox with Send-As and FullAccess. To trouble shoot this step i created a label in the GUI and if i try and update from inside the foreach loop it does not update and if i put it right after the foreach it updates no problem.

Code: Select all

$buttonConvert_Click={
	#TODO: Place custom script here
	foreach ($member in $Members)
	{
		$label1.Text = "Members?"
		Add-MailboxPermission -Identity "$Mailbox" -User "$member" -AccessRights FullAccess
		Add-ADPermission -Identity "$Mailbox" -User "$member" -ExtendedRights Send-As
		
	}
	#$label1.Text = "Complete"
}

[Alexander Riedel]

Call DoEvents in your loop.
https://www.sapien.com/blog/2011/07/15/ ... ive-loops/

[AKoval]

forgive me if i am just completely missing the point, but i'm not seeing how that helps. Its currently not even attempting to run the foreach loop at all.

[Alexander Riedel]

The label will not update while you are in that loop unless you call DoEvents. So I would guess at this point you do not know if you are entering the ForEach loop or not.
Why not just set a breakpoint and debug?

[jvierra]

Quoting and using the wrong variable will cause bad behaviors

Code: [Select all] [Expand/Collapse] [Download] (Untitled.ps1 expanded)

1. $buttonConvert_Click={
2.     #TODO: Place custom script here
3.     foreach ($member in $Members){
4.         $label1.Text = $members
5.                 [application]::DoEvents()
6.         Add-MailboxPermission -Identity $Mailbox -User $member -AccessRights FullAccess
7.         Add-ADPermission -Identity $Mailbox -User $member -ExtendedRights Send-As
8.        
9.     }
10.     $label1.Text = 'Complete'
11. }

Also Alex's suggestion is the best way to understand these things. Use the debugger and breakpoint at the start of the loop then "step".

String variables do not need quoting.