caleb-brown/unity-atoms
1
0
Fork 0
mirror of https://github.com/unity-atoms/unity-atoms.git synced 2025-01-22 08:08:51 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix #321 NRE when Event Reference Listeners event reference is not set

Browse Source
This commit is contained in:
Thimo de Ram 2022-03-13 16:02:15 +01:00 committed by Miika Lönnqvist
parent a35e7342cc
commit 46037ac590
1 changed files with 18 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
Packages/Core/Runtime/EventReferences/AtomEventReference.cs
View File

@ -27,9 +27,12 @@ namespace UnityAtoms
{
switch (_usage)
{
case (AtomEventReferenceUsage.VARIABLE): return _variable.GetOrCreateEvent<E>();
case (AtomEventReferenceUsage.VARIABLE_INSTANCER): return _variableInstancer.GetOrCreateEvent<E>();
case (AtomEventReferenceUsage.EVENT_INSTANCER): return _eventInstancer.Event;
case (AtomEventReferenceUsage.VARIABLE):
return _variable == null ? LogNullReference(nameof(_variable)) : _variable.GetOrCreateEvent<E>();
case (AtomEventReferenceUsage.VARIABLE_INSTANCER):
return _variableInstancer == null ? LogNullReference(nameof(_variableInstancer)) : _variableInstancer.GetOrCreateEvent<E>();
case (AtomEventReferenceUsage.EVENT_INSTANCER):
return _eventInstancer == null ? LogNullReference(nameof(_eventInstancer)) : _eventInstancer.Event;
case (AtomEventReferenceUsage.EVENT):
default:
return _event;
@ -81,5 +84,17 @@ namespace UnityAtoms
{
return reference.Event;
}
private E LogNullReference(string atomEventReferenceUsage)
{
if (AtomPreferences.IsDebugModeEnabled)
{
throw new NullReferenceException($"{atomEventReferenceUsage} reference is missing in inspector.");
}
else
{
return null;
}
}
}
}