TrainerFu shows that I can learn Boiled Clams, but I already learned it.
When you train soemthing from the first of 2 trainers available, TrainerFu doesn't see this and thinks you still need to learn it from the second one. It clears when talking to the second trainer.
I've wasted a lot of time running to a trainer on toons I have not log'd in a while, only to find I had already trained.
While I've never actually bothered to go to a trainer for something I'd already learned ;), I do think it would be nice if TrainerFu recognized that you learned the skill and removed it from the list everywhere it appears. For now, I just use the keypress removal method.
OK Just wanted to know if I missed something or maybe others have this problem but the author doesnt know or .... till the update, I'll just remove them manually :)
Yeah, it's odd that some professions are like that, where the trainers fall under different categories. To some extent it makes sense, but to others it really doesn't.
Until you get to skill 300 in enchanting, the grand master enchanting trainer can't teach you anything. But below that, there is some overlap between artisan and expert, and so on. It'd be nice if they all fell in the same category. And of course the example of the cooking trainer.
Also, it would make sense to me to note which weapons trainer teaches what. I mention this because while my low level is an alt and I have much more money on another character, I didn't want to hand it down from my main. Meanwhile, I did go to a few weapons trainers and their skills all fall in the same category so if I wanted to learn something specific I wouldn't know who to go to right off, I'd have to visit the nearest weapons trainer and ask from there.
I use TrainerFu's option to show trainer icons in Cartographer however I find that they don't all show up. I believe TrainerFu is using "trainer" as a keyword and so when trainers with titles like "Cook", "Herbalist", "Skinner", "Miner", and "Engineer" are encountered (those without the word "Trainer" in their titles), TrainerFu won't add a location icon to the map. I know this is the case for at least one cook in Silvermoon City and a Skinner whose location I can't remember off-hand. On the other hand, Weapon Masters show up fine but don't have "Trainer" in their titles, so I may be wrong. I'd really like to see TrainerFu tightened up to recognize trainers, and add location icons, no matter what their titles.
TrainerFu shows that I can learn Boiled Clams, but I already learned it.
When you train soemthing from the first of 2 trainers available, TrainerFu doesn't see this and thinks you still need to learn it from the second one. It clears when talking to the second trainer.
I've wasted a lot of time running to a trainer on toons I have not log'd in a while, only to find I had already trained.
That should not happen i the branch version.
Quote from D. F. Schmidt »
Yeah, it's odd that some professions are like that, where the trainers fall under different categories. To some extent it makes sense, but to others it really doesn't.
Until you get to skill 300 in enchanting, the grand master enchanting trainer can't teach you anything. But below that, there is some overlap between artisan and expert, and so on. It'd be nice if they all fell in the same category. And of course the example of the cooking trainer.
Also, it would make sense to me to note which weapons trainer teaches what. I mention this because while my low level is an alt and I have much more money on another character, I didn't want to hand it down from my main. Meanwhile, I did go to a few weapons trainers and their skills all fall in the same category so if I wanted to learn something specific I wouldn't know who to go to right off, I'd have to visit the nearest weapons trainer and ask from there.
I'll see what i can do about that. I am trying make the addon as "general" as possible but in this case i can see the need for some special treatment.
Quote from Mrs. Steele »
I use TrainerFu's option to show trainer icons in Cartographer however I find that they don't all show up. I believe TrainerFu is using "trainer" as a keyword and so when trainers with titles like "Cook", "Herbalist", "Skinner", "Miner", and "Engineer" are encountered (those without the word "Trainer" in their titles), TrainerFu won't add a location icon to the map. I know this is the case for at least one cook in Silvermoon City and a Skinner whose location I can't remember off-hand. On the other hand, Weapon Masters show up fine but don't have "Trainer" in their titles, so I may be wrong. I'd really like to see TrainerFu tightened up to recognize trainers, and add location icons, no matter what their titles.
This should also not happen anymore. I was trying to make the addon smart and "detect" trainers even if you don't knew the professions he was training. I removed that code and now it only detects a trainer and make a note if you actually can see that training window.
I use TrainerFu's option to show trainer icons in Cartographer however I find that they don't all show up. I believe TrainerFu is using "trainer" as a keyword and so when trainers with titles like "Cook", "Herbalist", "Skinner", "Miner", and "Engineer" are encountered (those without the word "Trainer" in their titles), TrainerFu won't add a location icon to the map. I know this is the case for at least one cook in Silvermoon City and a Skinner whose location I can't remember off-hand. On the other hand, Weapon Masters show up fine but don't have "Trainer" in their titles, so I may be wrong. I'd really like to see TrainerFu tightened up to recognize trainers, and add location icons, no matter what their titles.
This should also not happen anymore. I was trying to make the addon smart and "detect" trainers even if you don't knew the professions he was training. I removed that code and now it only detects a trainer and make a note if you actually can see that training window.
It's still happening in r35096. I just opened the training window for the Superior Cook in Grom'gol and I did not get a map icon for him. Ditto for the Superior Herbalist. Updated yesterday.
It's still happening in r35096. I just opened the training window for the Superior Cook in Grom'gol and I did not get a map icon for him. Ditto for the Superior Herbalist. Updated yesterday.
Sorry i wasn't more clear. I meant in branch version. I haven't touched the trunk (the version you get when downloading a zip/using a updater) for a long time.
I will probably merge the branch tonight when i get home.
It's still happening in r35096. I just opened the training window for the Superior Cook in Grom'gol and I did not get a map icon for him. Ditto for the Superior Herbalist. Updated yesterday.
Sorry i wasn't more clear. I meant in branch version. I haven't touched the trunk (the version you get when downloading a zip/using a updater) for a long time.
I will probably merge the branch tonight when i get home.
Since I have no idea what a branch version is, I hope you do.
Anything you see under trunk is what most people download. It's the latest official version.
Anything you see under branches is a branch version, and under each addon in that folder, there might be more than one branch version based on whether more people have tinkered with ideas for that addon.
I have no idea about the tags folder.
To download a branch version, you can for instance go to http://www.wowace.com/trac/browser/branches/FuBar_TrainerFu?order=name and at the bottom of the screen in the center is "Download in other formats: Zip archive". You'd want to select that if you want to download that. If you want to only look at it, you can click around in each folder and stuff, to take a look. Make sure, if you do install branches, that you move the correct folder to your addons directory. It's kind of a funky deal.
Thanks for the info! While it seems not too complicated to get a branch version, I'd much rather see the trunk updated. It's much more efficient to run the updater mostly because I don't need to know if there's a new version and also because the former seems much less stable to me. A delusion I'm sure, but I'm prone to those. ;)
On my reroll, no trainers indicated in trainerfu, i can see some spells if a activate all options, but all spells are indicated lvl 70...
On my main, trainerfu say i can train all comp herbalism, from 1 to 300. Too bad, i already have herbalism maxed to 375...
All was good before update last night (before you merge branch with trunk, right ?)
Good luck,
Yeah well there are some problems:
Old saved data is useless cus i have changed the data format, hopefully for the better. Just revisit trainers :-(
This piece of junk is probably in a bigger need of localization. I think there are some strings that have to be translated, but i am not sure.
Well anyway, i know spells might be shown as trainable, and in that case you can ctrl-alt-click it to mark it as learned. I have tried to make special case filters for those spells but i guess it's broken on localized clients.
Ive also found some strings that is missing in the enUS locals that i have to add. :-/
Just logged in with my pally and after attempting to configure TrainerFu, I got this error:
[2007/05/23 16:31:12-113-x1]: FuBar_TrainerFu-0.2\TrainerFu_FuBar.lua:71: attempt to compare number with nil
FuBarPlugin-2.0-34599:343: in function `UpdateText'
FuBarPlugin-2.0-34599:761: in function `func'
Dewdrop-2.0-36535 (FuBar):523: in function <...erface\AddOns\FuBar\libs\Dewdrop-2.0\Dewdrop-2.0.lua:462>
Just logged in with my pally and after attempting to configure TrainerFu, I got this error:
[2007/05/23 16:31:12-113-x1]: FuBar_TrainerFu-0.2\TrainerFu_FuBar.lua:71: attempt to compare number with nil
FuBarPlugin-2.0-34599:343: in function `UpdateText'
FuBarPlugin-2.0-34599:761: in function `func'
Dewdrop-2.0-36535 (FuBar):523: in function <...erface\AddOns\FuBar\libs\Dewdrop-2.0\Dewdrop-2.0.lua:462>
---
OK, I've added a extra check that hopefully solve this bug.
I deleted my old TrainerFu and it's SavedVariables files, reinstalled it from the trunk, and my issue still persists:
I am level 34 and have not learned my level 34 mage skills yet. I have revisited the Mage Trainer in UC. In the tooltip, it correctly shows that I have level 34 skills I can learn now, as well as the skills I can learn at level 36.
However, the FuBar text display remains "0"...it doesn't show that I have three skills I can learn, or how much it'll cost to learn them. I've tried everything I can think of.
The trunk TrainerFu r36630 is throwing up errors. Various errors first started occurring after switching to the heavily revised trunk r36407 yesterday. USEng client, WoW 2.1 live.
"FuBar_TrainerFu-0.2\\TrainerFu.lua:200: bad argument #1 to 'find' (string expected, got nil)\n<in C code>: ?\nAceEvent-2.0-36539 (Ace2):320: in function `TriggerEvent'\nAceEvent-2.0-36539 (Ace2):1019: in function <Interface\\AddOns\\Ace2\\AceEvent-2.0\\AceEvent-2.0.lua:1012>\n\n" occurs often. And it stops working after that.
Have not gone to any new trainers since the r36407 update on Tuesday. Don't know when its happening as the bugs don't appear in the BugSack log (unlike most other errors), only in the !BugGrabber.lua saved variables file after exiting.
And another thing: the revised edition does not seem to be deleting old saved variables info (in an individual character's FuBar_TrainerFu.lua saved variables file), as a mod should do if it uses a different data structure and can't use old data. Instead, it seems to be adding to them/changing them. Of course, database mods should always have some way of smoothly updating data, but hey.
EDIT: r35096 with its original (backed up) saved variables files is working fine so far in WoW 2.1, with no errors. That's the last version before the author moved the branch version into the trunk. Haven't been to a trainer yet, but the rest of the functionality (showing trainers in Cartographer, clicking trainable skills in the list and have the description pop up, etc.) is working fine.
Rollback Post to RevisionRollBack
To post a comment, please login or register a new account.
When you train soemthing from the first of 2 trainers available, TrainerFu doesn't see this and thinks you still need to learn it from the second one. It clears when talking to the second trainer.
I've wasted a lot of time running to a trainer on toons I have not log'd in a while, only to find I had already trained.
Until you get to skill 300 in enchanting, the grand master enchanting trainer can't teach you anything. But below that, there is some overlap between artisan and expert, and so on. It'd be nice if they all fell in the same category. And of course the example of the cooking trainer.
Also, it would make sense to me to note which weapons trainer teaches what. I mention this because while my low level is an alt and I have much more money on another character, I didn't want to hand it down from my main. Meanwhile, I did go to a few weapons trainers and their skills all fall in the same category so if I wanted to learn something specific I wouldn't know who to go to right off, I'd have to visit the nearest weapons trainer and ask from there.
That should not happen i the branch version.
I'll see what i can do about that. I am trying make the addon as "general" as possible but in this case i can see the need for some special treatment.
This should also not happen anymore. I was trying to make the addon smart and "detect" trainers even if you don't knew the professions he was training. I removed that code and now it only detects a trainer and make a note if you actually can see that training window.
It's still happening in r35096. I just opened the training window for the Superior Cook in Grom'gol and I did not get a map icon for him. Ditto for the Superior Herbalist. Updated yesterday.
Sorry i wasn't more clear. I meant in branch version. I haven't touched the trunk (the version you get when downloading a zip/using a updater) for a long time.
I will probably merge the branch tonight when i get home.
Since I have no idea what a branch version is, I hope you do.
Anything you see under trunk is what most people download. It's the latest official version.
Anything you see under branches is a branch version, and under each addon in that folder, there might be more than one branch version based on whether more people have tinkered with ideas for that addon.
I have no idea about the tags folder.
To download a branch version, you can for instance go to http://www.wowace.com/trac/browser/branches/FuBar_TrainerFu?order=name and at the bottom of the screen in the center is "Download in other formats: Zip archive". You'd want to select that if you want to download that. If you want to only look at it, you can click around in each folder and stuff, to take a look. Make sure, if you do install branches, that you move the correct folder to your addons directory. It's kind of a funky deal.
and now it's broken in french client :D
On my reroll, no trainers indicated in trainerfu, i can see some spells if a activate all options, but all spells are indicated lvl 70...
On my main, trainerfu say i can train all comp herbalism, from 1 to 300. Too bad, i already have herbalism maxed to 375...
All was good before update last night (before you merge branch with trunk, right ?)
Good luck,
Yeah well there are some problems:
- Old saved data is useless cus i have changed the data format, hopefully for the better. Just revisit trainers :-(
- This piece of junk is probably in a bigger need of localization. I think there are some strings that have to be translated, but i am not sure.
Well anyway, i know spells might be shown as trainable, and in that case you can ctrl-alt-click it to mark it as learned. I have tried to make special case filters for those spells but i guess it's broken on localized clients.Ive also found some strings that is missing in the enUS locals that i have to add. :-/
[2007/05/23 16:31:12-113-x1]: FuBar_TrainerFu-0.2\TrainerFu_FuBar.lua:71: attempt to compare number with nil
FuBarPlugin-2.0-34599:343: in function `UpdateText'
FuBarPlugin-2.0-34599:761: in function `func'
Dewdrop-2.0-36535 (FuBar):523: in function <...erface\AddOns\FuBar\libs\Dewdrop-2.0\Dewdrop-2.0.lua:462>
---
OK, I've added a extra check that hopefully solve this bug.
I am level 34 and have not learned my level 34 mage skills yet. I have revisited the Mage Trainer in UC. In the tooltip, it correctly shows that I have level 34 skills I can learn now, as well as the skills I can learn at level 36.
However, the FuBar text display remains "0"...it doesn't show that I have three skills I can learn, or how much it'll cost to learn them. I've tried everything I can think of.
Have posted over in the "official" thread about the buggy new trunk version, but as the author seems to be more active in this thread, reposting this here.
The trunk TrainerFu r36630 is throwing up errors. Various errors first started occurring after switching to the heavily revised trunk r36407 yesterday. USEng client, WoW 2.1 live.
"FuBar_TrainerFu-0.2\\TrainerFu.lua:200: bad argument #1 to 'find' (string expected, got nil)\n<in C code>: ?\nAceEvent-2.0-36539 (Ace2):320: in function `TriggerEvent'\nAceEvent-2.0-36539 (Ace2):1019: in function <Interface\\AddOns\\Ace2\\AceEvent-2.0\\AceEvent-2.0.lua:1012>\n\n" occurs often. And it stops working after that.
Have not gone to any new trainers since the r36407 update on Tuesday. Don't know when its happening as the bugs don't appear in the BugSack log (unlike most other errors), only in the !BugGrabber.lua saved variables file after exiting.
And another thing: the revised edition does not seem to be deleting old saved variables info (in an individual character's FuBar_TrainerFu.lua saved variables file), as a mod should do if it uses a different data structure and can't use old data. Instead, it seems to be adding to them/changing them. Of course, database mods should always have some way of smoothly updating data, but hey.
EDIT: r35096 with its original (backed up) saved variables files is working fine so far in WoW 2.1, with no errors. That's the last version before the author moved the branch version into the trunk. Haven't been to a trainer yet, but the rest of the functionality (showing trainers in Cartographer, clicking trainable skills in the list and have the description pop up, etc.) is working fine.