Add support for waxed varients for CopperRail simple admin hack#926
Add support for waxed varients for CopperRail simple admin hack#926bulbm1n wants to merge 3 commits into
Conversation
|
I left it using the current system so I didn't have to re-implement WeatheringCopper.getChanceModifier(). I can rewrite it though. I only read your original response from the notif, not your edited one, much easier fix. |
|
Let me know how that is, I rewrote a lot of it. More self-contained |
okx-code
left a comment
okx-code
left a comment
There was a problem hiding this comment.
You changed the axe damaging semantics so that it is no longer damaged for each oxidation step. also not sure why you removed the comment for the blockformevent listener?
| private enum CopperStage { | ||
| UNAFFECTED(Material.COPPER_BLOCK, Material.WAXED_COPPER_BLOCK, 1.0f), | ||
| EXPOSED(Material.EXPOSED_COPPER, Material.WAXED_EXPOSED_COPPER, 1.0f), | ||
| WEATHERED(Material.WEATHERED_COPPER, Material.WAXED_WEATHERED_COPPER, 0.75f), |
There was a problem hiding this comment.
also this is the wrong way round, it's only unaffected blocks that have a 75% chance
|
Looks better overall though |
|
Fixed all of the above. Got confused with the comment and the durability because I repeatedly had to pick this up and put it down |
2 participants
https://discord.com/channels/912074050086502470/1508545613782843642
Per my own suggestion, pretty simple implementation that treats it as a regular copper block then turns it back to a waxed copper block after it runs the checks and needs to replace the block. Felt that was the easiest way.