I don't have the time to try and understand how MMOCore works, so removing those TODOs. The only idea I have is that those fields are static so that on multiple calls of that packet (like spamming a button) the fields would always be updated with the data from the latest call. If so, that logic should have been documented because it is definitely not obvious. If not, I have no idea why someone thought it was a good idea to make them static.