TekNoLogic / TourGuide

WoW Addon - Powerleveling guide framework
http://www.tekkub.net/
17 stars 26 forks source link

|L|1234 2| not working as it should #36

Closed Satissis closed 13 years ago

Satissis commented 14 years ago

Yet another thing I have found or rather I hope it's a bug and not me that is just stupid :P

well here is my guide code line: B Buy 2 Shiny Baubles |R|Gnome, Dwarf| |L|6529 2| |Z|Dun Morogh| |N|Buy 2 Shiny Bauble from Gretta Ganter at Brewnal Village. (31.54, 44.66)|

When I then go and buy 2 Shiny Baubles it will complete at the first bauble I buy. if I change |L|6529 2| to |L|6529 3| and then try buying, it works with 2 buys.

Now I have not come unprepared since I have been looking in your code to see what the problem could be and I found this:

In QuestTracking.lua line 77 or (action == "BUY" or action == "KILL" or action == "NOTE") and lootitem and itemid == lootitem and (GetItemCount(lootitem) + 1) >= lootqty then

I saw this: (GetItemCount(lootitem) + 1) >= lootqty should that not just be: GetItemCount(lootitem) >= lootqty

I have not testet if this will fix the issue but in my logic it should work.

Kind regards Satissis