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

Skript world changing error #6610

Open
1 task done
ImTheLeviDR opened this issue Apr 26, 2024 · 2 comments
Open
1 task done

Skript world changing error #6610

ImTheLeviDR opened this issue Apr 26, 2024 · 2 comments
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. PR available Issues which have a yet-to-be merged PR resolving it

Comments

@ImTheLeviDR
Copy link

Skript/Server Version

[08:10:31 INFO]: [Skript] Skript's aliases can be found here: https://github.com/SkriptLang/skript-aliases
[08:10:31 INFO]: [Skript] Skript's documentation can be found here: https://docs.skriptlang.org/
[08:10:31 INFO]: [Skript] Skript's tutorials can be found here: https://docs.skriptlang.org/tutorials
[08:10:31 INFO]: [Skript] Server Version: git-Paper-196 (MC: 1.20.1)
[08:10:31 INFO]: [Skript] Skript Version: 2.8.4 (skriptlang-github)
[08:10:31 INFO]: [Skript] Installed Skript Addons:
[08:10:31 INFO]: [Skript]  - SkBee v3.4.3 (https://github.com/ShaneBeee/SkBee)
[08:10:31 INFO]: [Skript] Installed dependencies: None

Bug Description

Player not teleporting and error in the console (It's not SkBee issue)

Expected Behavior

The player teleports to the location

Steps to Reproduce

command /bedwarsadmin [] [] [] []:
permission: bedwars.setup
permission message: You can't do that.
trigger:
if arg-1 is not set:
send "&c/bedwars"
if arg-1 is "teleportplayer":
teleport arg-2 parsed as player to {map%arg-3%.team.%arg-4%}

Errors or Screenshots

https://paste.gg/p/anonymous/35b81dc185244f019564433a5b7d7964

Other

No response

Agreement

  • I have read the guidelines above and affirm I am following them with this report.
@bluelhf
Copy link
Contributor

bluelhf commented Apr 26, 2024

@Moderocky Looks like there's some more null-checking work to do!

What's the source of all these bugs? Is Location#getWorld() missing nullability annotations?

@Moderocky
Copy link
Member

So it's kind of left up to the user whether a world has a location, but the server really wants it to.
In a lot of cases we estimate the world if it's not present (e.g. if you teleport to X and don't say what world X is in, we use your world as a reference)

But in this case even though I fixed the teleporting, the chunk can't be obtained first because we don't know what world to fetch it from.
I'm going to attempt an estimation, but please specify a world.

@Moderocky Moderocky added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. PR available Issues which have a yet-to-be merged PR resolving it labels Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. PR available Issues which have a yet-to-be merged PR resolving it
Projects
None yet
Development

No branches or pull requests

3 participants