VanillaTweaks [Fabric & Forge]

VanillaTweaks [Fabric & Forge]

8M Downloads

Blazing + Fortune, compatible or not?

Duxtorm opened this issue · 14 comments

commented

At https://strikerrocker.github.io/vanillatweaks
The description for Blazing is

"This enchantment goes on your pickaxe. Automatically smelts everything the block drops.
If you have fortune on your item, it will increase the stack size of the drops accordingly - but not for drops that are blocks themselves."

I'm unable to combine Fortune and Blazing on a pickaxe.

  • Forge Version: 36.2.4
  • MC Version: 1.16.5
  • Mod Version: 1.5.40
  • JEI Version (if present): 7.7.1.121
  • Baubles Version (if present): n/a
  • Curios Version (if present): n/a
commented

I fixed the issue in 1.17 but forgot to backport to 1.16. Will fix this soon

commented

Oh no!
I tested it out.. it doesn't seem to be working.
I can combine them on a pickaxe now, but it's not smelting anything.

commented

Fixed in the latest build released on curseforge for 1.16.5

commented

Fixed in the latest build released on curseforge for 1.16.5

Thank you! ♥

commented
commented

Yeah thats the intended behavior. If both are present fortune takes precendence.

On Mon, 20 Sep, 2021, 2:20 pm Mishka, @.***> wrote: Oh no! I tested it out.. it doesn't seem to be working. I can combine them on a pickaxe now, but it's not smelting anything. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#145 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG3E56MSM6MUFMM2YG6UOEDUC3YTRANCNFSM5EKLEF5Q .

Wait.. so what's the point of being able to have them both on the same pick in the first place if having them both makes one of them useless?
Why not make it so that it adds the multiplier then smelts?

commented

You make a good point. I am big derp...

commented

You make a good point. I am big derp...

XD ur fine! keep me updated ♥

commented

How would it work? Apply blazing effect after fortune effect is what i am thinking?

commented

The only item this would apply in vanilla is potatoes as no other item is fortunable and drop has a smelting recipe thus i am only testing with it. I will soon release a new version which does the above.

commented

Properly fixed this issue i hope and released a new version on curseforge with the fix.

commented

The only item this would apply in vanilla is potatoes as no other item is fortunable and drop has a smelting recipe thus i am only testing with it. I will soon release a new version which does the above.

Haha! I didn't know blazing could be used on potatoes!
All is well and good, I don't expect it to do anything unreasonable like multiply iron or anything, it's just nice to have a pick that can smelt one and multiply the other accordingly, rather than just overriding one of the enchantments.
Also, awesome! I'll try it out!

commented

Could be used in 1.17 with new iron drops. 👀

commented

I tried it out!
It works perfectly!
(I also made sure it works fine with siphon, which it does)

I really wanna play 1.17.. but I don't want to play it until the whole thing is out!
I want the full caves & cliffs update experience first-hand, so I'm just sticking with 1.16.5 for the time being.
And yeah, haha, that'll be weird.. fortune working on iron.. feels wrong.. but soo good XD