p3lim-wow / molinari

One-click Milling, Prospecting, Disenchanting and Lockpicking!

Molinari

This is an addon for the popular MMORPG "World of Warcraft".
It can be downloaded at one of the following sites:

• CurseForge
• WoWInterface
• Wago
• GitHub

Description

Molinari allows the player to easily "salvage" items directly from the inventory by holding down the Alt key and clicking on the item in their bags, instead of dealing with the tradeskill UI, spellbook, macros or actions.

The following "salvaging" is supported:

• Milling
• Prospecting
• Disenchanting
• Lockpicking
  • Skeleton Pinkie (Mechagnome racial)
  • Kevin's Keyring (Necrolord soulbind ability)
  • Blacksmithing picks
  • Jewelcrafting picks
  • Inscription picks
  • Engineering picks
• Scrapping
• Crushing

In addition, Molinari also works for lockpicking in the trade window.

Translations

If you want to help translate Molinari, please use this form on GitHub:
https://github.com/p3lim-wow/Molinari/issues/new?template=translate.yaml

Feedback & issues

If you have a question, please use the discussion section on GitHub.
If you would like to report a bug or contribute to the project, please visit GitHub:
https://github.com/p3lim-wow/Molinari/issues/new/choose

Legal

Please see the LICENSE file.

molinari's Issues

Whitelist Feature

I was wondering if you could add a whitelist function that serves the opposite function of the blacklist. Forces an action on an item. For instance, there are several profession question items that are not supported by this addon, Specifically ones like ID 137195 (Highmountain Armor) quest green for enchanting quest.

I run multiple Enchanters so I've gone through these types of quests several times, I would be nice if I can add a item to a whitelist to force the disenchant. Configuration could be as simple as holding alt and right clicking an item to record the item ID and checking for professions to figure out if it's disenchant, mill or prospecting. (open a dialog to have the user confirm if they have more than one these professions)

Anyways, it would be nice but I thank you for your wonderful work here. I find this addon a must have now days.

Disenchant of "special" Enchanter Items in Dragonflight does not work

Disenchant of "special" Enchanter Items in Dragonflight does not work, means Molinari does not appear on it.

Example Items are:
Chromatic Pocketwatch
Seal of Order
Carving of Awakening
Valiant Hammer
and many more it seems!

Classic not working for enchanting since update

Since tuesdays classic update Molinari no longer works. My modifier key is alt. when I press alt and hover over an item to disenchant the item no longer has the sparkle effect and clicking on an item doesn't do anything, I don't have any other professions to test with to see if the issue is the same on other characters.

When I first login to the game this error happens

Date: 2019-12-12 04:53:29
ID: 1
Error occured in: Global
Count: 2
Message: ...dOns\Molinari\libs\LibProcessable\LibProcessable.lua line 264:
attempt to call global 'GetProfessions' (a nil value)
Debug:
[C]: GetProfessions()
...dOns\Molinari\libs\LibProcessable\LibProcessable.lua:264:
...dOns\Molinari\libs\LibProcessable\LibProcessable.lua:250
Locals:

AddOns:
MoncaiCompare, v7.2.0
Swatter, v8.2.6377 (SwimmingSeadragon)
ACP, v3.5.6
AdiBags, vv1.9.17-classic
AdvancedInterfaceOptions, v1.3.3-3-g58f2fa4
AlreadyKnown, v1.22
Armory, vv1.4
Atlas, vv1.45.01.11302
AtlasClassicWoW, vv1.45.00.11302.05
AtlasLootClassic, vv1.4.2
AtlasLootClassicData, vv1.4.2
AtlasLootClassicDungeonsAndRaids, vv1.4.2
AtlasLootClassicMaps, vv1.0.0
AtlasQuest, v4.11.58
ATTClassic, v0.1.2
AucAdvanced, v8.2.6430 (SwimmingSeadragon)
AucFilterBasic, v8.2.6364 (SwimmingSeadragon)
AucStatHistogram, v8.2.6366 (SwimmingSeadragon)
AucStatiLevel, v8.2.6370 (SwimmingSeadragon)
AucStatPurchased, v8.2.6367 (SwimmingSeadragon)
AucStatSimple, v8.2.6399 (SwimmingSeadragon)
AucStatStdDev, v8.2.6369 (SwimmingSeadragon)
AucUtilFixAH, v8.2.6371 (SwimmingSeadragon)
Auctionator, v8.1.0
AutoAmmo, v1.1.3
AutoBiographer, v0.3.6
AutoDismount, v1.0.3
AutolootEnable, v1.0
AutoTrackQuests, v1.2
BankStack, vv11302.0
Bartender4, v4.8.8
BasicChatMods, vv4-classic
BeanCounter, v8.2.6434 (SwimmingSeadragon)
BetterAutoLoot, v1.0.4
BetterVendorPrice, vv1.11.00
BlizzMove, v1.9
BootyBayGazette, v6.1.20191209
buffOmat, v1.11
ClassicSpellActivations, v
Cleanup, v
ColoredInventoryItems, v1.4
ComboWhore, v1.1
DBMCore, v1.13.24
DBMDefaultSkin, v
DBMStatusBarTimers, v
DejaClassicStats, v1302r039
DTweaks, v1.4
DungeonFinder, v0.1.7
Enchantrix, v8.2.6428 (SwimmingSeadragon)
EnchantrixBarker, v8.2.6469 (SwimmingSeadragon)
FieldGuide, vv1.5-release
FleecingTip, v113.1475(011019)
GatherMate2, v1.45.5
GigaAutoResourcesTracker, v0.0.1
GuildTooltip, v2.4.5
HandyNotes, vv1.5.9-1-g81432ff
HandyNotesNPCsClassic, v1.21
HandyNotesTravelGuideClassic, vv2.1.6
HideQuestTracker, v1.0
Immersion, v1.2.2
Informant, v8.2.6374 (SwimmingSeadragon)
ItemRack, v
LeatrixMaps, v1.13.39
LeatrixPlus, v1.13.40
LockboxCracker, v1.13.2 build 2
MacroMakerclassic, v1.5
MageButtons, v1.01
MissingTradeSkillsList, v1.13.27
Molinari, v80200.63-Release
MonkeyBuddy, v
MonkeyLibrary, v
MonkeyQuest, v
MonkeyQuestLog, v
NeatPlates, v1.13.2.49
NeatPlatesAlvara, v5a
NeatPlatesBlizzardPlates, v1.5
NeatPlatesClassicPlates, v
NeatPlatesGraphite, v
NeatPlatesGrey, v
NeatPlatesNeon, v
NeatPlatesQuatre, v
NeatPlatesRenaitre, vv7.0.3
NeatPlatesRoth, v
NeatPlatesSimple, v7.3.0
NeatPlatesSlimHorizontal, v2.2
NeatPlatesSlimVertical, v2.2
NeatPlatesHub, v
NeatPlatesWidgets, v
OmniCC, v8.2.5
PeaceAndQuiet, v8.2.0
Puggle, v2.8
Questie, v5.3.1
QuestXP, v0.3
Recount, vv1.13.3a
SilverDragon, vv11302.0
SilverDragonClassic, vv11302.0
SLFG, v1.5
SlideBar, v8.2.6375 (SwimmingSeadragon)
SLoTe, v
SortBags, v0.1.3
SpartanUI, v5.2.13
SpartanUIArtwork, v5.2.13
SpartanUIFilmEffects, v5.2.13
SpartanUIPartyFrames, v5.2.13
SpartanUIPlayerFrames, v5.2.13
SpartanUIRaidFrames, v5.2.13
SpartanUISpinCam, v5.2.13
SpartanUIStyleFel, v5.2.13
SpartanUIStyleMinimal, v5.2.13
SpartanUIStyleTransparent, v5.2.13
SpartanUIStyleWar, v5.2.13
StickyTarget, v
Stubby, v8.2.6376 (SwimmingSeadragon)
TFTB, v1.10
TitanClassic, v1.1.4.11302
TitanClassicAmmo, v1.1.4.11302
TitanClassicBag, v1.1.4.11302
TitanClassicClock, v1.1.4.11302
TitanClassicGold, v1.1.4.11302
TitanClassicItemDed, v1.0.0.3
TitanClassicLocation, v1.1.4.11302
TitanClassicLootType, v1.1.4.11302
TitanClassicPerformance, v1.1.4.11302
TitanClassicRegen, v1.1.4.11302
TitanClassicRepair, v1.1.4.11302
TitanClassicVolume, v1.1.4.11302
TitanClassicXP, v1.1.4.11302
TomPoints, v11302_05
TomTom, vv11302-1.0.8
TradeSkillMaster, vv4.8.23
TradeSkillMasterAppHelper, vv4.0.5
WeakAuras, v2.15.5
BlizRuntimeLib_enUS v1.13.3.11303
(ck=ecb)

