Fixes Guardian/ElderGuardian API #968

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

Fixes Guardian/ElderGuardian API #968 #970

Closed j1o1h1n closed 5 years ago

j1o1h1n commented 5 years ago

This picks up from #046673f by @Pr0methean to fix #968

code to handle setElder is all in GlowGuardian, and works for both classes
added ElderGuardianStore, using the single GuardianStore for both didn't work out well
removed call to setElder in Store.load
tests for setElder

CLAassistant commented 5 years ago

All committers have signed the CLA.

aramperes commented 5 years ago

Hey @j1o1h1n, apologies for the delay. Could you merge this repo's dev branch back into your fork's guardian_968 branch? We've pushed a fix for the compilation errors.

aramperes commented 5 years ago

Thanks for your contribution!