326 - Altoholic 4.0.004c not storing tradeskills.
What steps will reproduce the problem?
1. Installed World of Warcraft.
2. Installed Altoholic 4.0.004c on a clean wow installation (no old wtf or anything).
3.
What is the expected output? What do you see instead?
I'd expected my tradeskills to be saved.
What version of the product are you using?
4.0.004c
Do you have an error log of what happened?
No errors occured.
Please provide any additional information below.
After the installation, I went through all my chars as usual, to build up my character database.
Looks like most other stuff got saved, except my primary tradeskills.
Secondary skills, like First Aid, Cooking and Fishing is saved.
Gathering profession are saved (mining as an example).
Productions skills, like blacksmithing, leatherworking, alchemy, etc isn't saved.
Just the ranks part, like this:
["Leatherworking"] = {
["MaxRank"] = 450,
["Rank"] = 442,
},
| Name | Description | Size | MD5 |
|---|---|---|---|
| DataStore_Crafts.zip | Compressed copy of my ... | 9.4 KiB | 81a4754bdec0... |
| DataStore_Crafts.lua.diff | Diff between r29 and my working ... | 553 B | 44c43217c301... |
| User | When | Change |
|---|---|---|
| BoaConstrictor | Feb 12, 2011 at 08:21 UTC | Changed status from New to Invalid |
| BoaConstrictor | Dec 01, 2010 at 22:42 UTC | Added attachment DataStore_Crafts.lua.diff |
| BoaConstrictor | Nov 30, 2010 at 19:50 UTC | Added attachment DataStore_Crafts.zip |
| BoaConstrictor | Nov 30, 2010 at 19:49 UTC | Create |
- 4 comments
- 4 comments
Facts
- Last updated
- Mar 30, 2012
- Reported
- Nov 30, 2010
- Status
- Invalid - This was not a valid report.
- Type
- Defect - A shortcoming, fault, or imperfection
- Priority
- Medium - Normal priority.
- Votes
- 1
Reported by
Possible assignees
-
- Manager
- Author
-
- Author
- Reply
- #4
BoaConstrictor Feb 11, 2011 at 20:26 UTC - 0 likesRetried again with r30 of DataStore_Crafts.lua And I'm getting the same. Seems the issue is that I'm too impatient.
After reading the comments you've added in r30, I waited a bit longer before trying (read, I waited until the WoWHeadLooter status messages that my char was now scanned appeared).
Now it stores the crafts. Might be that the event/timer you're write about in your todo list is what triggers wowheadlooter. Could be that looking at that source will give you some extra hints. :)
- Reply
- #3
BoaConstrictor Dec 01, 2010 at 22:44 UTC - 0 likesAdded a diff between r-29 of DataStore_Crafts and my locally changed one that works. As you can see, the only difference is the removal of the code that checks for "skillType = "header""
- Reply
- #2
BoaConstrictor Dec 01, 2010 at 22:41 UTC - 0 likesTracked it down to this line in r28: "if skillType = "header" then return end"
If that line is removed, it's storing all the tradeskills again.
- Reply
- #1
BoaConstrictor Dec 01, 2010 at 22:09 UTC - 0 likesReverting DataStore_Crafts to revision 27 fixes the issue. Now all the tradeskills are scanned. Has to be something with the code that was added in rev 28 or 29.