Make it possible to modify loot from leaves.
Daripher opened this issue · 9 comments
It seems to be impossible to modify loot from DymanicTrees leaves using LootModifiers.
Please, allow mod creators to modify loot from leaves.
using rlcraft reimagined, the loot (lack of sticks %) from these leaves make notreepunch tech tree very difficult.
You can modify the loot dropped by dynamic trees (when broken by hand/tool manually) by adding a loot table like normal. Other than that loot is dynamic and as such uses our own systems (drop creators) which are customisable in their own right.
To be honest I am considering trialing fully embracing loot tables at some point in the future, but unfortunately until then customising drops is not easy (requires modifying the species Json themselves except for manual breaking as explained above).
My point is that we don't integrate with loot tables properly at the moment, but it is something I plan on changing so stuff like this is easier. As I said before you can still change the drops for manual leaves breaks using a custom loot table, but not with loot modifiers.
Drop creators are effectively just our custom loot tables. I ported them from 1.12 to work with our customisation options for species* not realising how internally customisable loot tables are. The current system means you have to override a species, changing its drop_creators array which certainly isn't ideal.
As this is now a planned change I'm going to leave this issue open.
* The trees folder in the MC directory effectively acts as a resources directory like DT and its add-ons use (see DT's resources here), allowing the user to override or create your own trees, leaves, species, etc in Json.
And yes, DT uses its own leaves. This is a requirement for the functionality. All DT leaves use the same registry names as their vanilla counterparts except with dynamictrees as the namespace (e.g. dynamictrees:oak_leaves). Also note that this block is only used in the tree multi blocks, so when you break the leaves with shears it will give you the vanilla ones.
Has this feature been implemented yet? I tried modifying the leaves loot table with KubeJS, and was very confused when it didn't work.
I have done most of the work for fully embracing loot tables here. I will be opening a PR for it soon where the changes will be fully explained.