WoW-Pro Guides

WoW-Pro Guides

3M Downloads

Carbonite Arrow Support

jlink7 opened this issue ยท 14 comments

commented

I've been having a problem completing the Icecrown guide. This issue started in 1.3.0 and has continued through 1.4.0, although the error is slightly different.

Everything was fine until I got to the quest "Free Your Mind." In 1.3.0, it never recognized that the quest was complete, I could complete the following quests and they WOULD be recognized and removed from the list. If I used the check box to manually complete it, I got the WoWError text box. I don't have that error, as I've switched to 1.4.0 now.

Now, when I log in my character in Icecrown I log in to the below error. Clicking the check box on the quest now does nothing, and the check just disappears again. I can recreate the error by doing a simple /reloadui. I've encountered this error in a couple other guides as well, but just ignored it and moved on... unfortunately I do not remember what zone guides were affected.

Date: 2010-09-27 17:13:59
ID: 1
Error occured in: Global
Count: 2
Message: ..\AddOns\WoWPro\WoWPro_Mapping.lua line 238:
attempt to index field 'db' (a nil value)
Debug:
(tail call): ?
WoWPro\WoWPro_Mapping.lua:238: MapPoint()
WoWPro\WoWPro_Broker.lua:47: LoadGuide()
WoWPro_Leveling\WoWPro_Leveling.lua:48:

commented

Working on fixing Carbonite Arrow support. If someone can test this when I'm done, that would be excellent.

commented

I'm not sure I fixed it, but it didn't break anything with TomTom enabled or with TomTom disabled. I don't have Carbonite and really don't feel like going to their website to get it, but if no one else gets a chance to test this, I will. Uploading now.

commented

I tested with Carbonite and made a small fix (in a pull request). It should work fine now. The fix has to do with how Carbonite emulates TomTom: if Carbonite is installed instead of TomTom, then Carbonite == nil, but TomTom exists (but not TomTom.db). So, the original code would work with Carbonite if you had (if TomTom) instead of (if TomTom or Carbonite), because the Carbonite variable is always nil.

commented

Ah :) Thanks, accepting the request!

commented

I tested silvann's fix on many toons in many areas and I am very glad to say it works well
Thank you silvann

commented

Nice. Sorry for breaking that in the first place. >.<

commented

Haha, no problem at all, like I said, I didn't notice it either!

commented

Did you clear your wtf folder of all wow-pro files? Try that and see if it resolves the issue, if you have not done so already. Please advise if this does not resolve the issue.

commented

Appears to be a problem involving TomTom.db. Looks like you were using Carbonite instead of TomTom, which until this update has been fine. I neglected to account for the fact that our new coordinate completion feature only works with TomTom. I'll see if I can make a work around so other arrow functions still work with Carbonite, but for now, add the following to line 234 (should be a blank line) of WoWPro_Mapping.lua in the main WoWPro addon folder:

if not TomTom then return end

This will unfortunately prevent you from having coordinates mapped at all, unless you download TomTom. This is a temporary fix, I'm hoping in the future to still provide support for other arrow addons such as Carbonite.

Sorry about that and hope that fix works for you!

commented

Ops, sorry about that!

commented

Yeah, clearing my SavedVariables folder of wowpro.lua didn't have any effect, as soon as I clicked on the Icecrown guide it gave me the same (or similar) error.

One thing I don't understand though is that the first quest that it's telling me to complete in this circumstance is one that I've already completed. (It's All Fun And Games) I suppose that's a separate issue.

commented

You shouldn't have to disable Carbonite, though you might need to disable the Carbonite arrow. The error occurs only if TomTom is not present, I don't think it will cause a problem if Carbonite is there.

As for your second problem, will look into it, I've had reports of that before. Are you using the Horde or the Alliance version of the Icecrown guide?

commented

Answering myself: I'm betting you were using Alliance, as I immediately see the issue. You can fix it yourself for now by doing the following:

Open WoWPro_Leveling\Alliance\002_79_80_NLX3647_Icecrown.lua in a notepad type application.

On lines 11-13 you should see the following:

A Free Your Mind|QID|
C Free Your Mind|QID| |U|41366|
T Free Your Mind|QID|

Change those to:

A Free Your Mind|QID|12893|
C Free Your Mind|QID|12893|U|41366|
T Free Your Mind|QID|12893|

This should fix your problem, I'll be fixing this in the next update. Sorry about that!

commented

Ok... I think I "Comment and Closed" this issue by mistake.

I disabled Carbonite and Installed TomTom and that solves issue #1. My second issue still stands though-- loading the guide loads "It's All Fun And Games" as the first unfinished quest. It DOES allow me to check that quest off manually, and then displays Free Your Mind, which it doesn't allow me to check off. When I click the check box, it pauses momentarily and then nothing happens. No check, nothing.