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

fix: #5542 Fixing damage on Teleport #5544

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tonitch
Copy link
Contributor

@tonitch tonitch commented Feb 20, 2024

Reseting the LastGroundHeight on teleport
Using inheritence for teleporting to avoid rewriting code

fix: #5542 (and potentialy other)

@tonitch
Copy link
Contributor Author

tonitch commented Feb 20, 2024

should close : #5286

@tonitch
Copy link
Contributor Author

tonitch commented Feb 20, 2024

@bearbin there seem to be a broblem with the CI (git not installed or something) and I heard that you are the one in charge ^^ can you take a look ?

@cuberite-bot
Copy link

OK that was a bizarre problem, somehow it had just got stuck and refused to actually try a build. Restarting solved it though, so it's all good now :)

Copy link
Member

@bearbin bearbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong account, this looks fine as a hotfix.

@bearbin bearbin enabled auto-merge (squash) February 20, 2024 21:04
auto-merge was automatically disabled February 20, 2024 21:07

Head branch was pushed to by a user without write access

@tonitch
Copy link
Contributor Author

tonitch commented Feb 23, 2024

tbh I don't really know how to add the bindings ^^

@12xx12
Copy link
Member

12xx12 commented Feb 23, 2024

Add it in the APIDump plugin into the APIDesc.lua file. Search for the other functions from cPawn

Reseting the LastGroundHeight on teleport
Using inheritence for teleporting to avoid rewriting code
@tonitch tonitch force-pushed the tonitch/fix/#5542-teleport-damage-fix branch from 3ebf66f to d0bbba7 Compare February 26, 2024 14:17
@@ -9829,6 +9829,25 @@ a_Player:OpenWindow(Window);
},
Notes = "Removes a currently applied entity effect",
},
TeleportToCoords =
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have two questions/concerns:

  • How do the API docs handle this override? Do we have other cases where we do this?
  • Can this export ever be reverted in the future if necessary, without breaking things?

It's been a while since I touched any of this. If everything is fine, feel free to dismiss this comment.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I honestly don't know how this api export works ^^ I though it would be better to use inheritence in this patch but it's indeed not strictly necessary so I can just do a dirty patch instead ^^

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.

Taking damage on TP
5 participants