search

GlowstoneMC / Glowstone

A fast, customizable and compatible open source server for Minecraft: Java Edition
https://glowstone.net
Other
1.89k stars 271 forks source link

Use resolved UUID instead of illegal string. #979

Closed crashdemons closed 5 years ago

crashdemons commented 5 years ago

Reference #977 #978

PR #978 attempted to fix Issue #977 (Completablefuture typename was being used as the UUID string in NBT data), however after the null check, the uuid variable that was resolved is discarded instead of used in the NBT.

This change uses the resolved UUID in the NBT instead of the CompletableFuture type-string as the uuid.

In #977 I list some exceptions that occur when this string is read before adding this PR. After adding this PR, these don't seem to appear.

EDIT: double word

CLAassistant commented 5 years ago

CLA assistant check
All committers have signed the CLA.