LATEST NEWS

General Bug Reporting

img
Dec
01

Known Bugs:

Before submitting a bug, please check this list to make sure you aren’t repeating a submission!

Submit Your Bug:

When submitting your addon bug, you MUST include the following:

  • The version number of your WoW-Pro installation
  • Any error text that WoW provided when your bug happened
  • Information about the guide you were using when the bug happened, including:
    • Faction (Alliance/Horde)
    • Zone
    • What step you were on
  • As many details as possible about what you were doing when the bug occurred.

Including this information will help us troubleshoot and possibly get you a temporary solution to the bug, and get it fixed as quickly as possible!

  • img
    lilyhaven
    Jul 25, 2016 @ 20:27 pm

    Shadowmoon valley and Frostfridge ridge guides breakingUsing the current version, 7.0.3H from this site.At about 80-95% of the way through the zone, the guide just breaks on both of these zones. It stops updating steps, showing 1 step and not populating the other 4 I normally have showing, showing steps I’ve completed with checkmarks but not scrolling on. If I switch to a different guide, it works fine, when I switch back it shows the steps of the guide I had previously switched to, but with them checked off (for guides I haven’t done yet).I’ve completely uninstalled the guide, deleted all lua’s from the act saved variables and the character specific, and then reinstalled. Still broken at the same place.It seems to be related to garrison things. The shadowmoon valley broke just before the podlings section, when I was back in the garrison turning in to the building manager guy. The horde one broke on the quest turning timber into profit.

  • img
    maerdred
    Jul 23, 2016 @ 15:27 pm

    WowPro addon lua errors upon loginAs soon as I log in to Wow I get three Lua errors referencing WoWPro_Broker. Pasted below are the three errors from WoW. I am using the latest version of the addon installed this morning from this website: 7.0.3H1:Message: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: Usage: GetBuildings(garrisonType)Time: 07/23/16 09:17:44Count: 1Stack: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: in function `NextStep’InterfaceAddOnsWoWProWoWPro_Broker.lua:1092: in function `NextStepNotSticky’InterfaceAddOnsWoWProWoWPro_Mapping.lua:439: in function `MapPoint’InterfaceAddOnsWoWProWoWPro_Broker.lua:1151: in function `CompleteStep’InterfaceAddOnsWoWProWoWPro_Broker.lua:646: in function `NextStep’InterfaceAddOnsWoWProWoWPro_Broker.lua:1092: in function `NextStepNotSticky’InterfaceAddOnsWoWProWoWPro_Mapping.lua:439: in function `MapPoint’InterfaceAddOnsWoWProWoWPro_Events.lua:231: in function `AutoCompleteQuestUpdate’InterfaceAddOnsWoWProWoWPro_Parser.lua:722: in function <InterfaceAddOnsWoWProWoWPro_Parser.lua:647>(tail call): ?[C]: ?[string “safecall Dispatcher[1]”]:9: in function <[string “safecall Dispatcher[1]”]:5>(tail call): ?…OnsElkFactionslibsAceBucket-3.0AceBucket-3.0.lua:118: in function `func’…sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:55: in function <…sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:48> Locals: <none> 2:Message: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: Usage: GetBuildings(garrisonType)Time: 07/23/16 09:18:28Count: 2Stack: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: in function `NextStep’InterfaceAddOnsWoWProWoWPro_Broker.lua:1092: in function `NextStepNotSticky’InterfaceAddOnsWoWProWoWPro_Mapping.lua:439: in function `MapPoint’InterfaceAddOnsWoWProWoWPro_Events.lua:231: in function `AutoCompleteQuestUpdate’InterfaceAddOnsWoWProWoWPro_Events.lua:703: in function <InterfaceAddOnsWoWProWoWPro_Events.lua:417> Locals: k = 396i = 1GID = “EmmFrostfire”skip = falseQID = “36137”Name = “lumbermill”ids = “40;41;138”numList = 3idHash = <table> { 138 = true 41 = true 40 = true}QidMapReduce = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:12Rep2IdAndClass = <table> { good friend = <table> { } best friend = <table> { } hated = <table> { } friend = <table> { } friendly = <table> { } stranger = <table> { } revered = <table> { } honored = <table> { } hostile = <table> { } buddy = <table> { } unfriendly = <table> { } neutral = <table> { } exalted = <table> { } acquaintance = <table> { }}3:Message: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: Usage: GetBuildings(garrisonType)Time: 07/23/16 09:17:44Count: 1Stack: InterfaceAddOnsWoWProWoWPro_Broker.lua:937: in function `NextStep’InterfaceAddOnsWoWProWoWPro_Broker.lua:289: in function <InterfaceAddOnsWoWProWoWPro_Broker.lua:253>(tail call): ?[C]: ?[string “safecall Dispatcher[1]”]:9: in function <[string “safecall Dispatcher[1]”]:5>(tail call): ?…OnsElkFactionslibsAceBucket-3.0AceBucket-3.0.lua:118: in function `func’…sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:55: in function <…sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:48>Locals: <none>

  • img
    Aerendill
    Jul 22, 2016 @ 13:04 pm

    Russian symbols did noot showI installed 7.0.3Hit seems works fine but in quests log i have somthing like this  russian discription did not show correctly. I try change font for it, but it did not help (

  • img
    Kahlibaja
    May 30, 2016 @ 1:14 am

    Game won’t loadHey! Been using the addon for years, never had a problem, until about 3 weeks ago, and I couldn’t get the game to load if I’m running the addon. It only does it with this computer, any other computer in my house that I play wow on it works. I even copied the interface folder from a computer I confirmed it worked with. I’ve disabled every addon except tomtom and wow-pro, and it still doesn’t load, but I’m pretty sure it is wow-pro because the game loads with tomtom enabled. I’ve done everything I can think of short of a full reinstall, and thought I’d try you guys first, and maybe alert you to the problem. The addon (and all my other addons) are fully updated, the game is updated, all my drivers are updated. I’m using Windows 10.

  • img
    xeronin
    Apr 2, 2016 @ 22:41 pm

    Dalaran MapI’m using version 6.2.4. When I went into Dalaran I got the error: Zone [4395] is not in Zone2MapID. Please Report! I’m currently trying to use The Storm Peaks guide for lvl 77-80. The TomTom arrow isn’t showing up for inside of Dalaran.

  • img
    Mar 22, 2016 @ 18:46 pm

    Re: Guide selector generates a bugI’m pretty sure I fixed this bug in release 6.2.4, but I’ll double check when the server comes back before I release.

  • img
    Radiance
    Mar 21, 2016 @ 22:01 pm

    Guide selector generates a bugI’m just levelled up my rogue to 100 and wanted to start doing the professions. As soon as I hover over a guide bugsack reports an error.I’m also using LUA V3 if that makes any difference.53x WoWProWoWPro-6.2.3.lua:766: attempt to compare number with stringWoWProWoWPro-6.2.3.lua:766: in function `LevelColor’…nsWowPro_ProfessionWoWPro_Profession_GuideList.lua:60: in function `GuideTooltipInfo’WoWProWoWPro_GuideList.lua:345: in function <WoWProWoWPro_GuideList.lua:344>Locals:row = WoWPro_Profession_Guide_Row4 { 0 = <userdata> GID = “LudoShopBlacksmithing” Name = <unnamed> { } Progress = <unnamed> { } Author = <unnamed> { } module = <table> { }}tooltip = tooltip { 0 = <userdata>}guide = <table> { sequence = <function> defined @WowPro_ProfessionVendorsBlacksmithPlans.lua:24 name = “Blacksmithing Plans” zone = “Blacksmithing” guidetype = “Profession” author = “Ludovicus” endlevel = ” “ startlevel = “0”}(*temporary) = <function> defined =[C]:-1(*temporary) = GameTooltip { 0 = <userdata> updateTooltip = 0.14799999892712 comparing = false shoppingTooltips = <table> { } SetPetAction = <function> defined =[C]:-1 SetShapeshift = <function> defined =[C]:-1 hasMoney = 1 numMoneyFrames = 1 needsReset = true SetAction = <function> defined =[C]:-1}(*temporary) = “Category:”(*temporary) = nil(*temporary) = 1(*temporary) = 1(*temporary) = 1(*temporary) = <function> defined =[C]:-1

  • img
    amasokin
    Dec 17, 2015 @ 16:46 pm

    Compatibility problem with Dugi Questing Essential?I can’t get the WoWPro addon to work with Dugi Questing Essential (http://www.curse.com/addons/wow/dugi-questing-essential), which is imo THE new major quest addon on the “market” today and replaced Carbonite for me. Dugi Questing Essential (DQE) has a TomTom Emulation function but when I use WoWPro it doesn’t create any waypoint in DQE. The /way x y command does work with DQE though, so any idea why WoWPro isn’t working with DQE? I’m using WoWPro 6.2.2A and I don’t have any error message, the waypoints coords are just not “pushed” to DQE.Thanks!

  • img
    Aug 18, 2015 @ 0:54 am

    Re: Sound Not Working for Completed Steps?Huh!   I did not know about that feature.   Let me look into it.   Blizzard proibably changed the path to the sound and we never noticed the change. PS:   Yup.  In 6.1 they changed from .wav files to .ogg files.   Fixing the pathnames makes it immediatly noisy.Fixed in https://github.com/Jiyambi/WoW-Pro-Guides/commit/6330665d8d4e59d034c3e49cb7c36050ac007a7c for the next patch.

  • img
    Jayshua
    Aug 14, 2015 @ 23:42 pm

    Sound Not Working for Completed Steps?The Sound options for completed steps are not working. Is this a known bug or has it been removed from the addon and the options still exsist?

  • img
    Jun 27, 2015 @ 2:20 am

    Re: WoWPro.lua error on line 588 when trying to auto switchFixed in https://github.com/Jiyambi/WoW-Pro-Guides/commit/e22670cf0c7c292229bf6939709a9264da41a96d

  • img
    Jun 26, 2015 @ 14:58 pm

    Re: Normally run all my guidesAs of 6.2.0 all guides now have a “D” step added to the end to fix this problem for real.I will also be adding an L step to the start of every guide to enforce the minimal level and will remove the ones you started adding.It is a good idea, but there is no need for guide writers to implement it.

  • img
    Jun 26, 2015 @ 14:55 pm

    Re: WoWPro.lua error on line 588 when trying to auto switchThanks for the report.   I’ll fix this later today and it will be released as part of 6.2.0A next Thursday.Without even looking at the code, I can see in my head what I did wrong ;-).

  • img
    Ookami.kun
    Jun 25, 2015 @ 8:04 am

    WoWPro.lua error on line 588 when trying to auto switch guides.Upgraded to the newest version of the addon today (6.2.0) and when I was going through checking off all the guides I had completed after deleting my lua save (just in case of a conflict) I got the following error when the addon tried to automatically switch from the Spires of Arrak guide to the Nagrand.Date: 2015-06-25 01:44:29ID: 9Error occured in: GlobalCount: 1Message: ..AddOnsWoWProWoWPro.lua line 588:   attempt to index field ‘?’ (a nil value)Debug:   WoWProWoWPro.lua:588: GetGuideName()   WoWProWoWPro_Parser.lua:475: ParseSteps()   WoWProWoWPro_Parser.lua:528:      WoWProWoWPro_Parser.lua:506   (tail call): ?   [C]: ?   [string “safecall Dispatcher[1]”]:9:      [string “safecall Dispatcher[1]”]:5   (tail call): ?   …ceAddOnsArchyLibsAceBucket-3.0AceBucket-3.0.lua:118: func()   …sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:55:      …sAckisRecipeListlibsAceTimer-3.0AceTimer-3.0.lua:48Locals:NoneAddOns: 

  • img
    Dec 2, 2014 @ 3:00 am

    Found right QID for Pale Loot sackI was randomly stumbling around on wowhead and came across their guides to treasure by zone, I know have the CORRECT QID for Pale Loot Sack.  I spent so long trying to figure that out in Beta 🙁

  • img
    Nov 29, 2014 @ 8:06 am

    Pale Loot Sack SpecificallyThat step will not auto check off, I have tried and tried to figure out why and can’t.  I have debated taking it out of the guide but havent yet, I think I put a comment you have to manually check this off.. If I havent I will go do that now. 🙁  Im just going to change it to the next QID then it still wont check off when you do it, but at least once you check it off manually, it will stay checked off.

  • img
    EnigmaTL
    Nov 26, 2014 @ 3:09 am

    Same ProblemI am having all the same issues are Jardi but in the Draenor Zones both on Horde and Alliance. Its like sometimes the addon wont advance after picking up or turning in a quest. And sometime when I come back online its on a quest way back (Example Frostfire ridge Horde side quest Pale Loot bag. The one from rescuring the baby frost wolf by the first town).  Tried uninstalling and reinstalling no luck. Am using version F and set to rank 3.

  • img
    Nov 9, 2014 @ 19:14 pm

    Re: Normally run all my guidesWhoops!   I suspect that if you skip quests in a guide, that the same thing will happen.I know where to look.  The fact that the count is off means that the completeness total is off.Thansk for the report.   I’ll have it fixed for Wednesday,.

  • img
    Nov 9, 2014 @ 17:17 pm

    Normally run all my guidesNormally run all my guides with Difficulty/Completeness set to 3, so I get everything.Today I decided to run it Difficulty/Completeness set to 1, which was great until it got to the end of the guide. In the title, I had 114/127, and the window was blank, it didn’t automatically move onto the next guide.

  • img
    Dec 28, 2013 @ 19:06 pm

    If you have bad framerates with WoWPro …Please try the development version of the addon at :https://github.com/Jiyambi/WoW-Pro-Guides/archive/master.zipand be sure to disable the recorder, or you will get confused.  I updated all of the support libraries, which I forgot to do after 5.4 was released.   Some were VERY stale.Also the addon saves a copy of the latest log in your world of warcraft directory.  You can get it at:/Applications/World of Warcraft/WTF/Account/6666666#1/SavedVariables/WoWPro.luaC:Program files (x86)World of WarcraftWTFAccount6666666#1SavedVariablesWoWPro.luaJust send me that file.   If you want to be even more helpful.  Download and install this addon:http://www.curse.com/addons/wow/brokercpuand then left click on the minimap button that looks like a computer with a red wave on it and select “CPU profiling”.  Then quit out of WoW and start it again.  It will be monitoring your FPS and collecting data on all your addons.  To see a report, just bring the mouse over the minimap button.  Send me a screenshot of things that are bad.  

  • img
    Dec 26, 2013 @ 16:56 pm

    Re: I am using ElvUI,Hmm, let me see if I can rig a test for you to run.I’ll also double-check I am using the latest support libraries for the addon.   Sometimes they need updates after a release. 

  • img
    ishirra
    Dec 23, 2013 @ 13:24 pm

    I am using ElvUI,I am using ElvUI, arkinventory, deadly bossmods and a few battlepet addons.I did notice this: it only happens when accepting certain quests, I disable the addon, do what I have to do, run back to where I have to turn in the quests, enable the addon and turn them in WITH the addon enabled.  My FPS goes back to normal as soon as the quests are turned in.  It’s the quests where the horde enter pandaria.    I do have to say, I tried it on another alt and it hasn’t happened anymore as soon as my first char bit through the FPS drops.  Maybe it was an addon causing this that has been updated since, I have no clue.  But at the moment I can play without FPS drops.  In case this changes again I will let you know.Is there any other way to check the debug log (file in the WTF folder or something), maybe it will point out what was going wrong when I tried to open it in-game.I’m running wow on windows 7 Home Premium, 6 gigs of RAM, AMD  A8 – 4500M APU radeon (it’s a laptop).  I never had problems running wow, cept for those few times that the FPS dropped in the past.

  • img
    bskoros
    Dec 23, 2013 @ 4:02 am

    Also experiencing bad frameratesNo LUA errors, just really bad framerates, so bad I had to disable the addon. I am running a Core2Quad 4.2Ghz system, 4 gig ram, 1 TB HD, nVidia GForce GT460 video card.

  • img
    Dec 19, 2013 @ 23:22 pm

    Re: FPS drops when WoWPro is enabledCan you tell me what kind of computer you are running on?Can you tell me what other addons you have loaded?The fact that you can’t get a debug log makes me think something is very wrong.My FPS never gets below 60, so I suspect that our addon is interacting with another addon. 

  • img
    ishirra
    Dec 18, 2013 @ 20:54 pm

    FPS drops when WoWPro is enabledWhen the addon isn’t enabled I have an FPS of 50 – 80. When enabled, and after doing a few quests, it drops to 2 – 10 which is not really playable. When disabling the addon, the FPS goes up right away. This happens to me in all Pandaria leveling guides.  It has been happening to me for the last few versions actually.I tried to look at the debug log, to see if that would provide anything. Upon clicking to show the log, it first froze my WoW and then my entire computer, forcing me to reboot.  So debug log is not really possible, LUA error is not given.

  • img
    Dec 16, 2013 @ 19:33 pm

    Re: Lua ErrorI had this happen to me and I checked in a fix for it (https://github.com/Jiyambi/WoW-Pro-Guides/commit/be07342893fad0d147525b5739e8fc96871351e5) and it stopped happening to me.I guess I need to look harder!  

  • img
    ishirra
    Dec 15, 2013 @ 20:50 pm

    Lua ErrorI get the following LUA-error when starting WoW.  I use the 2.7.3 version. Date: 2013-12-15 20:44:34ID: 1Error occured in: GlobalCount: 1Message: ..AddOnsWoWProWoWPro.lua line 55:   bad argument #4 to ‘format’ (string expected, got nil)Debug:   [C]: ?   [C]: format()   WoWProWoWPro.lua:55: dbp()   WoWProWoWPro_Broker.lua:306: NextStep()   WoWProWoWPro_Broker.lua:722: NextStepNotSticky()   WoWProWoWPro_Mapping.lua:431: MapPoint()   WoWProWoWPro_Events.lua:192: AutoCompleteQuestUpdate()   WoWProWoWPro_Events.lua:586:      WoWProWoWPro_Events.lua:349

  • img
    Dec 4, 2013 @ 15:21 pm

    Is the guide recorderIs the guide recorder something that is an optional function?  If so, perhaps just make the guide recorder use the complicated form of getting the Blizzard quest list, and have the normal addon use a simpler form?  For the complicated form I have no idea what the problems are when the Blizzard quest panel is open, but I would normally think walking through the list one time to open all the headers, recording which ones I opened, and then going through the entire list to get the quest information (since the API that returns the counts should be valid since all the headers are open, and even if it weren’t you could just do the same type of while loop until the title is nil), and then go back and close the headers that I open.

  • img
    Dec 4, 2013 @ 11:44 am

    Re: That sort of sucks. Is thereIts there for new guides, so by definition, the quests may not be in the usual lists, particularly in the PTR.

  • img
    Dec 4, 2013 @ 1:23 am

    That sort of sucks.  Is thereThat sort of sucks.  Is there no other way around this?  Perhaps you can just hardcode the list of class specific quests internally and reference those?

  • img
    Dec 3, 2013 @ 23:14 pm

    Re: My function works with allTurns out we do use the header in *1* place:   The guide recorder uses it to detect class specific quests.  

  • img
    Dec 2, 2013 @ 21:38 pm

    Re: My function works with allSo I had to poke at why both work. I used this function to test:
    [code]
    function WoWPro:Tepes()
    local i = 1
    while (true) do

    local questTitle, level, questTag, suggestedGroup, isHeader, isCollapsed, isComplete, isDaily, questId, startEvent = GetQuestLogTitle(i)
    if not questTitle then
    break
    else
    i = i + 1
    end
    if not isHeader then
    WoWPro:Print(“%d: %s”,questId,questTitle)
    else
    WoWPro:Warning(“%s: %s”,tostring(isCollapsed),questTitle)
    end
    end
    end
    [/code]

    and I got the following:
    [code]
    20131202/1529.646 ~ WoWPro: 1: Battle Pets
    20131202/1529.647 ~ WoWPro: nil: Blackrock Caverns
    20131202/1529.648 ~ WoWPro: 28735: To the Chamber of Incineration!
    20131202/1529.649 ~ WoWPro: nil: Brawler’s Guild
    20131202/1529.650 ~ WoWPro: 32851: Blingtron 3000
    20131202/1529.651 ~ WoWPro: 32844: Secret of the Ooze
    20131202/1529.652 ~ WoWPro: 1: Hallow’s End
    20131202/1529.653 ~ WoWPro: 1: Legendary
    20131202/1529.654 ~ WoWPro: 1: Mana-Tombs
    20131202/1529.655 ~ WoWPro: 1: Pandaren Brewmasters
    20131202/1529.656 ~ WoWPro: 1: Stormstout Brewery
    20131202/1529.657 ~ WoWPro: 1: The Arboretum
    20131202/1529.658 ~ WoWPro: 1: Vale of Eternal Blossoms
    20131202/1529.659 ~ WoWPro: nil: Valley of the Four Winds
    20131202/1529.660 ~ WoWPro: 31323: Buy A Fish A Brewery?
    20131202/1529.661 ~ WoWPro: 31813: Dagra the Fierce
    20131202/1529.662 ~ WoWPro: 31889: Battle Pet Tamers: Kalimdor
    20131202/1529.663 ~ WoWPro: 11131: Stop the Fires!
    20131202/1529.664 ~ WoWPro: 31902: Battle Pet Tamers: Eastern Kingdoms
    20131202/1529.665 ~ WoWPro: 8311: Hallow’s End Treats for Jesper!
    20131202/1529.666 ~ WoWPro: 31468: Trial of the Black Prince
    20131202/1529.667 ~ WoWPro: 31473: The Strength of One’s Foes
    20131202/1529.668 ~ WoWPro: 10977: Stasis Chambers of the Mana-Tombs
    20131202/1529.669 ~ WoWPro: 30567: Blanche’s Boomer Brew
    20131202/1529.670 ~ WoWPro: 31327: Trouble Brewing
    20131202/1529.671 ~ WoWPro: 31539: A Thing of Beauty
    20131202/1529.672 ~ WoWPro: 33229: A Flash of Bronze…
    20131202/1529.673 ~ WoWPro: 29371: A Time to Lose
    [/code]
    Note that all of the “hidden” quests show after the last item that shows in the UI (Buy A Fish A Brewery?). So your code does work if you don’t care about which header they were under.

    So now I gotta see if we use that information anywhere.
    If not, I can simplify our code.

  • img
    Dec 2, 2013 @ 13:07 pm

    My function works with allMy function works with all the headers collapsed.  At least this new version does.

  • img
    Dec 2, 2013 @ 1:49 am

    Re: This is the routine I use inYou might want to try the following experiment.  Collapse all the headers in the Quest Log UI.  Then run your function.It used to be the case that a straight loop like yours would not pick up any quests!

  • img
    Dec 2, 2013 @ 1:07 am

    This is the routine I use inThis is the routine I use in Grail.  (The next release as I updated it based on your previous findings.) — Returns a table whose key is the questId and whose value is a table made of the quest title and the completedness
    — of the quest for each quest in the Blizzard quest log.  If there is nothing in the log, an empty table is returned.
    _QuestsInLog = function(self)
    if nil == self.cachedQuestsInLog then
    local retval = {}
    — It tuns out that numQuests will be correct, but numEntries will not reflect the total number of values that
    — will be returned from GetQuestLogTitle() if any of the headers are closed.  With closed headers, the quests
    — that would normally be in them are going to be at the end of the list, but not necessarily in any specific
    — order that is helpful.
    — local numEntries, numQuests = GetNumQuestLogEntries()
    — for i = 1, numEntries do
    local i = 1
    while (true) do
    local questTitle, level, questTag, suggestedGroup, isHeader, isCollapsed, isComplete, isDaily, questId, startEvent = GetQuestLogTitle(i)
    if not questTitle then
    break
    else
    i = i + 1
    end
    if not isHeader then
    retval[questId] = { questTitle, isComplete }
    end
    end
    self.cachedQuestsInLog = retval
    end
    return self.cachedQuestsInLog
    end
      After looking at your code I believe there is no need to do all the work with expansion and collapsing of the headers.  If you based your control structure on the while loop I implemented here I imagine you can just do all your work in the “if not is header then” block.  Of course I do not know if the calls to things like GetNumQuestLeaderboards() will work with the indexes that are past the “entries” value that is returned from GetNumQuestLogEntries().  But since GetQuestLogTitle() does work with those values, I would hope the rest of the Blizzard API would. I just realized you use the header name for each quest, so my suggestion will not be that great since my loop does not care about which header a quest is in, and in fact cannot know this because the quests in closed headers are at the end of the list in an unknown order.

  • img
    Dec 1, 2013 @ 23:08 pm

    Re: GetNumQuestLogEntries() crazinessA code review of https://github.com/Jiyambi/WoW-Pro-Guides/blob/master/WoWPro/WoWPro_Broker.lua#L765 would be appreciated, but I think I ended up implementing something close to your suggestion. When I expanded a header section and was about to collapse it because we hit the next header, I reset the index into the quests to end up being one past the old header, which is where the new header was ending up after the collapse.

    To further refine it, I will change the repeat loop termination condition from:
    [code]
    if ( i > 50 ) then
    break
    end
    until num == numQuests
    [/code]
    to:
    [code]
    until not questTitle
    [/code]
    and eliminate:
    [code]
    if not questTitle and (num < numQuests) then
    WoWPro:Error("PopulateQuestLog: return value from GetQuestLogTitle(%d) is nil.",i)
    end
    [/code]

    PS: There are two new undocumented return values from GetQuestLogTitle() 🙂

  • img
    Dec 1, 2013 @ 20:33 pm

    GetNumQuestLogEntries() crazinessHi.  I have not checked your code, but you are correct in the number of quests being returned is correct no matter which headers are open or closed.  However, the location (index into GetQuestLogTitle()) of quests will move depending on whether they are in closed headers.  For example, quest at index 2 (in an open header) will move to the end of the list if that header is closed.  Not knowing the total count of the quests through which to iterate (because GetNumQuestLogEntries() will not return the proper number if headers are closed) means to get the complete list of quests in the log one needs to continually iterate until no title is returned.  I believe this should give all quests and headers, though not necessarily in any convenient order.

  • img
    Dec 1, 2013 @ 19:11 pm

    Re: Blizzard bug 170644 (Testers Needed)OK, so there are two bugs here.We relied on the number of entries returned from GetNumQuestLogEntries to be constant.  But anytime you expand or collapse a header that number changes.  So a simple linear for loop iterating over the number of entries can get into trouble as the number of entries under a header may be bigger than the number of headers!If the QuestLog UI is up, this code can’t run or horrible things happen.So I will check the code in for this shortly.  Any testers?  Use the development zip at https://github.com/Jiyambi/WoW-Pro-Guides/archive/master.zip 

  • img
    Dec 1, 2013 @ 17:26 pm

    Blizzard bug 170644Well, I dug around the source code to the Blizzard Quest UI and found this gem:

    [code]
    –The counts may have changed with SetNearestValidSelection expanding quest headers.
    –Bug ID 170644
    numEntries, numQuests = GetNumQuestLogEntries();
    [/code]

    I ran a test on a Level 90 toon that had a bunch of quests in progress (17) and the dammed function would return 12,17 and find only 7 quests!

    Trying to use the UI to even look at 17 quests was hard, as things would misbehave left and right.

    It looks like numEntries is wrong, but numQuests is right.
    I’ll see what I can do to toughen up our code.

  • img
    Dec 1, 2013 @ 11:59 am

    QuestLog Being GoofyThis bug occured previously and we added code to “uncollapse” the sections so we could see the bugs, but with account wide quests, people are probably running right near the edge of what the UI is allowing to be displayed/queryied.I’ll give that code a looking over.  It would not be the first time Blizzard changed its behaviour. 

  • img
    Dec 1, 2013 @ 11:51 am

    Debug Log from demonllama@CurseIn a nutshell, it’s WoW-Pro conflicting (somehow) with the Account-wide Pet Battle quests. Abandon those, and it works fine. Other than the guides “not working”, observably behavior is that in the Quest Log, it keeps auto-collapsing all the quest sections every time something happens to update the log (accepting a new question, completing an existing one, etc). For some reason, that nailed the Grizzly Hills guide and the Mount Hyjal guides pretty hard. The error I was getting was that, for some reason, it was saying it couldn’t find the quest in the Quest Log. For example:

    [code]
    20131201/0123.501 ~ WoWPro: Called WoWPro:NextStep(41,1)
    20131201/0123.502 ~ WoWPro: Signaled for UpdateGuide
    20131201/0123.503 ~ WoWPro: Running: UpdateGuideReal()
    20131201/0123.504 ~ WoWPro: Called WoWPro:NextStep(1,1)
    20131201/0123.505 ~ WoWPro: Step C [Flamebreaker] skipped as not in QuestLog
    20131201/0123.506 ~ WoWPro: Step T [Flamebreaker] skipped as not in QuestLog
    20131201/0123.507 ~ WoWPro: Step C [The Return of Baron Geddon] skipped as not in QuestLog
    20131201/0123.508 ~ WoWPro: Running: WoWPro:RowUpdate()
    20131201/0123.509 ~ WoWPro: Called WoWPro:NextStep(41,1)
    20131201/0123.510 ~ WoWPro: Called WoWPro:NextStep(42,2)
    20131201/0123.511 ~ WoWPro: Step T [The Return of Baron Geddon] skipped as not in QuestLog
    20131201/0123.512 ~ WoWPro: Step T [Emerald Allies] skipped as not in QuestLog
    20131201/0123.513 ~ WoWPro: Step T [The Captured Scout] skipped as not in QuestLog
    20131201/0123.514 ~ WoWPro: Step C [Twilight Captivity] skipped as not in QuestLog
    20131201/0123.515 ~ WoWPro: Step T [Twilight Captivity] skipped as not in QuestLog
    20131201/0123.516 ~ WoWPro: Step T [Return to Alysra] skipped as not in QuestLog
    20131201/0123.517 ~ WoWPro: Called WoWPro:NextStep(54,3)
    [/code]

    When Flamebreaker was most definitely in my quest log.

  • img
    Nov 10, 2013 @ 16:48 pm

    Re: Goofy Grizzly HillsCould someone give me a  log with debugging enabled?Also, pull up the current guide and mouse over the quests that should be checked off, but are not.It should tell you why the quest was not selected.

  • img
    Nov 8, 2013 @ 14:56 pm

    Re: Goffy Grizzly HillsOK.  I’ll take a look at the guide to see if I can guess why.But I may need a debug log for this one.

  • img
    Nov 2, 2013 @ 20:48 pm

    Can confirmI just entered Grizzly Hills with my druid twink and I get the same problem.

  • img
    Jardi
    Oct 13, 2013 @ 1:33 am

    Addon being goofyVersion is the newest one, the 2.7.0 from Sept 10th, 2013.No exact error message, but everytime i log on it starts me back at the begining of the guide for the zone i am in. And once i get back to where i was it wont complete the steps, so i have to do it manually, and then after a couple quests it goes backwards mutiple steps.Current Guide: Grizzly HillsFaction: Its happening on both for my toonsStep: all of them

  • img
    Iconoclast
    Sep 11, 2013 @ 15:35 pm

    Cannot download newest version 2.7.0The addon page says the newest version is 2.7.0 released on September
    10, 2013. When I click download, it is giving me version 2.6.4. I would
    really like to download the newest version that is for WoW 5.4. Please
    fix this! Thank you for your guides.-Iconoclast

  • img
    starseeker
    Aug 25, 2013 @ 21:21 pm

    What happened to being able to mark things as sticky?I remember that I used to be able to right click (I think?) on a step and see a menu that included an option to make it sticky.  It was helpful and I miss being able to do that. Now, when I click or right click on the text for the step, it highlights the text block in yellow, which isn’t useful and doesn’t let me mark things as sticky.

  • img
    starseeker
    Aug 25, 2013 @ 21:19 pm

    Addon doesn’t remember skipped steps anymoreI’m having a consistent issue where I can’t skip steps anymore – if I reload or relog, the skipped steps all come back.  It doesn’t seem to be remembering them. I have tried deleting the addon with all saved variables and re-installing, and it doesn’t fix it.  I’m running 2.6.4.

  • img
    Jardi
    Jul 15, 2013 @ 23:24 pm

    Error with matching up to CarboniteEverytime i start WOW-PRO it tells me that WOWPro: Waiting for TomTom or Carbonite to intializeWOWPro: Waiting for TomTom or Carbonite to intializeWOWPro: Waiting for TomTom or Carbonite to intializeWOWPro: Waiting for TomTom or Carbonite to intializeWOWPro: Waiting for TomTom or Carbonite to intializeWOWPro: It Looks like you dont have TomTom or Carbonite Installed. WOWPro Guides dont have their full functionality without it! Download it for free from wowinterface.com or http://www.curse.comWOWPro: If you have Carbonite Installed, do not forget to enable Carbonite’s TomTom emulation (Tracking HUD section) The problem is, i do have caronite enabled and this meesage just keeps repeating itself over and over filling my chat box, and will not quit until i copmpletely disable WOWPro.This started with the last update.Version: 2.6.3Faction: BothStep: None it happens as soon as i log on.

  • img
    raggaren1994
    Jul 11, 2013 @ 21:33 pm

    Not advancing the steps at allI’m currently using v.2.6.3 and the addon isn’t moving onwards to the next step when i pick up the quest it tells me to, this happens on every single guide at all lvls and on both horde/alliance, this has been going on for about 1-2 weeks now i believe. the only error message i was getting is: Date: 2013-07-11 21:20:43ID: 1Error occured in: GlobalCount: 1Message: ..AddOnsWoWProWoWPro_Events.lua line 117:   attempt to index field ‘?’ (a nil value)Debug:   WoWProWoWPro_Events.lua:117: AutoCompleteQuestUpdate()   WoWProWoWPro_Events.lua:559:      WoWProWoWPro_Events.lua:336Locals:self = <table> { InitLockdown = false note = <table> { } index = <table> { } CarboniteProfileHack = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:1066 DeactivateTab = <function> defined @InterfaceAddOnsWoWProWoWPro_GuideList.lua:155 spell = <table> { } CreateGuideWindowScrollbar = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:516 CreateGuideTabFrame_Rows = <function> defined @InterfaceAddOnsWoWProWoWPro_GuideList.lua:285 NextGuide = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:123 ach = <table> { } newQuest = 6922 CreateItemButton = <function> defined @InterfaceAddOnsWoWProWoWPro_Widgets.lua:63 CreateHeading = <function> defined @InterfaceAddOnsWoWProWoWPro_Widgets.lua:104 UpdateQuestTracker = <function> defined @InterfaceAddOnsWoWProWoWPro_Events.lua:252 RowDropdownMenu = <table> { } oldQuests = <table> { } CurrentGuideFrame = <unnamed> { } rep = <table> { } Tags = <table> { } modules = <table> { } MapPoint = <function> defined @InterfaceAddOnsWoWProWoWPro_Mapping.lua:402 SetDefaultModulePrototype = <function> defined @InterfaceAddOnsDataStorelibsAceAddon-3.0AceAddon-3.0.lua:440 CreateSkipStepsDialog = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:654 UpdateGuide = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:143 SetDefaults = <function> defined @InterfaceAddOnsWoWProWoWPro_Config.lua:14 Leveling = <table> { } stickycount = 15 IsEnabled = <function> defined @InterfaceAddOnsDataStorelibsAceAddon-3.0AceAddon-3.0.lua:482 optionalcount = 3 RegisterTags = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:352 ZoneInfo = <function> defined @InterfaceAddOnsWoWProWoWPro_Mapping.lua:627 LoadAllGuides = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:402 Setup_TitleRow = <function> defined @InterfaceAddOnsWoWProWoWPro_GuideList.lua:199 OrderSteps = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:876 RegisterMessage = <function> defined @InterfaceAddOnsDataStorelibsCallbackHandler-1.0CallbackHandler-1.0.lua:118 UnregisterMessage = <function> defined @InterfaceAddOnsDataStorelibsCallbackHandler-1.0CallbackHandler-1.0.lua:181 FirstMapCall = false GuideList = <unnamed> { } AutoCompleteLevel = <function> defined @InterfaceAddOnsWoWProWoWPro_Events.lua:235 UnregisterEvents = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:373 ActiveStickyCount = 0 chat = <table> { } CreateMiniMapButton = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:624 buff = <table> { } faction = <table> { } SetEnabledState = <function> defined @InterfaceAddOnsDataStorelibsAceAddon-3.0AceAddon-3.0.lua:455 Zone2MapID = <table> { } RegisterEvents = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:363 lootitem = <table> { } Disable = <function> defined @InterfaceAddOnsDataStorelibsAceAddon-3.0AceAddon-3.0.lua:345 questtext = <table> { } StickyFrame = WoWPro.StickyFrame { } ResetMainFramePosition = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:15 NPC = <table> { } OnEnable = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:271 Version = “2.6.3” CreateMouseNotes = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:594 PickQuestline = <function> defined @InterfaceAddOnsWoWProWoWPro_Broker.lua:961 PaddingSet = <function> defined @InterfaceAddOnsWoWProWoWPro_Frames.lua:69 missingQuest = false ActiveStep = 4 active = <table> { } TitleText = <unnamed> { } AutoCompleteZone = <function> defined @InterfaceAddOnsWoWProWoWPro_Events.lua:203 CreateGuideTabFrame = <function> defined @InterfaceAddOnsWoWProWoWPro_GuideList.lua:368 OnInitialize = <function> defined @InterfaceAddOnsWoWProWoWPro.lua:241 UnregisterAllEvents = <function> defined @InterfaceAddOnsDataStorelibsCallbackHandler-1.0CallbackHandler-1.0.lua:202 Achievements = <table> { } why = <table> { } GuideTabFrame_RowOnClick = <function> defined @InterfaceAddOnsWoWProWoWPro_GuideList.lu

Leave a Reply

Your email address will not be published.

Categories

Archives