[bugfix] Always use the first game event as primary, except for penalty

Author: g3force

internal/app/controller/gameEvent.go

@@ -182,6 +182,7 @@ func (e GameEvent) IsSecondary() bool {
 		GameEventUnsportingBehaviorMinor,
 		GameEventUnsportingBehaviorMajor,
 		GameEventMultipleFouls,
+		GameEventMultiplePlacementFailures,
 		GameEventBotCrashUniqueSkipped,
 		GameEventBotPushedBotSkipped,
 		GameEventPlacementFailedByTeamInFavor,

internal/app/controller/placementPos.go

@@ -6,7 +6,7 @@ import (
 	"math"
 )
 
-// BallPlacementPos determines the ball placement position based on the game event
+// BallPlacementPos determines the ball placement position based on the primary game event
 func (e *Engine) BallPlacementPos() *Location {
 	event := e.State.PrimaryGameEvent()
 	if event == nil {

internal/app/controller/state.go

@@ -394,13 +394,16 @@ func (s State) BotSubstitutionIntend() Team {
 	return TeamUnknown
 }
 
-func (s State) PrimaryGameEvent() *GameEvent {
+func (s State) PrimaryGameEvent() (e *GameEvent) {
+	e = nil
 	for i := len(s.GameEvents) - 1; i >= 0; i-- {
-		if !s.GameEvents[i].IsSecondary() {
-			return s.GameEvents[i]
+		e = s.GameEvents[i]
+		if e.Type == GameEventMultipleCards {
+			// only this event causes a penalty kick and must be prioritized.
+			return
 		}
 	}
-	return nil
+	return
 }
 
 func newTeamInfo() (t TeamInfo) {