-
-
Notifications
You must be signed in to change notification settings - Fork 410
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
triggerEvent/addEvent issue #2448
Comments
This is because you originally add the event in Since your event is already registered and owned by When I do think that ownership of events should be cascaded down to other resources when the owner resource has been stopped. It should be done in the order of the event handlers being added (oldest -> newest). In this instance, A small concern is whether we have to worry about backwards compatibility for this (regardless of our stance on bwc. for 1.6) - people may expect events to be destroyed when their resource owner is stopped. Or rather, should there be a parameter to choose the behaviour? |
I have made a PR that resolves this issue a while ago, perhaps it's a good time to give it a new review? |
Closed by #2333 |
Describe the bug
triggerEvent is not working when a given resource start/stop order is used.
To reproduce
test.zip
Expected behaviour
The event should be triggered (message should appear in debug console)
Version
Multi Theft Auto v1.5.9-release-21024.1
The text was updated successfully, but these errors were encountered: