laynebritton / jl-fast-treecapitator

A more efficient tree and ore vein miner for Minecraft Bedrock Edition. Cross-platform supported. Faster runtime and lower space complexity than pre-existing solutions.
4 stars 0 forks source link

Remove faulty initialization logic from User Config #33

Closed laynebritton closed 1 year ago

laynebritton commented 1 year ago

Delete the after worldInitialize event that was setting bad data in the dynamic properties.

The helper classes in the dynamic properties cover the not instantiated yet case, so this is redundant and harmful.

See https://github.com/laynebritton/jl-fast-treecapitator/blob/main/scripts/DynamicProperties/AllowSet.ts#L19-L22 and https://github.com/laynebritton/jl-fast-treecapitator/blob/main/scripts/DynamicProperties/JLTreeCapConfig.ts.ts#L16-L19

jessejohn01 commented 1 year ago

@laynebritton Please push a v1.1.1 tag to generate a new release. I pulled the old one.