It appears that the issue occurs when Molinari is running with auctioneer at the same time. If I disable auctioneer then the error message goes away. Any of Auctioneers suite enabled will cause the error to happen, even if the main addon is disabled but the libraries loaded such as swatter or stubby the error message will keep happening.

Add support for Draenic Mortar (Garrison scribe milling)

When you are not a scribe, but have the scribe building in your garrison you get an item (Draenic Mortar http://www.wowhead.com/item=114942/draenic-mortar). Having support for alt-click milling with this item as well would be great.

Support disenchanting profession equipment

Ref #80

Molinari.lua:290: attempt to call method 'IsScrappable' (a nil value)

Describe the bug
Version 100005.96. Trying to prospect DF ore on my Jewelcrafter causes an error.

How to reproduce
ALT+Left clicking ore stack to manually prospect.

What did you expect?
No error to occur.

Screenshots
41x Molinari/Molinari.lua:290: attempt to call method 'IsScrappable' (a nil value)
[string "@Molinari/Molinari.lua"]:290: in function <Molinari/Molinari.lua:239>
[string "@Molinari/Molinari.lua"]:306: in function <Molinari/Molinari.lua:298>
[string "=(tail call)"]: ?
[string "=[C]"]: in function securecallfunction' [string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:60: in function <SharedXML/Tooltip/TooltipDataHandler.lua:55> [string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:80: in function <SharedXML/Tooltip/TooltipDataHandler.lua:76> [string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:116: in function <SharedXML/Tooltip/TooltipDataHandler.lua:106> [string "=[C]"]: in function SetAttribute'
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:141: in function <SharedXML/Tooltip/TooltipDataHandler.lua:134>
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:278: in function <SharedXML/Tooltip/TooltipDataHandler.lua:236>
[string "=[C]"]: in function securecallfunction' [string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:233: in function <SharedXML/Tooltip/TooltipDataHandler.lua:232> [string "=(tail call)"]: ? [string "=[C]"]: ? [string "=[C]"]: in function SetInventoryItem'
[string "@FrameXML/BankFrame.lua"]:62: in function <FrameXML/BankFrame.lua:59>

Locals:
itemLink = "|cff1eff00|Hitem:1206::::::::70:577:::::::::|h[Moss Agate]|h|r"
itemID = 1206
isMillable = nil
millingSpellID = nil
useMortar = nil
isProspectable = nil
prospectingSpellID = nil
isOpenable = nil
spellID = nil
isOpenableProfession = nil
keyItemID = nil
(*temporary) = nil
(*temporary) =

{
IsProspectable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:105
GetProfessionSkillLines = defined @Molinari/libs/LibProcessable/LibProcessable.lua:396
IsOpenableProfession = defined @Molinari/libs/LibProcessable/LibProcessable.lua:299
IsMillable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:65
HasProfession = defined @Molinari/libs/LibProcessable/LibProcessable.lua:369
IsOpenable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:269
GetProfessionCategories = defined @Molinari/libs/LibProcessable/LibProcessable.lua:382
IsDisenchantable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:142
}
(*temporary) = 1206
(*temporary) = "attempt to call method 'IsScrappable' (a nil value)"
shouldActivate = defined @Molinari/Molinari.lua:215
addon = {
CreateButtonPool = defined @Molinari/config/ButtonPoolMixin.lua:123
db = {
}
L = {
}
eventMixin = {
}
CreateBlocklistOptions = defined @Molinari/config/OptionsBlocklist.lua:155
}
LibProcessable = {
IsProspectable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:105
GetProfessionSkillLines = defined @Molinari/libs/LibProcessable/LibProcessable.lua:396
IsOpenableProfession = defined @Molinari/libs/LibProcessable/LibProcessable.lua:299
IsMillable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:65
HasProfession = defined @Molinari/libs/LibProcessable/LibProcessable.lua:369
IsOpenable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:269
GetProfessionCategories = defined @Molinari/libs/LibProcessable/LibProcessable.lua:382
IsDisenchantable = defined @Molinari/libs/LibProcessable/LibProcessable.lua:142
}
Molinari = Molinari {
CalculateAction = defined @FrameXML/SecureTemplates.lua:609
ApplyTradeSkill = defined @Molinari/Molinari.lua:70
0 =
ApplySpell = defined @Molinari/Molinari.lua:22
RegisterEvent = defined @Molinari/mixins/event.lua:7
IsEventRegistered = defined @Molinari/mixins/event.lua:39
GetModifier = defined @Molinari/Molinari.lua:136
UpdateStateDriver = defined @Molinari/Molinari.lua:201
SetGlowColor = defined @Molinari/Molinari.lua:128
Attach = defined @Molinari/Molinari.lua:123
GetBagAndSlotID = defined @Molinari/Molinari.lua:93
ApplyItem = defined @Molinari/Molinari.lua:47
GetGlowColor = defined @Molinari/Molinari.lua:132
sparkles = {
}
UnregisterEvent = defined @Molinari/mixins/event.lua:22
GetModifierCondition = defined @Molinari/Molinari.lua:148
}
CLASSIC = false

Disenchanting Two-handed maces

I noticed it was fixed for the retail version but the issue still exists in the TBC classic version.
Running this version

Feature Request

Thank you for combining Dolemite and Molinari.

I would like to request a feature. I enjoyed the "look" of Dolemite when I pressed the hotkey over an item. However, in Molinari, all I get is a yellow square. I was wondering if we could have the option to have the other look.

Thanks in advance...

Not working in Wrath classic

Describe the bug
Tried all the options to show the milling button but doesnt work. Tried it with release 84 and 83.
How to reproduce
Holding modifier key over northrend herbs but nothing happens.

What did you expect?
To work as intented

Screenshots

modifier option occurs error in classic era

Describe the bug
when I tired to change modifer options, lua error occurs

How to reproduce
whenever change the modifier option

What did you expect?
modifier should be changed without error.

Screenshots

i searched all the lua files and there is no more UpdateStateDriver func. so i changed _G.Molinari:UpdateStateDriver() to _G.Molinari:UpdateAttributeDriver() and it's fine.

8x Molinari/api/items.lua:46: attempt to index field 'disenchantable' (a nil value)

WOTLK ==
Prospecting and Milling working well..
But when I try to disenchant the green item by ALT+Clicking, LUA error occurs.

8x Molinari/api/items.lua:46: attempt to index field 'disenchantable' (a nil value)
[string "@Molinari/api/items.lua"]:46: in function IsDisenchantable' [string "@Molinari/api/items.lua"]:101: in function IsSalvagable'
[string "@Molinari/addon.lua"]:33: in function <Molinari/addon.lua:8>
[string "@Molinari/utils.lua"]:44: in function <Molinari/utils.lua:41>
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "=[C]"]: ?
[string "@Bistooltip/lib/LibExtraTip/LibExtraTip.lua"]:408: in function SetBagItem' [string "@FrameXML/ContainerFrame_Shared.lua"]:1362: in function <FrameXML/ContainerFrame_Shared.lua:1333> [string "=[C]"]: in function UpdateTooltip'
[string "@FrameXML/GameTooltip.lua"]:444: in function <FrameXML/GameTooltip.lua:423>

