Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Slimechunk expression wrong in 1.17 #4106

Closed
Primorior opened this issue Jun 23, 2021 · 1 comment
Closed

Slimechunk expression wrong in 1.17 #4106

Primorior opened this issue Jun 23, 2021 · 1 comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.

Comments

@Primorior
Copy link

Description

I used the slimeball expression to make it easier for my players to locate a slimechunk. While rightclicking with a slimeball it shows if there is a slimechunk or not. I attached the code.

Steps to Reproduce

-> Used this code as sk file:

on rightclick with slimeball:
	if chunk at player is a slime chunk:
		send "&aSlimechunk!"
	else:
		send "&cNo Slimechunk" 

-> Create world in 1.17 with seed "-8580642221070289223" (probably wrong with all seeds but in this one we used when one of my players found this issue)

-> Teleport to 178 / -2069

-> Use slime ball to see if it's a slimechunk and it says it is.
-> To validate if this is true, see here it's wrong, there is no slimechunk: https://www.chunkbase.com/apps/slime-finder#-8580642221070289223

Server Information

  • **Server version/platform: 3158-Spigot-f773da8-470050
  • **Skript version: 2.5.3
@TPGamesNL
Copy link
Member

CondIsSlimeChunk should use Chunk#isSlimeChunk if that method exists

@TPGamesNL TPGamesNL added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. priority: low Issues that are not harmful to the experience but are related to useful changes or additions. labels Jun 23, 2021
@APickledWalrus APickledWalrus added the PR available Issues which have a yet-to-be merged PR resolving it label Jun 25, 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 Jun 26, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. completed The issue has been fully resolved and the change will be in the next Skript update. priority: low Issues that are not harmful to the experience but are related to useful changes or additions.
Projects
None yet
Development

No branches or pull requests

4 participants