Loot table conditions based on breaking item enchantments

Published by Anonymous (not verified) on
Status
Fixed
Issue description

The UI version of the block drops are not been affected by enchantments, please fix or include this in the updated version to react like all vanilla blocks and enchantments

EG fortune or at lest have a option to enable and disable enchantment usage like we do for silktouch but a more general enchantment enabled and disabled option.

Issue comments

Author Submitted by Anonymous (not verified) on Tue, 05/14/2019 - 18:46 Permalink

When you use the option don't drop it's self option it does not take the vanilla enchantment system in to account what we really need I sure a older version did do this or it did when I tested it but that could of been a bug or another mod doing that lol

But if you never thought about it could you add the vanilla enchantment's so we can get drops calucation from the vanilla base so fortune and other things like this will work.

I think this might be more of a request then a bug as I don't think you added this I sure when it did work for me it was something else making it work or older versions dealt with drops different.

If this is a request rather then a bug could you also change the title for me to "Enable block drops affected by enchantments"

I will check what can be done on this. Now that I have found that once can actually drop itemstack and not item, which is a nice API addition by Forge developers, this could be possible to some extent.

Author Submitted by Anonymous (not verified) on Tue, 05/14/2019 - 18:53 Permalink

Cool I think it would help your normal users, you might have to add a option to disable it or enable it tho since we will want to still have the freedom to use the procedure, but cool can't wait to see what you come up with.

Could you give me some examples of what enchantments used to work when the block was mined aside from fortune and silk touch? Or are these two the main concern?

Efficiency didn't seem to do anything but it might do just to my quick view I couldn't see it deff when we set it same as stone.
but ya reading up what can be affected them two are the main two.

Friendly little bump lol I still think this will be a really good option.

Add support for fortune and for shears, check boxes like the silk touch one.

No it is not, no enchantment logic can now be controlled in blocks, this is why silk touch option was removed. All this is now controlled by the loot tables and conditions/predicates.

  • Added option to disable block dropping and use loot tables instead
  • Added option to make loot table entries decay on explosion dropping
  • [#64917] Loot entries can now have drop count amount set
  • [#59923] Added option to make loot table items enchanted
  • [#49691] Added option to condition loot table entries with enchantments such as silk touch and fortune