FIX: Don't install action trigger hook when notifications set to UnityEvents (case ISXB-711) (#2089)

AlexTyrer · web-flow · commit aec5efcdc304 · 2024-12-17T15:05:30.000+02:00
diff --git a/Packages/com.unity.inputsystem/InputSystem/Plugins/PlayerInput/PlayerInput.cs b/Packages/com.unity.inputsystem/InputSystem/Plugins/PlayerInput/PlayerInput.cs
@@ -981,7 +981,9 @@ private void UpdateDelegates()
             }
             if (m_AllMapsHashCode != allMapsHashCode)
             {
-                InstallOnActionTriggeredHook();
+                if (m_NotificationBehavior != PlayerNotifications.InvokeUnityEvents)
+                    InstallOnActionTriggeredHook();
+
                 CacheMessageNames();
                 m_AllMapsHashCode = allMapsHashCode;
             }

Original file line number	Diff line number	Diff line change
`@@ -981,7 +981,9 @@ private void UpdateDelegates()`
`981`	`981`	`}`
`982`	`982`	`if (m_AllMapsHashCode != allMapsHashCode)`
`983`	`983`	`{`
`984`		`- InstallOnActionTriggeredHook();`
	`984`	`+ if (m_NotificationBehavior != PlayerNotifications.InvokeUnityEvents)`
	`985`	`+ InstallOnActionTriggeredHook();`
	`986`	`+`
`985`	`987`	`CacheMessageNames();`
`986`	`988`	`m_AllMapsHashCode = allMapsHashCode;`
`987`	`989`	`}`