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] Tilepos on blob hits being set to 0,0 instead of an actual tilepos #2114

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

Conversation

Vam-Jam
Copy link
Member

@Vam-Jam Vam-Jam commented Sep 2, 2024

Unsure what is actually going in the pickaxe code. Hitdata.tilepos only contains data when actually hitting a tile, but tilepos seems to contain the correct position when hitting a blob.

Another fix could be editing line 610 and use b.getPosition() as suggested by Gingerbeard.

Will do more testing later.

Credits to @Gingerbeard5773

Closes #2082

Unsure what is actually going in the pickaxe code. Another fix could be
editing line 610 and use b.getPosition() as suggested by Gingerbeard
@Vam-Jam Vam-Jam added the fix Fixes a bug label Sep 2, 2024
@Vam-Jam Vam-Jam added this to the post-4762 milestone Sep 3, 2024
TerminalHash added a commit to TerminalHash/Gruhsha-CTF that referenced this pull request Sep 3, 2024
@Vam-Jam Vam-Jam added testing needed An issue needs to undergo a testing sessions to verify the change or check for newly introduced bugs ready A pull request that functions correctly and is ready for testing labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix Fixes a bug ready A pull request that functions correctly and is ready for testing testing needed An issue needs to undergo a testing sessions to verify the change or check for newly introduced bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Particles are spawned in the wrong location (Build 4762)
1 participant