SmartSell (Tooltip Info) 

10.0.2 caused too many errors and I was hip-deep in /reloadui and still playing whack-a-mole with errors; *minor* API changes and argument tweaks I can usually handle but this was a major revamp for tooltip scanning and eh... Unless someone with more acumen and interest picks this up I think it's reached EOL. Was a great run while it lasted!

I've swapped to using ItemType and a mod to expose vendor value away from vendors (Auctionator, in my case, but there are numerous options); it's not as efficient or elegant looking but it covers basically the same details so good enough. :-)

Got a mostly-working copy updated (I finding new bugs to squash), got a set of edits-so-far over on Curse or you could just download and replace Addons/SmartSell/SmartSell.lua with the contents of this Pastebin link . Caveat patch-er, ofc. :-P

Kind of late with this, but I just found this addon and I really like it!

Anyway, I'm no expert in lua by any means, but they removed the ranged/relic slot and thrown weapons in MoP, so this is what I think should fix your error cprafferty.

Use a text editor and open up the SmartSell.lua file in the SmartSell folder and go to lines 618-620 or search for ""ranged" if you don't have something that shows you line numbers. (this is the only place that mentions ranged) It should say this:

Delete this part:

Make sure you leave the closing parenthesis and the word then at the end of the list.

It should look like this:

That should take care of the error you're getting. (at least I didn't get one when I compared a bow to my dagger and off hand )

Just a bug report

When comparing a Ranged Weapon (in my case a bow, but i've managed to trigger it on wands) it produces the following error.

Message: Interface\AddOns\SmartSell\SmartSell.lua:626: Invalid inventory slot in GetInventorySlotInfo
Time: 09/03/12 18:43:23
Count: 4
Stack: [C]: in function `GetInventorySlotInfo'
Interface\AddOns\SmartSell\SmartSell.lua:626: in function <Interface\AddOns\SmartSell\SmartSell.lua:573>
[C]: ?
[C]: in function `SetHyperlinkCompareItem'
Interface\FrameXML\GameTooltip.lua:232: in function `GameTooltip_ShowCompareItem'
[string "*:OnTooltipSetItem"]:3: in function <[string "*:OnTooltipSetItem"]:1>
[C]: ?
[C]: ?
[C]: ?
[C]: in function `SetInventoryItem'
Interface\FrameXML\BankFrame.lua:38: in function `UpdateTooltip'
Interface\FrameXML\GameTooltip.lua:195: in function <Interface\FrameXML\GameTooltip.lua:185>

Locals: (*temporary) = "RangedSlot"

Anyway, thanks for keeping this addon updated.

*edit* Grammer

Oh WOW () Thank-you so much!!!!

Just wanted to say thank-you for this addon, I've been looking for a replacement for QualityID for a while & this hits the nail on the head.

Just one thing is it possible when using styles 0-2 can it also display the the price per piece?

Styles 3-6 show the price per piece in brackets.

SmartSell and 5.0-era loot frames are not playing nice with each other. Tried with and without a loot-window mod (at first I thought it was the other mod), but even with just SmartSell enabled it's throwing errors and failing to replace the sell price in the loot tooltips - the tooltip just shows the item's name, nothing else.


Message: Interface\AddOns\SmartSell\SmartSell.lua:449: attempt to call global 'LootSlotIsItem' (a nil value)
Time: 08/29/12 13:05:40
Count: 1
Stack: Interface\AddOns\SmartSell\SmartSell.lua:449: in function <Interface\AddOns\SmartSell\SmartSell.lua:447>
[C]: in function `SetLootItem'
Interface\FrameXML\LootFrame.lua:270: in function `LootItem_OnEnter'
[string "*:OnEnter"]:1: in function <[string "*:OnEnter"]:1>

Locals: self = GameTooltip {
0 = <userdata>
SetInboxItem = <function> defined =[C]:-1
SetBuybackItem = <function> defined =[C]:-1
shoppingTooltips = <table> {
}
SetMerchantItem = <function> defined =[C]:-1
SetHyperlink = <function> defined =[C]:-1
SetGuildBankItem = <function> defined =[C]:-1
SetTradeTargetItem = <function> defined =[C]:-1
SetTradePlayerItem = <function> defined =[C]:-1
Hide = <function> defined =[C]:-1
SetSendMailItem = <function> defined =[C]:-1
updateTooltip = 0.097999993525445
SetInventoryItem = <function> defined =[C]:-1
MoneyCount = <table> {
}
comparing = false
SetLootRollItem = <function> defined =[C]:-1
SetQuestLogItem = <function> defined =[C]:-1
SetAuctionSellItem = <function> defined =[C]:-1
SetBagItem = <function> defined =[C]:-1
SetTradeSkillItem = <function> defined =[C]:-1
SetAction = <function> defined =[C]:-1
SetLootItem = <function> defined =[C]:-1
SetQuestItem = <function> defined =[C]:-1
SetAuctionItem = <function> defined =[C]:-1
}
slot = 1
(*temporary) = nil
(*temporary) = 1
(*temporary) = "attempt to call global 'LootSlotIsItem' (a nil value)"
PreCheck = <function> defined @Interface\AddOns\SmartSell\SmartSell.lua:359
AddToTooltip = <function> defined @Interface\AddOns\SmartSell\SmartSell.lua:176

Cool thing, thank you : )


have a nice day

slaxi

Thanks for the new version!
You should update the FAQ and Features...

Hi
I was using this earlier on and noticed that when you have the roll for an item on group loot, when you hover over the item on the thing where you choose need or greed, it doesn't give the additional information in the tooltip. Is it possible for this feature to be added to this otherwise great addon
Cheers, Jonny

I have another feature request. Maybe you can add the ItemType?
http://www.wowwiki.com/ItemType

Thx for add

Thanks for this fine Addon, but have a feature request...
It would be nice it has a blank line before the informations (maybe a new command "bl" ).