Locals:
self =

{
eventMixin = {
}
IsClassicTBC = defined @Molinari/libs/Dashi/modules/misc.lua:12
IsRetail = defined @Molinari/libs/Dashi/modules/misc.lua:4
IsEventRegistered = defined @Molinari/libs/Dashi/modules/event.lua:76
TriggerCombatEvent = defined @Molinari/libs/Dashi/modules/event.lua:219
Defer = defined @Molinari/libs/Dashi/modules/defer.lua:51
GetItemLinkFromID = defined @Molinari/libs/Dashi/modules/misc.lua:62
GetUnitAura = defined @Molinari/libs/Dashi/modules/misc.lua:108
GetNPCID = defined @Molinari/libs/Dashi/modules/misc.lua:53
CreateButton = defined @Molinari/libs/Dashi/modules/widgets.lua:19
UnregisterUnitEvent = defined @Molinari/libs/Dashi/modules/event.lua:150
TriggerEvent = defined @Molinari/libs/Dashi/modules/event.lua:89
Printf = defined @Molinari/libs/Dashi/modules/output.lua:14
CreateBlocklistOptions = defined @Molinari/config/OptionsBlocklist.lua:146
HookTooltip = defined @Molinari/utils.lua:32
IsClassicWrath = defined @Molinari/libs/Dashi/modules/misc.lua:16
UnregisterCombatEvent = defined @Molinari/libs/Dashi/modules/event.lua:242
tsize = defined @Molinari/libs/Dashi/modules/misc.lua:78
IsUnitEventRegistered = defined @Molinari/libs/Dashi/modules/event.lua:175
CreateButtonPool = defined @Molinari/config/ButtonPoolMixin.lua:123
SetPixelScale = defined @Molinari/libs/Dashi/modules/misc.lua:72
GetProfessionSkillLevel = defined @Molinari/api/skill.lua:59
IsOpenableProfession = defined @Molinari/api/items.lua:111
AddBackdrop = defined @Molinari/libs/Dashi/modules/widgets.lua:42
ExtractIDFromGUID = defined @Molinari/libs/Dashi/modules/misc.lua:48
IsDisenchantable = defined @Molinari/api/items.lua:43
IsScrappable = defined @Molinari/api/items.lua:35
Print = defined @Molinari/libs/Dashi/modules/output.lua:3
IsCrushable = defined @Molinari/api/items.lua:27
Hide = defined @Molinari/libs/Dashi/modules/misc.lua:24
IsProspectable = defined @Molinari/api/items.lua:5
data = {
}
db = {
}
RegisterUnitEvent = defined @Molinari/libs/Dashi/modules/event.lua:118
GetBagAndSlotID = defined @Molinari/utils.lua:3
GetPlayerPosition = defined @Molinari/libs/Dashi/modules/misc.lua:118
HookAddOn = defined @Molinari/libs/Dashi/modules/load.lua:4
HookSettings = defined @Molinari/libs/Dashi/modules/settings.lua:12
OpenSettings = defined @Molinari/libs/Dashi/modules/settings.lua:3
IsSalvagable = defined @Molinari/api/items.lua:82
RegisterEvent = defined @Molinari/libs/Dashi/modules/event.lua:40
RegisterSlash = defined @Molinari/libs/Dashi/modules/input.lua:3
L = {
}
TriggerUnitEvent = defined @Molinari/libs/Dashi/modules/event.lua:195
Dump = defined @Molinari/libs/Dashi/modules/output.lua:18
IsClassic = defined @Molinari/libs/Dashi/modules/misc.lua:8
colors = {
}
IsMillable = defined @Molinari/api/items.lua:16
RegisterCombatEvent = defined

Feature Request: Cloth Bolts

Is it possible to add the ability to create cloth bolts from cloth? I'd be happy to help, but I'd need a primer on where to start in the codebase.

Lua error [Classic ERA]

How to reproduce
at every login

2x ...dOns\Molinari\libs\LibProcessable\LibProcessable-49.lua:316: table index is nil
...dOns\Molinari\libs\LibProcessable\LibProcessable-49.lua:316: in main chunk

Locals:
MAJOR = "LibProcessable"
MINOR = 49
lib =

{
IsProspectable = defined @molinari\libs\LibProcessable\LibProcessable.lua:59
IsOpenableProfession = defined @molinari\libs\LibProcessable\LibProcessable.lua:223
IsMillable = defined @molinari\libs\LibProcessable\LibProcessable.lua:31
HasProfession = defined @molinari\libs\LibProcessable\LibProcessable.lua:288
IsOpenable = defined @molinari\libs\LibProcessable\LibProcessable.lua:192
GetProfessionCategories = defined @molinari\libs\LibProcessable\LibProcessable.lua:301
IsDisenchantable = defined @molinari\libs\LibProcessable\LibProcessable.lua:84
}
oldMinor = nil
CLASSIC = true
LE_ITEM_QUALITY_UNCOMMON = 2
LE_ITEM_QUALITY_EPIC = 4
LE_ITEM_CLASS_ARMOR = 4
LE_ITEM_CLASS_WEAPON = 2
LE_ITEM_CLASS_GEM = 3
LE_ITEM_ARMOR_COSMETIC = 5
LE_ITEM_SUBCLASS_ARTIFACT = 11
LE_ITEM_EQUIPLOC_SHIRT = 4
professions = {
}
GetBlacksmithingPick = defined @molinari\libs\LibProcessable\LibProcessable.lua:107
GetJewelcraftingPick = defined @molinari\libs\LibProcessable\LibProcessable.lua:156
GetInscriptionPick = defined @molinari\libs\LibProcessable\LibProcessable.lua:164
GetEngineeringPick = defined @molinari\libs\LibProcessable\LibProcessable.lua:174
(*temporary) = {
Tailoring = 197
Blacksmithing = 164
Skinning = 393
Alchemy = 171
Mining = 186
Engineering = 202
Leatherworking = 165
Herbalism = 182
Enchanting = 333
}
(*temporary) = nil
(*temporary) = 25229
(*temporary) = "Tailoring"
(*temporary) = nil
(*temporary) = 136249
(*temporary) = 0
(*temporary) = 0
(*temporary) = 0
(*temporary) = 3908
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "table index is nil"

Not working with ArkInventory?

• What version are you using? 80000.58-Release

Not 100% sure if this is a Molinari bug or with ArkInventory. After the patch yesterday, this addon is no longer working with ArkInventory loaded. I unload it and it works. I use another version of AI that this WAS working with just fine prior to the patch and it still doesn't work. :(

Lua Error on Startup or Reload

Describe the bug
On loading into WoW Classic the game puts up 4 Lua Errors.

How to reproduce
Login or Reload the game.

What did you expect?
For it to load without errors.

Screenshots

Global variable 'disenchanting'

In line 57, disenchanting is set as a global variable.
The local declaration before that line only accounts for 'disenchanter'.

Cheers

• ckaotik

Support mass milling

6.2 added new "Mass Mill" spells for Draenor herbs that mill 20 herbs at a time instead of only 5. It would be nice if Molinari supported these to avoid clicking 4x as much as is really needed.

http://www.wowhead.com/search?q=mass+mill#professions

Disenchanted wrong item!

I attempted to disenchant a world quest reward I didn't need. Used the molinari click shortcut (I use Shift-Alt-Left Click for this) and instead of DEing the quest reward, it disenchanted my offspec main hand weapon from heroic Vault! The original world quest drop was still there so I thought it just hadn't worked, did it again and now I have no offhand (got them back via item restore).

I used the regular in-game method to DE it (click disenchant, click item) and it worked as expected.

Feature Request: use other key

Hi p3lim, Thank you for making this great addon. Could you please add an option to choose key other than Alt? Because my bag addon already uses Alt+clicking for customize categories. It will be very very helpful If I can use key like Ctrl+Alt+clicking or Alt+right clicking to process the items.

Feature Request

Hello.
First of all, thank you for making this simple working, yet awesome add-on.

The feature here is just a simple Milling/Prospecting/Disenchant result odds in the item tooltip.
Something like that:

Icethorn
Millable

Milled: 150
300% Azure Pigment (450)
66% Icy Pigment (100)

An add-on which does a similar action is: http://www.curse.com/addons/wow/prospect-me

If this request come to be developed, then Molinari would do all the things I am looking for.
Thanks in advance.

[New Feature] Support for feeding Pets via ALT+Click in Bag

Hi there,

first of all thank you for this addon. Im using this for my classic rogue to open locks in my backpack and it saves me so much time.

Recently i had an idea since i rolled a hunter. I would like to ask if you could add something like isFeedableFood to the modifier section (when pressing alt) like isOpenable already is for rogues.

Im really tired of clicking "feed pet" -> open my backpack -> click on food. I would like to shorten this down to "alt+click". I believe others would agree since afaik there is currently no such addon that allows hunters to feed pet easier.

For that I quickly implemented such feature without any checking via LibProcessable since i was just testing if this is doable.

Molinari.lua:~L139 (After isOpenable check)

local IsFeedableFood, spellID = LibProcessable:IsFeedableFood(itemID)
if (IsFeedableFood) then
    Molinari:Apply(itemLink, spellID, 1, 0, 0)
end

LibProcessable.lua

function lib:IsFeedableFood(itemID)
	if(type(itemID) == 'string') then
		assert(string.match(itemID, 'item:(%d+):') or tonumber(itemID), 'item must be an item ID or item Link')
		itemID = (tonumber(itemID)) or (GetItemInfoFromHyperlink(itemID))
	end

	local name, link, quality, level, minLevel, type, subType, stackCount, equipLoc, icon, sellPrice, classID, subclassID, bindType, expacID, setID, isCraftingReagent = GetItemInfo(itemID)
	print("level", level, "type:", type, "subType:", subType, "classID:subclassID", classID, subclassID, "isConsumable", IsConsumableItem(itemID))

	if(classID== 0) then
		return true, 6991
	end
end

Ignore the print and the ultra long GetItemInfo var assignment in the function since i was trying to find a way to detect feedable food via subclass but it always is consumable like classID ..

I think if LibProcessable would have some itemID list like lib.herbs but for food i.e. Meat like Tough Jerky and maybe detect other sources via Profession Cooking to filter out other consumables like Bandages would be good?

In any case there is probably a cleaner solution available for this but for now this is working just fine for me.

I chose the red color because it can be eaten ;)

