Skip to content
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

Why the order of event occurrence changes? #3661

Closed
vhone opened this issue Dec 26, 2020 · 0 comments
Closed

Why the order of event occurrence changes? #3661

vhone opened this issue Dec 26, 2020 · 0 comments
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something.

Comments

@vhone
Copy link

vhone commented Dec 26, 2020

Description

this is a problem where the order of events is reversed depending on when they are loaded.

On Exp Spawn event occur before On Fishing event.

Steps to Reproduce

on right click:
	add 1 to {step}
on unload:
	clear {step}
on fishing:
	send "[%{step}%] on fishing" to console
on item spawn:
	send "[%{step}%] on item spawn" to console
# on exp spawn:
	# send "[%{step}%] on exp spawn" to console

[case 1]
On Exp Spawn event is not registered When the server starts.
This event is registered through script reload.

on right click:
	add 1 to {step}
on unload:
	clear {step}
on fishing:
	send "[%{step}%] on fishing" to console
on item spawn:
	send "[%{step}%] on item spawn" to console
on exp spawn:
	send "[%{step}%] on exp spawn" to console

[case 2]
On Exp Spawn event is registered when the server starts.

Expected Behavior

Tests on both cases should show the same result.

Errors / Screenshots

001
[case 1]
After On Fishing event, On Exp Spawn event and the On Item Spawn event are occurs.
I think this case is right.

002
[case 2]
On Exp Spawn event occurs first.
I don't think this progression is correct because experience and items occur after collecting lure.

Server Information

  • Server version/platform: Paper 1.16.4
  • Skript version: 2.5.3

Additional Context

@TPGamesNL TPGamesNL added enhancement Feature request, an issue about something that could be improved, or a PR improving something. PR available Issues which have a yet-to-be merged PR resolving it labels Aug 19, 2021
@TPGamesNL TPGamesNL added completed The issue has been fully resolved and the change will be in the next Skript update. and removed PR available Issues which have a yet-to-be merged PR resolving it labels Aug 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
completed The issue has been fully resolved and the change will be in the next Skript update. enhancement Feature request, an issue about something that could be improved, or a PR improving something.
Projects
None yet
Development

No branches or pull requests

3 participants