croxis / Lift

Minecraft/Bukkit elevator plugin
23 stars 26 forks source link

Getting random numbers in console #160

Closed RazihelYT closed 5 years ago

RazihelYT commented 5 years ago

Hey there, Whenever someone uses a lift button on the server, my server console gets the number 2 outputted 3 times. Ive tried disabling metrics and other checks, but it still persists. Any help is appreciated!

croxis commented 5 years ago

What build/version?

On Thu, Apr 18, 2019, 5:18 PM Razihel notifications@github.com wrote:

Hey there, Whenever someone uses a lift button on the server, my server console gets the number 2 outputted 3 times. Ive tried disabling metrics and other checks, but it still persists. Any help is appreciated!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/croxis/Lift/issues/160, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA5V55BEE5L2RQCTL3ZGEDPREFT5ANCNFSM4HHBFEIQ .

RazihelYT commented 5 years ago

Its the most recent one actually, v59

croxis commented 5 years ago

I honestly can't find it (unless I already removed it). Do you know what is happening when it prints?

RazihelYT commented 5 years ago

Its only when a player clicked the button to activate the elevator, Happens when player is going up or down, and is not floor specific.

croxis commented 5 years ago

Ok. I think a had already removed it for v60 which I hope for next weekend (famous last words). I'll leave this open in case it continues to happen.

RazihelYT commented 5 years ago

Would you mind if I test it out on my server? I have a lot of free-time on my hands and would be glad to test it out and see if the issue was resolved.

croxis commented 5 years ago

If you can you can build from this repo. My current code is a bit broken as I am adding support for mounted horses.

On Mon, May 6, 2019, 9:14 PM Razihel notifications@github.com wrote:

Would you mind if I test it out on my server? I have a lot of free-time on my hands and would be glad to test it out and see if the issue was resolved.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/croxis/Lift/issues/160#issuecomment-489900966, or mute the thread https://github.com/notifications/unsubscribe-auth/AAA5V5ZXQIGNQNEAPSHXHHDPUD64FANCNFSM4HHBFEIQ .

RazihelYT commented 5 years ago

Noticed right away after importing Spigot and CraftBukkit 1.14, that the Material.WALL_SIGN is an invalid material, I've made a pull request with fixes related to the issues.

RazihelYT commented 5 years ago

Noticed an error in 59, The coordinates are for the 2nd floor with the glass blocks, and I think it's checking for redstone blocks under the glass? 07.05 01:24:39 [Server] Server thread/INFO [Lift] WARNING: Unable to get sign for redstone pulse. 07.05 01:24:39 [Server] Server thread/INFO [Lift] Sign coords: Location{world=CraftWorld{name=Gallifrey},x=-247.0,y=92.0,z=-2.0,pitch=0.0,yaw=0.0} 07.05 01:24:39 [Server] Server thread/INFO [Lift] Sign material: OAK_WALL_SIGN

croxis commented 5 years ago

Fixed! I think. I'm not getting the error any more either