Thoughts about further improvement
If this is done via LibProcessable .. some could also check for level of food and check if its in range of the pet level .. color of the border when pressing alt could be variance of chosen color (here it was red) to visualize the quality of the food you are about to feed without checking action log. This would be a cosmetic feature i guess but maybe can add later if you like the idea?

What do you think?

Best wishes,
Lycano

Broken in 10.0.2

Errors showing on logging in:

"4x Interface/AddOns/Molinari/Molinari.lua:241: GameTooltip doesn't have a "OnTooltipSetItem" script
[string "=[C]"]: in function `HookScript'
[string "@Interface/AddOns/Molinari/Molinari.lua"]:241: in main chunk

Locals:
(*temporary) = GameTooltip {
 0 = <userdata>
 SetQuestLogRewardSpell = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetQuestPartyProgress = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 ProcessLineData = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:303
 SetPvpTalent = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetRuneforgeResultItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetHyperlink = <function> defined =[C]:-1
 SetUnit = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetAzeriteEssence = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetVoidDepositItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetVoidWithdrawalItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 ClearLines = <function> defined =[C]:-1
 SetInfoBackdropStyle = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:338
 ItemTooltip = Frame {
 }
 shoppingTooltips = <table> {
 }
 SetSocketGem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 updateTooltipTimer = 0.200000
 Show = <function> defined =[C]:-1
 StatusBar = GameTooltipStatusBar {
 }
 SetLootCurrency = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetSlottedKeystone = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetWorldCursor = <function> defined @Interface/FrameXML/GameTooltip.lua:952
 SetLootItem = <function> defined =[C]:-1
 SetMerchantCostItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 GetUnit = <function> defined @Interface/FrameXML/GameTooltip.lua:994
 SetOwnedItemByID = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetItemInteractionItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetQuestLogSpecialItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 TextRight1 = GameTooltipTextRight1 {
 }
 SetEnhancedConduit = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetRecipeReagentItem = <function> defined =[C]:-1
 SetQuestCurrency = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 supportsDataRefresh = true
 SetWeeklyReward = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetMerchantItem = <function> defined =[C]:-1
 SetPvpBrawl = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetAchievementByID = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 BottomOverlay = Texture {
 }
 SetSpellByID = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 supportsItemComparison = true
 SetSocketedRelic = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 ProcessInfo = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:236
 ProcessLines = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:293
 NineSlice = Frame {
 }
 SetInventoryItem = <function> defined =[C]:-1
 RefreshData = <function> defined @Interface/FrameXML/GameTooltip.lua:933
 SetTransmogrifyItem = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetLootRollItem = <function> defined =[C]:-1
 OnLoad = <function> defined @Interface/FrameXML/GameTooltip.lua:927
 SetQuestLogCurrency = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetUnitDebuffByAuraInstanceID = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetBagItem = <function> defined =[C]:-1
 GetSpell = <function> defined @Interface/FrameXML/GameTooltip.lua:990
 SetAction = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetItemKey = <function> defined @Interface/SharedXML/Tooltip/TooltipDataHandler.lua:444
 SetCompanionPet = <function> defined @Interface/Sh"

"4x FrameXML\Bindings.xml:1 Interface/AddOns/Molinari/Molinari.lua:1 (null)"

"4x Interface/AddOns/Molinari/Molinari.lua:1 (null)"

Please update to latest .toc

What version are you using? (don't say "latest" or "newest").
60000.35 (hope i got the correct number of zeros)

Can you reproduce it, and if you can, how?

If there was an error message, include it.
I keep getting the eror in my chat frame approximately every two or three minutes.
"There are too many errors in your IU. As a result, your game experience may be degraded. Diable or update the failing addons if you don't want to see this message again."

If it is a graphical issue, include a screenshot (use imgur or something)

Classic Error Upon Logging In

If you're opening a feature request, please search through the existing issues first.
If you're submitting an issue, please fill in the following:

• What version are you using?
  Molinari-80200.63-Release-classic.zip
• Can the issue be reproduced, and if so, how?
  _ it happens when I log in_
• If there was an error message, please include it.
  7x ...dOns\Molinari\libs\LibProcessable\LibProcessable-37.lua:264: attempt to call global 'GetProfessions' (a nil value)
  ...dOns\Molinari\libs\LibProcessable\LibProcessable-37.lua:264: in function <...dOns\Molinari\libs\LibProcessable\LibProcessable.lua:250>

Locals:
self = {
0 =
}
event = "SKILL_LINES_CHANGED"
(*temporary) = nil
(*temporary) = "attempt to call global 'GetProfessions' (a nil value)"
professions =

{
}
CLASSIC = false
classicIDs = {
Tailoring = 197
Blacksmithing = 164
Skinning = 393
Alchemy = 171
Mining = 186
Engineering = 202
Leatherworking = 165
Herbalism = 182
Enchanting = 333
}

• If there is a graphical issue, please include a screenshot (you can drag-n-drop here).
  none
  Make sure you check back often until the issue is resolved for any follow-up questions.

3x Lua Errors with pre-patch

5x Interface/AddOns/Molinari/config/Options.lua:55: attempt to index global 'InterfaceOptionsFrameAddOns' (a nil value)
[string "@Interface/AddOns/Molinari/config/Options.lua"]:55: in main chunk

Locals:
addonName = "Molinari"
ns =

{
CreateButtonPool = defined @Interface/AddOns/Molinari/config/ButtonPoolMixin.lua:123
L = {
}
}
L = {
}
UpdateOptions = defined @Interface/AddOns/Molinari/config/Options.lua:4
CreateOptions = defined @Interface/AddOns/Molinari/config/Options.lua:10
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = nil
(*temporary) = "attempt to index global 'InterfaceOptionsFrameAddOns' (a nil value)"

---

1x Interface/AddOns/Molinari/config/Options.lua:1 (null)

---

1x FrameXML\Bindings.xml:1 Interface/AddOns/Molinari/config/Options.lua:1 (null)

March 3rd update is the only version that works with Classic Era.

As the title says.

every time i've updated molinari it stops working so i've had to manually download the March 3rd update just to use it.

OnClick passthrough targeting wrong frame

Needs to pass parent instead of Molinari.

Lua error

I'm using the newest version of your addon, and keep getting a lua error. The addon seems to work properly, but wanted to make you aware. The error:

1x [ADDON_ACTION_BLOCKED] AddOn 'Molinari' tried to call the protected function 'SecureStateDriverManager:SetAttribute()'.
!BugGrabber\BugGrabber.lua:519: in function <!BugGrabber\BugGrabber.lua:519>
[C]: in function SetAttribute' FrameXML\SecureStateDriver.lua:10: in function <FrameXML\SecureStateDriver.lua:8> (tail call): ? Molinari\Molinari-80000.58-Release.lua:98: in function UpdateModifier'
Molinari\Config.lua:27: in function callback' Molinari\libs\Wasabi\Wasabi-6.lua:327: in function Fire'
Molinari\libs\Wasabi\Wasabi-6.lua:160: in function <Molinari\libs\Wasabi\Wasabi.lua:153>
[C]: in function pcall' FrameXML\InterfaceOptionsFrame.lua:209: in function <FrameXML\InterfaceOptionsFrame.lua:208> [C]: in function securecall'
FrameXML\InterfaceOptionsFrame.lua:232: in function <FrameXML\InterfaceOptionsFrame.lua:224>

Locals:
InCombatSkipped

Thanks, and great addon.

Thorium Lockboxes

Unable to target and open Thorium Lockboxes as a rogue. All other lockboxes work so far.

Translation:

Which locale are these translations for?

ruRU (Russian)

"Modifier to activate %s"

No response

"Item Blocklist"

Блок-список предметов

"Block Item"

Заблокированный предмет

"Items in this list will not be processed."

Предметы из этого списка не будут обрабатываться.

"Block a new item by ID"

Заблокировать новый предмет по ID

LUA Error, ALT + Mouseover "processable" Item in Chat -> attempt to call method 'GetID' (a nil value)

Describe the bug
See Title

How to reproduce
ALT + Mouseover "processable" Item in Chat

Lua Error

6x Molinari/Molinari.lua:118: attempt to call method 'GetID' (a nil value)
[string "@Molinari/Molinari.lua"]:118: in function `GetBagAndSlotID'
[string "@Molinari/Molinari.lua"]:23: in function `ApplySpell'
[string "@Molinari/Molinari.lua"]:277: in function <Molinari/Molinari.lua:242>
[string "@Molinari/Molinari.lua"]:303: in function <Molinari/Molinari.lua:295>
[string "=(tail call)"]: ?
[string "=[C]"]: in function `securecallfunction'
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:60: in function <SharedXML/Tooltip/TooltipDataHandler.lua:55>
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:80: in function <SharedXML/Tooltip/TooltipDataHandler.lua:76>
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:116: in function <SharedXML/Tooltip/TooltipDataHandler.lua:106>
[string "=[C]"]: in function `SetAttribute'
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:141: in function <SharedXML/Tooltip/TooltipDataHandler.lua:134>
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:282: in function <SharedXML/Tooltip/TooltipDataHandler.lua:240>
[string "=[C]"]: in function `securecallfunction'
[string "@SharedXML/Tooltip/TooltipDataHandler.lua"]:237: in function <SharedXML/Tooltip/TooltipDataHandler.lua:236>
[string "=(tail call)"]: ?
[string "@TradeSkillMaster/LibTSM/Service/ItemTooltipClasses/Wrapper.lua"]:111: in function <...Master/LibTSM/Service/ItemTooltipClasses/Wrapper.lua:109>
[string "=[C]"]: in function `SetHyperlink'
[string "@ElvUI/Core/Modules/Chat/Chat.lua"]:1528: in function `?'
[string "@Ace3/AceHook-3.0-9/AceHook-3.0.lua"]:90: in function <Ace3/AceHook-3.0/AceHook-3.0.lua:87>

