p3lim-wow / Molinari

One-click Milling, Prospecting, Disenchanting and Lockpicking!
MIT License
16 stars 7 forks source link

It doesn't disenchanting ilv316 BOE items #150

Open seakoria opened 5 months ago

seakoria commented 5 months ago

Describe the bug blacksmithing craft that ilv 316 waist,item id 55035 and can't disenchanting it press alt+left button

How to reproduce alt + LB it?

What did you expect? disenchanting it correctly

Screenshots the item icon glowing 3 times like a new item first time loot in bag 1 1

p3lim commented 5 months ago

Try this version: https://github.com/p3lim-wow/Molinari/actions/runs/9068309172/artifacts/1498383170

Ragedaug commented 5 months ago

I was having a lot of trouble disenchanting and milling most items in Cata Classic running version 112, and so far this version seems to have fixed it. Thank you.

Ragedaug commented 5 months ago

Milling seems to work, but enchanting is still hit or miss. I have a level 80 mage. All the level 78-80 items in my bag glow when I mouse over and press my Alt key, but the Level 81 greens do not glow.

image image

p3lim commented 5 months ago

@Ragedaug Can you show me the same items(ish) tooltips while you are casting the Disenchanting spell? As in while your cursor is glowing. This will show additional tooltip information.

From what I can tell those "brightwood" belts require 425 enchanting to disenchant, and if your skill is lower than that then Molinari won't activate.

Ragedaug commented 5 months ago

@p3lim The only change I see in the tooltip is: "Disenchanting requires Enchanting (425)" Both the Level 78 green and the 81 green show the 425 requirement. My enchanting skill is 452.

I also disabled all other addons except Molinari to ensure nothing else was causing a conflict. Here's what I see, on the 81 item that will not glow when I press Alt and mouse over it, when I select Disenchant, then mouse over the item. image

p3lim commented 5 months ago

I see the problem, I'll work on a fix when I have the time. In the mean time I'm gonna remove the restriction checker in Molinari and push a release.

Ragedaug commented 5 months ago

.113 shows the glow! Thank you @p3lim !