FIX: Don't install action trigger hook when notifications set to UnityEvents (case ISXB-711) #2089
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
The fix for https://jira.unity3d.com/browse/ISXB-711 (PR #2077) introduced an bug - the action trigger hooks should not be installed when the notification setting is set to use UnityEvents.
This PR addresses that defect - only install the action trigger hooks for SendMessage / BroadcastMessage and CSharp event notification types.
Testing status & QA
Local tests (SimpleDemo_UsingPlayerInput scene)
Automated tests.
Overall Product Risks
Comments to reviewers
None in particular.
Checklist
Before review:
Changed
,Fixed
,Added
sections.Area_CanDoX
,Area_CanDoX_EvenIfYIsTheCase
,Area_WhenIDoX_AndYHappens_ThisIsTheResult
.During merge:
NEW: ___
.FIX: ___
.DOCS: ___
.CHANGE: ___
.RELEASE: 1.1.0-preview.3
.After merge: