hsyyid / EssentialCmds

EssentialCmds adds useful commands to Minecraft
https://bit.ly/2HPQ9tR
MIT License
31 stars 21 forks source link

/setspawn /spawn yaw point to south #74

Closed ghost closed 8 years ago

ghost commented 8 years ago

When do /setspawn and then /spawn yaw set to south, looks like not saved what I want using /setspawn.

Also pitch looks like not saved.

hsyyid commented 8 years ago

Send me your spawn.conf via PasteBin

ghost commented 8 years ago

http://pastebin.com/GDYhs3tr

ghost commented 8 years ago

another http://pastebin.com/dcixKJ7w

version 7.8b, spongeforge 1193

hsyyid commented 8 years ago

I have been testing this out locally, can you confirm this is still an issue?

ghost commented 8 years ago

version 7.8c, spongeforge 1195, issue still persist

ghost commented 8 years ago

7.8d, spongeforge 1207, issue still persist

ghost commented 8 years ago

I added some debug code to Utils.java getSpawn() and looks like problem is here: https://github.com/hsyyid/EssentialCmds/blob/master/src/main/java/io/github/hsyyid/essentialcmds/utils/Utils.java#L1142

Its wrongly detect if transform is in the config.

My config: http://pastebin.com/XuW0QKT0

Debug patch: http://pastebin.com/aXtY0xLv

Instead of "SPAWN PITCH" in the console I see "SPAWN"

ghost commented 8 years ago

I think found problem - typo - instead of warpsConfig must be spawnConfig on this line: https://github.com/hsyyid/EssentialCmds/blob/master/src/main/java/io/github/hsyyid/essentialcmds/utils/Utils.java#L1142

ghost commented 8 years ago

patch: http://pastebin.com/MwsqJYtb

Tested and fixed this issue.

hsyyid commented 8 years ago

@hunter688 thanks! I'll push a fix today

ghost commented 8 years ago

not fixed in 7.9 :(