Locals:
self = Molinari {
 CalculateAction = <function> defined @FrameXML/SecureTemplates.lua:609
 ApplyTradeSkill = <function> defined @Molinari/Molinari.lua:74
 0 = <userdata>
 ApplySpell = <function> defined @Molinari/Molinari.lua:22
 RegisterEvent = <function> defined @Molinari/mixins/event.lua:7
 IsEventRegistered = <function> defined @Molinari/mixins/event.lua:39
 GetModifier = <function> defined @Molinari/Molinari.lua:139
 UpdateStateDriver = <function> defined @Molinari/Molinari.lua:204
 SetGlowColor = <function> defined @Molinari/Molinari.lua:131
 Attach = <function> defined @Molinari/Molinari.lua:126
 GetBagAndSlotID = <function> defined @Molinari/Molinari.lua:101
 ApplyItem = <function> defined @Molinari/Molinari.lua:47
 GetGlowColor = <function> defined @Molinari/Molinari.lua:135
 sparkles = <table> {
 }
 UnregisterEvent = <function> defined @Molinari/mixins/event.lua:22
 GetModifierCondition = <function> defined @Molinari/Molinari.lua:151
}
parent = FontString {
 0 = <userdata>
 messageInfo = <table> {
 }
}
bagID = 0
slotID = nil
(*temporary) = nil
(*temporary) = FontString {
 0 = <userdata>
 messageInfo = <table> {
 }
}
(*temporary) = "attempt to call method 'GetID' (a nil value)"

utilizing alt does not activate the addon. I have checked my keybindings in the addon and it is set to "alt"

Describe the bug

How to reproduce

What did you expect?

Screenshots

Item Equip when loot window is open

It seems that alt-left clicking on an item when the loot window is open (which happens occasionally when disenchanting lots of items due to blizzards autoloot bug) causes the selected item to be swapped with the characters currently equipped item.

This is a MAJOR problem - I already lost an item before noticing my item was swapped.

Error

Message: [ADDON_ACTION_BLOCKED] AddOn 'Molinari' tried to call the protected function 'SecureStateDriverManager:SetAttribute()'.
Time: Wed Aug 12 06:00:16 2020
Count: 1
Stack: [ADDON_ACTION_BLOCKED] AddOn 'Molinari' tried to call the protected function 'SecureStateDriverManager:SetAttribute()'.
Interface\FrameXML\SecureStateDriver.lua:10: in function <Interface\FrameXML\SecureStateDriver.lua:8>
(tail call): ?
Interface\AddOns\Molinari\Molinari.lua:99: in function `UpdateModifier'
Interface\AddOns\Molinari\Config.lua:27: in function `callback'
Interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua:329: in function `Fire'
Interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua:162: in function <Interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua:155>
[C]: in function `pcall'
Interface\FrameXML\InterfaceOptionsFrame.lua:209: in function <Interface\FrameXML\InterfaceOptionsFrame.lua:208>
[C]: in function `securecall'
Interface\FrameXML\InterfaceOptionsFrame.lua:232: in function <Interface\FrameXML\InterfaceOptionsFrame.lua:224>
[C]: in function `Click'
...\AddOns\RCLootCouncil_Classic\RCLootCouncil\core.lua:1113: in function <...\AddOns\RCLootCouncil_Classic\RCLootCouncil\core.lua:1112>
[C]: in function `Click'
Interface\FrameXML\UIParent.lua:3353: in function `ToggleGameMenu'
[string "TOGGLEGAMEMENU"]:1: in function <[string "TOGGLEGAMEMENU"]:1>

