Start-OBSEffect - Fixing -LoopCount (Fixes #133)

Author: James Brundage

Commands/Effects/Start-OBSEffect.ps1

@@ -104,6 +104,9 @@ function Start-OBSEffect
 
             if ($loop -or $Bounce) {
                 $obsEffect | Add-Member -MemberType NoteProperty Mode "$(if ($Bounce) {"Bounce"})$(if ($loop) {"Loop"})" -Force
+                if (-not $LoopCount) {
+                    $obsEffect | Add-Member -MemberType NoteProperty LoopCount -1 -Force
+                }
             } else {
                 $obsEffect | Add-Member -MemberType NoteProperty Mode "Once" -Force
             }

Types/OBS.PowerShell.Effect/Start.ps1

@@ -84,7 +84,7 @@ if ($this.Messages) {
 
                 $null = New-Event -SourceIdentifier "OBS.PowerShell.Effect.Ended" -MessageData $event.MessageData
                 $effectInfo = $event.MessageData
-                if ($effectInfo.LoopCount) {
+                if ($effectInfo.LoopCount -is [int] -and $effectInfo.LoopCount -ge 1) {
                     $effectInfo.LoopCount = $effectInfo.LoopCount - 1
                 }
 
@@ -94,8 +94,9 @@ if ($this.Messages) {
                     $effectInfo.Reversed = -not $effectInfo.Reversed
                     $effectInfo.Start()
                 } elseif ($effectInfo.Mode -match 'Loop') {
-                    if (($null -eq $effectInfo.LoopCount) -or 
-                        $effectInfo.LoopCount) {
+                    if ($effectInfo.LoopCount -is [int] -and $effectInfo.LoopCount -ge 1) {
+                        $effectInfo.Start()
+                    } elseif ($effectInfo.LoopCount -isnot [int] -or $effectInfo.LoopCount -lt 0) {
                         $effectInfo.Start()
                     }                    
                 } elseif ($effectInfo.Mode -match 'Bounce') {