mcMMO-Dev / mcMMO-Classic

mcMMO Classic.
https://www.spigotmc.org/resources/official-mcmmo-classic.2445/
Other
37 stars 44 forks source link

Inspecting an offline player broken. #13

Closed andrewkm closed 5 years ago

andrewkm commented 5 years ago

https://pastebin.com/raw/3ETJFt8w Running on build 31 via: https://ci.ecocitycraft.com/job/McMMO-Classic/

nossr50 commented 5 years ago

I thought t00thpick1 already fixed this

andrewkm commented 5 years ago

Nope, inspect is pretty badly screwed up.

andrewkm commented 5 years ago

It's also causing tons of weird ass issues such as: https://i.imgur.com/r5jPNU4.png

Also the offline name you're trying to inspect dissappears from mctop until the player goes online.

Look at above screenshot. I just did /inspect bobbyblack who is offline.

andrewkm commented 5 years ago

When bobby joins back online, inspect and mctop work and show him back in the list: https://i.imgur.com/zy0szCY.png

andrewkm commented 5 years ago

So basically using mctop on an offline player shoots the error, and then causes this: https://i.imgur.com/nwtMVKR.png invalid old username shit.

Also when bobbyblack logged off, I used inspect on him, and it once again shot error and made him go back to invalid: https://i.imgur.com/QhYxoOR.png

oof :(

t00thpick1 commented 5 years ago

On it.

On Sat, Mar 30, 2019, 2:09 PM Andrewkm notifications@github.com wrote:

So basically using mctop on an offline player shoots the error, and then causes this: https://i.imgur.com/nwtMVKR.png invalid old username shit.

Also when bobbyblack logged off, I used inspect on him, and it once again shot error and made him go back to invalid: https://i.imgur.com/QhYxoOR.png

oof :(

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/mcMMO-Dev/mcMMO-Classic/issues/13#issuecomment-478272943, or mute the thread https://github.com/notifications/unsubscribe-auth/ABRIaQK3j6R-ueB4jaIGe9FdQO8qgQJUks5vb6hjgaJpZM4cTy9L .