You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've discovered a small regression (?) which seems to have been introduced with #877 affecting disabled breakers.
Before
this.emit('fire', args);
The fire event was triggered for disabled breakers as well.
After
Moving the disablement check before the fire event changes the existing behaviour for disabled breakers.
I am not sure if this change was intentional - however it could lead to unwanted side effects for disabled breakers. Moving the emit call before the disablement check would revert the behaviour to the existing status.
The text was updated successfully, but these errors were encountered:
ullumullu
added a commit
to ullumullu/opossum
that referenced
this issue
Oct 25, 2024
I've discovered a small regression (?) which seems to have been introduced with #877 affecting disabled breakers.
Before
The fire event was triggered for disabled breakers as well.
After
Moving the disablement check before the fire event changes the existing behaviour for disabled breakers.
I am not sure if this change was intentional - however it could lead to unwanted side effects for disabled breakers. Moving the emit call before the disablement check would revert the behaviour to the existing status.
The text was updated successfully, but these errors were encountered: