With the 20091012 update my fps drop down to 5-7 while I normally run about 60. As soon as I unload Reagent Restocker it goes back to normal. It is fine with the previous version. Just thought I'd give you a heads up.

__________________
Thank you for your time

Still error after login:

[2009/10/12 19:40:53-489-x109]: ReagentRestocker-1.1.3 beta\Core.lua:1492: attempt to index field '?' (a nil value)

Locals:

---

I went through my addons and found that the drop target square button thingy does not show with Decursive (http://wow.curse.com/downloads/wow-a...Decursive.aspx) or Quartz (http://wow.curse.com/downloads/wow-a...ls/Quartz.aspx) enabled.

Hope that can be fixed

Quick question: Is the drag and drop support already in?
Asking because I see: "Drag item into drop target." but no matter where I drag my reagent it doesn't add to my shopping list.

Good to see updates to this, I have been using it for a loooong time

Hi,

Really like what you have done with this add on its great. One problem though and not sure if it is because of Chocolate Bar (my LDB Bar add on I use to display the pluggins) or something else but what is happening it is putting everything in my bags onto the Chocolate Bar and there is heaps especially since I logged first onto my AH toon and now all my bars are covered in RR Icons and quantities lol. Here is the link to the Bar add on http://wow.curse.com/downloads/wow-a...colatebar.aspx . Hope you can help me or let me know if I need to contact the other Author.

Thanks heaps

PS: En-US client playing Alliance

The broker support is very well laid out, and I have a new favorite mod!

A couple of small points though. I only carry 20 of item x at any one time and it seems broker is coloring the text properly as green at the full 20 however it turns red as soon as i use one. I guess thats coming from the max number 640.

is it possible to set a threshold based on the total or one that's user defined?

Second , I don''t know if that's intended but while the green color looks great but for some reason the red is a bit washed out on a black background.

In any case thanks for the hard work, I've already switched over, and that's worth a banana!

Error after login:

[2009/10/12 03:13:55-481-x6]: ReagentRestocker-1.1.1 beta\Core.lua:1503: attempt to index field '?' (a nil value)
Broker2FuBar-1.0\pluginPrototype.lua:58: in function `OnFuBarEnter'
LibFuBarPlugin-3.0-90063:1319: in function <...FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1311>
FuBar-3.0.90012\FuBar.lua:1234: in function <FuBar\FuBar.lua:1231>

Locals:
self = <table> {
GetFuBarText = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:943:
IsDisabled = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1247:
CheckWidth = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:864:
data_object = <table> {}
ToggleFuBarTextColored = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:529:
ToggleFuBarIconShown = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:974:
UpdateFuBarPlugin = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:578:
ShowFuBarIcon = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1017:
GetFuBarIcon = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:850:
Hide = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:787:
enabled = true
SetFuBarIcon = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:821:
CreatePluginChildFrame = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1402:
GetPanel = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:459:
GetName = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:423:
Show = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:738:
GetFrame = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:447:
ShowFuBarText = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1094:
IsFuBarTextColored = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:519:
OpenMenu = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1491:
GetCategory = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:435:
GetDefaultPosition = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1120:
GetTitle = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:406:
CreateBasicPluginFrame = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1294:
db = <table> {}
SetPanel = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1210:
ReattachFuBarTooltip = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1181:
SetFuBarOption = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:332:
HideFuBarText = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1106:
IsFuBarMinimapAttached = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:542:
DetachFuBarTooltip = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1168:
SetFontSize = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1216:
SetFuBarText = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:918:
ToggleFuBarTooltipDetached = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1146:
UpdateFuBarTooltip = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:681:
UpdateFuBarText = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:590:
data_object_name = "RR_Leichtes Pergament"
IsFuBarTooltipDetached = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1130:
ToggleFuBarTextShown = <function> @ FuBar\libs\LibFuBarPlugin-3.0\LibFuBarPlugin-3.0.lua:1056:
ToggleFuBarMinimapAttached
---

So this mod has now grown from 70kb to its current 260kb. I dont particularly care for this. In terms of functionality, I dont see any difference. This mod has served me well for a long time, silently doing exactly what I need it to do. Please dont turn this into some bloated piece of crap that promises to do way more than it needs to. Simple is better.

I'm glad to see that you used my fix, but I have to point out I think you might have misused it a bit, although that's partially cause I explained it's working a bit loosely.

As it is the line also gives 0 if the confirmed discount is 0(duh), ie the vendor is confirmed neutral or worse (although if worse I don't think the vendoring works), still it will get subjected to the old function which might say it still gives a discount while it doesn't.

If you want to pass to the old code while no one is targeted, ie one reason for 0 I'd rather put it

local reaction = UnitReaction("target", "player")

if reaction then

return max((reaction-4) * 5, 0);

end

because the UnitReaction returns targets reaction number (in this case the vendors) or nil if no one is selected

I myself simply have the whole getMerchantDiscount have one line which is

return max(((UnitReaction("target", "player") or 0)-4)*5, 0)


sorry for the rant I just felt that in the form it is now my fix is useless as it still passes to the odd code while the discount is already cleared in case with neutrals.. just mention if you don't want me stealing in your work..