Skip to content

Conversation

@nivcoo
Copy link
Contributor

@nivcoo nivcoo commented Mar 15, 2021

Hi of course you can test, the concern was that the time continued when we were in a world without fly and being on the ground.
It is therefore necessary when people can not fly deactivate the fly and pause the timer.

@Spazzinq Spazzinq changed the title FIX fly on world change Disable Timer on World Change Mar 15, 2021
@Spazzinq
Copy link
Owner

This commit didn't change anything and added a few lines of code, so I just reverted it.

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

He add world change event....

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

It's my mainly change in this pull request

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

Player world in the teleport event is the previous world so we have an issue with that

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

And i've add check to detect if the teleport event is a world change or not to remove the useless check

@Spazzinq
Copy link
Owner

He add world change event....
And i've add check to detect if the teleport event is a world change or not to remove the useless check

The teleport event includes if it's a world change. How does it add any functionality/fix anything?

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

Add debug on p.getWorld() on teleport event and you can look the issue :p

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

it sends the check to the old world and therefore activates the fly in a world where the fly is deactivated

@nivcoo
Copy link
Contributor Author

nivcoo commented Mar 15, 2021

Okay thx i will edit the file to add a function to do that more properly

@nivcoo
Copy link
Contributor Author

nivcoo commented May 15, 2022

Hello, some news ? thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants