Civcraft / RealisticBiomes

Do not open issues here; open them on the maintained fork @ DevotedMC
https://github.com/DevotedMC/RealisticBiomes
7 stars 14 forks source link

Cocoa persistence #20

Closed benjajaja closed 9 years ago

benjajaja commented 9 years ago

Before moving on to other crops, I did some cleanup of cocoa persistence. I remember there was a bug with cocoa persistence, and in the config it is commented-out so I assume it was disabled for buggy behaviour.

Previously the age of cocoa was set with deprecated Block.setData and quite some magic to set the correct bits, I changed this to using bukkit's MaterialData API.

I have tested locally and it seemed like it works correctly, cocoa pods were set to their correct age when loading chunks.

WildWeazel commented 9 years ago

IIRC the bug happened when manually checking/updating the state with a stick so you should probably try that too.

benjajaja commented 9 years ago

Will do and report back ASAP

benjajaja commented 9 years ago

Checked cocoa with stick, working fine: reports grown time of total growth and updates block correctly. Also tried reloading chunk in between checks, all ok.

WildWeazel commented 9 years ago

@ttk2 http://civcraft.co:8080/job/RealisticBiomes-master/14/

ghost commented 9 years ago

On Civtest