Locals: <none>

Not working with Tukui

Doesn't seem to work with Tukui. Any clue to why? Understand that it's not your addon.

Could it maybe have something to do with that in the newer versions the "sparkle" is gone and you now recolour the border of the item, which Tukui already does and thus interfering?
Liked the sparkle but I guess that's just down to taste.

Don't enable the button if the MailFrame is shown

Commit: bf7e370

I'm curious, why was the button disabled when the MailFrame is shown? That's when I use it the most!

Error when holding ALT over a vendor item

• What version are you using? 70100.42-Release
• Can the issue be reproduced, and if so, how? Yes. Hold ALT while mousing over an item that a vendor is selling.
• If there was an error message, please include it.

[C]: in function `assert'
...dOns\Molinari\libs\LibProcessable\LibProcessable-14.lua:23: in function `IsMillable'
Molinari\Molinari-70100.42-Release.lua:100: in function <Molinari\Molinari.lua:85>
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
...rfaceTradeSkillMaster\Private\TooltipLib.lua:95: in function <...rfaceTradeSkillMaster\Private\TooltipLib.lua:93>
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: ?
[C]: in function `SetHyperlink'
TradeSkillMaster\CoreAPI\LuaUtil.lua:97: in function `SafeTooltipLink'
...ceTradeSkillMaster_Vendoring\Modules\Buy.lua:331: in function `handler'
...ns\TradeSkillMaster\GUI\TSMFrames\ScrollingTable-10145.lua:55: in function <...ns\TradeSkillMaster\GUI\TSMFrames\ScrollingTable.lua:47>

Locals:
(*temporary) = nil
(*temporary) = "itemID needs to be a number or convertable to a number"

TBC Issue

3x ...dOns\Molinari\libs\LibProcessable\LibProcessable-48.lua:329: attempt to call global 'GetProfessions' (a nil value)
[string "@Molinari\libs\LibProcessable\LibProcessable-48.lua"]:329: in function <...dOns\Molinari\libs\LibProcessable\LibProcessable.lua:315>

Locals:
self = <unnamed> {
 0 = <userdata>
}
event = "SKILL_LINES_CHANGED"
(*temporary) = nil
(*temporary) = "attempt to call global 'GetProfessions' (a nil value)"
professions = <table> {
}
CLASSIC = false
classicIDs = <table> {
 Alchimie = 171
 Minage = 186
 Herboristerie = 182
 Enchantement = 333
 Travail du cuir = 165
 Couture = 197
 Ingénierie = 202
 Dépeçage = 393
 Forge = 164
}

Animation not playing

Might be specific to Bagnon on Classic, reported on CurseForge:

Cannot find a library instance of "Wasabi"

Date: 2016-08-25 14:54:42
ID: 1
Error occured in: Global
Count: 1
Message: ..\AddOns\Molinari\Config.lua line 9:
Cannot find a library instance of "Wasabi".
Debug:

!Swatter\Support\LibStub.lua:23: LibStub()
Molinari\Config.lua:9: in main chunk
Locals:
(*temporary) = "Cannot find a library instance of "Wasabi"."

AddOns:
Swatter, v5.21.5490 (SanctimoniousSwamprat)
AchieveIt, v
ACP, v3.4.28
AddOnSkins, v3.36
Altoholic, vr171
Auctionator, v4.0.10
DataStore, vr59
DataStoreAchievements, vr42
DataStoreAgenda, vr23
DataStoreAuctions, v6.0.001
DataStoreCharacters, vr36
DataStoreContainers, vr53
DataStoreCrafts, vr73
DataStoreCurrencies, vr31
DataStoreGarrisons, vr16
DataStoreInventory, vr46
DataStoreMails, vr46
DataStorePets, vr39
DataStoreQuests, vr36
DataStoreReputations, vr37
DataStoreSpells, vr27
DataStoreStats, vr26
DataStoreTalents, vr54
DBMCore, v
DBMDefaultSkin, v
DBMStatusBarTimers, v
ElvUI, v10.14
ElvUISLE, v3.12
FollowerLocationInfo, v0.12-release2
FollowerLocationInfoData, v0.9-release
HandyNotes, vv1.4.8
HandyNotesDraenorTreasures, v1.20
InFlightLoad, v6.2.001
MacroToolkit, vv7.0.1.1
MasterPlanA, vA
MikScrollingBattleText, v5.7.146
MogIt, v3.6.3
MogItCloth, v3.6.3
MogItCosmetic, v3.6.3
MogItLeather, v3.6.3
MogItMail, v3.6.3
MogItOneHanded, v3.6.3
MogItOther, v3.6.3
MogItPlate, v3.6.3
MogItRanged, v3.6.3
MogItTwoHanded, v3.6.3
Molinari, v70000.37-Release
BlizRuntimeLib_enUS v7.0.3.70000
(ck=4ac)

Items not showing up in the item blacklist on first load

Need to improve the caching handler

alt key doesn't

Molinari addon correctly installed and works. No LUA error.

Simply, alt key doesn't work anymore

Lockboxes not highlighting on rogue

Describe the bug
When I open my inventory and use alt over a lockbox, nothing happens. I reloaded after turning off any other addon throwing up an error.

How to reproduce
Log in and hit the alt key while the mouse is over a lockbox in inventory

What did you expect?
For the lockbox to be highlighted

Error:

4x ...dOns\Molinari\libs\LibProcessable\LibProcessable-44.lua:94: attempt to compare number with nil
[string "@molinari\libs\LibProcessable\LibProcessable-44.lua"]:94: in function IsDisenchantable' [string "@Molinari\Molinari-90000.69-Release.lua"]:150: in function <Molinari\Molinari.lua:130> [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "=[C]"]: ? [string "@TradeSkillMaster\LibTSM\Service\ItemTooltipClasses\Wrapper.lua"]:98: in function SetBagItem'
[string "@FrameXML\ContainerFrame.lua"]:1321: in function ContainerFrameItemButton_OnUpdate' [string "@FrameXML\ContainerFrame.lua"]:1368: in function ContainerFrameItemButton_OnEnter'
[string "@bagnon\common\Wildpants\classes\item.lua"]:363: in function ShowTooltip' [string "@Bagnon\common\Wildpants\classes\item.lua"]:334: in function UpdateTooltip'
[string "@bagnon\common\Wildpants\classes\item.lua"]:174: in function <...faceBagnon\common\Wildpants\classes\item.lua:173>

Locals:
self =

{
professionCategories = {
}
HasProfession = defined @molinari\libs\LibProcessable\LibProcessable.lua:254
containers = {
}
herbs = {
}
IsOpenableProfession = defined @molinari\libs\LibProcessable\LibProcessable.lua:189
enchantingItems = {
}
IsProspectable = defined @molinari\libs\LibProcessable\LibProcessable.lua:58
IsDisenchantable = defined @molinari\libs\LibProcessable\LibProcessable.lua:81
ores = {
}
IsOpenable = defined @molinari\libs\LibProcessable\LibProcessable.lua:161
GetProfessionCategories = defined @molinari\libs\LibProcessable\LibProcessable.lua:267
IsMillable = defined @molinari\libs\LibProcessable\LibProcessable.lua:31
}
item = "|cffffffff|Hitem:169475::::::::50:260:::::::|h[]|h|r"
itemID = 169475
_ = nil
_ = nil
quality = nil
_ = nil
_ = nil
_ = nil
_ = nil
_ = nil
_ = nil
_ = nil
_ = nil
class = nil
subClass = nil
(*temporary) = 2
(*temporary) = "attempt to compare number with nil"
LE_ITEM_QUALITY_UNCOMMON = 2
LE_ITEM_QUALITY_EPIC = 4
LE_ITEM_CLASS_ARMOR = 4
LE_ITEM_CLASS_WEAPON = 2
LE_ITEM_CLASS_GEM = 3
LE_ITEM_SUBCLASS_ARTIFACT = 11

Have to click more than once to disenchant

Before patch 10.1, i was able to hold ALT and click on an item, and it would disenchant it.
Now, if i hold ALT and click on an item, it turns the icon Blue and i have to let go of ALT and click on the item again to make it disenchant.

error??

Molinari blocked from using SecureStateDriverManager:SetAttribute()
Count: 4

Call Stack:
[string "=[C]"]: in function SetAttribute' [string "@Interface\FrameXML\SecureStateDriver.lua"]:10: in function <Interface\FrameXML\SecureStateDriver.lua:8> [string "=(tail call)"]: ? [string "@Interface\AddOns\Molinari\Molinari.lua"]:99: in function UpdateModifier'
[string "@interface\AddOns\Molinari\Config.lua"]:27: in function callback' [string "@Interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua"]:329: in function Fire'
[string "@interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua"]:162: in function <Interface\AddOns\Molinari\libs\Wasabi\Wasabi.lua:155>
[string "=[C]"]: in function pcall' [string "@Interface\FrameXML\InterfaceOptionsFrame.lua"]:206: in function <Interface\FrameXML\InterfaceOptionsFrame.lua:205> [string "=[C]"]: in function securecall'
[string "@interface\FrameXML\InterfaceOptionsFrame.lua"]:229: in function <Interface\FrameXML\InterfaceOptionsFrame.lua:221>

Lockpicking no longer working in Classic Era version.

Describe the bug

Following the last 2 updates, lockpicking is no longer working in the Classic Era version.
Not tried it in the other versions of WoW. I'm using the standard "Alt" keybind.

How to reproduce

Anytime I try to unlock a locked box.

What did you expect?

I expected the box to be engagable with Molinari but, it isn't any longer.

Screenshots

Compability with HaloPro

Edit: you can close it, after second review it has absolutely nothing to do with Molinari. It is with all "Swirls" (Pet bar, etc pp)

Sorry. :)

Need a reloadui

I use Molinari with Arkinventory, French client (frFR). Tried to use it alone too, but have the same bug. In order to have Molinari working, i need to reload UI. Without doing this, it's as if Molinari was not loaded.

prospecting info

Describe the solution you'd like
Would it be possible to add to the tooltip when holding alt down what gems you can get from Prospecting the ore your'e hovering over?
im asking because on the TBC wowhead it dont show prospecting as a tab when on an ore type

Update for WOTLK Classic

Describe the bug
This addon does not work for milling in WOTLK classic.

How to reproduce
Get stacks of herbs, press key combination and select herbs. Herbs just flash all of a similar type, no action is taken.

What did you expect?
The stack of herbs to be milled

I verified disenchant is still working on a different toon, so it appears to be an issue with milling specifically, probably because this build is for TBC. Prepatch also launches tomorrow so you should update the TOC if you have a chance.