Download
(15Kb)
Download
Updated: 04-17-07 11:54 AM
Pictures
File Info
Updated:04-17-07 11:54 AM
Created:unknown
Downloads:9,707
Favorites:65
MD5:

DemonTrainer (Continued)  Popular! (More than 5000 hits)

Version: q2.4-20003
by: quel [More]

Continuation of DemonTrainerFrame, updated for use with WoW 2.0.

Converts the merchant for warlock pets' books to a more friendly, trainer-style format.

Changes in 2.4
*might* solve the problem some users have reported with blank windows. This is almost certainly caused by some other mod you've installed conflicting with the vendor tooltip, DTF 2.4 includes more robust tooltip parsing to attempt to account for this.

Changes in 2.3
Fixed nil errors regarding the UnHookSecureFunction pseudo code left in by accident. This is largely cosmetic, though. There are no functional changes between 2.2 and 2.3.

Changes in 2.2
Due to the secure mapping and tainting functionality added in WoW2.0, there is no way for DemonTrainer to prevent you from accidentally reselling grimoires back to the trainer. If you attempt to learn the grimoire while DemonTrainer is open, it will sell the book back to the vendor. YOU MUST CLOSE DEMON TRAINER BEFORE ATTEMPTING TO LEARN NEW GRIMOIRES.

Since I'm unable to hook the functions to prevent accidental reselling, I've added two popups, one when you open DemonTrainer, another that triggers if/when you accidentally sell the book back. You'll still lose money, but at least you'll be aware of what happened.

Optional Files (0)


Post A Reply Comment Options
Unread 11-22-07, 03:47 PM  
Althena
A Kobold Labourer
 
Althena's Avatar

Forum posts: 0
File comments: 8
Uploads: 0
Very fine Addon, but i have the Empty Frame too. I have deactivate some Addons but i can´t fix that Problem.

A Tip for the next Version. Change the Addon like Lywellyn it has made it. No one wants a Popup when he open the Window.


And here are my German Translation for the Warning:
- Pleaaaase don´t translate only some Words of a sentence I have to read it here in English bevor i understand what you want to say

DTF_WARNING = "|cffff0000Du musst erst das Fenster des Demon Trainer schlie\195\159en bevor du deinem Pet einen neuen Zauberfolianten lernst. Wenn du das nicht tust, dann wird der Zauberfoliant mit Verlust zur\195\188ckverkauft.|r"

DTF_SOLDITEM = "Du hast soeben den Zauberfolianten mit verlusten zur\195\188ckverkauft. Es wurde Dir gesagt das Du das Demon Trainer Fenster schließen mu\195\159t bevor du deinem Pet den Zauberfolianten lernst."
If anyone knows what Addons made the Problems with the Empty Frame. Please let us know.

Ash-rak
Last edited by Althena : 11-22-07 at 03:48 PM.
Report comment to moderator  
Reply With Quote
Unread 07-24-07, 11:28 AM  
Dreadlorde
A Pyroguard Emberseer
 
Dreadlorde's Avatar
AddOn Author - Click to view AddOns

Forum posts: 2302
File comments: 141
Uploads: 4
Is this being updated anymore? It still works, just wondering if it every breaks ifit's getting an update.
__________________

Funtoo - Plan 9 - Windows 7
Report comment to moderator  
Reply With Quote
Unread 06-04-07, 10:23 AM  
BorgkingZero
A Kobold Labourer
 
BorgkingZero's Avatar

Forum posts: 1
File comments: 21
Uploads: 0
Sorry i did the Fix but i have the same Error again.

Every time i speak with the Demon Trainer the Window is empty and i have the message: "You must close DemonTrainer before you attempt to learn a new grimoire. If you don't, you will sell the grimoire back at a loss." in the Chat Window.

Sorry i dont now what's wrong.
Report comment to moderator  
Reply With Quote
Unread 05-02-07, 04:23 AM  
Lown Ignitus
A Defias Bandit

Forum posts: 2
File comments: 5
Uploads: 0
thing is this is same message that i got when i originally posted that in had issues, and i even installed the update since my post, with same error message....
Report comment to moderator  
Reply With Quote
Unread 05-01-07, 08:22 AM  
Lywellyn
A Wyrmkin Dreamwalker
 
Lywellyn's Avatar

Forum posts: 53
File comments: 34
Uploads: 0
Until there's some sort of official reply, you can patch this mod with my edits a couple comments down to fix that error.
__________________
Report comment to moderator  
Reply With Quote
Unread 05-01-07, 03:51 AM  
Lown Ignitus
A Defias Bandit

Forum posts: 2
File comments: 5
Uploads: 0
Error message:

C:\Program Files\World of Warcraft\Interface\AddOns/DemonTrainerFrame/DemonTrainerFram.lua:372: attempt to call global 'message' (a string value)
and nothing shows up.
Report comment to moderator  
Reply With Quote
Unread 04-29-07, 07:49 PM  
Xanithon
A Cobalt Mageweaver
 
Xanithon's Avatar
AddOn Compiler - Click to view compilations

Forum posts: 203
File comments: 5
Uploads: 2
Yay! Updated Warlock Pet goodness!
__________________


Report comment to moderator  
Reply With Quote
Unread 04-24-07, 04:42 AM  
Lywellyn
A Wyrmkin Dreamwalker
 
Lywellyn's Avatar

Forum posts: 53
File comments: 34
Uploads: 0
A fix, if you will.

First off, let me say that this is an excellent mod, I really do appreciate its existence because the vendor frame for training was rather dumb.

This was sort of posted prior, but that post completely eliminated the message, which takes away from the purpose of the message. I simply did the following instead, and it works for me:

Around line 372 in the DemonTrainerFrame.lua file, find:

Message(DTF_WARNING);
and change it to:

DEFAULT_CHAT_FRAME:AddMessage(DTF_WARNING);
When I did this, instead of a pop up box (which I am a firm believer that a constant pop up box is REALLY annoying), it shoots the message to your Default chat frame. But when I tested it, the text was all white. That's not very eye-catching, so I decided I'd change that text to red, as so:

In the localization.lua file for DTF, find:

DTF_WARNING = "You must close DemonTrainer before you attempt to learn a new grimoire. If you don't, you will sell the grimoire back at a loss."
and change it to:

DTF_WARNING = "|cffff0000You must close DemonTrainer before you attempt to learn a new grimoire. If you don't, you will sell the grimoire back at a loss.|r"
That'll make it nice and red, stands out. I think this would serve the same purpose as the popup box: Lets new users of the mod know that trying to use a grimoire while the DTF is open will sell it, but doesn't annoy users that have been using the mod for a while and are used to that fact.

Just my two cents. :3
__________________
Last edited by Lywellyn : 04-24-07 at 04:57 AM.
Report comment to moderator  
Reply With Quote
Unread 04-18-07, 08:51 AM  
quel
A Kobold Labourer
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 14
Uploads: 3
Re: error report

Since the errors being reported on various sites do not occur on my system, I must assume they are the result of conflicts with other mods. We need someone to spend a mere 10 minutes disabling other addons until the DTF errors go away. Without some useful debugging info, I don't even have a guess about what needs fixing.

Q

P.S. I also need context details about what you were doing: which trainer were you at? which pet did you have out? What level are you?
Last edited by quel : 04-18-07 at 08:53 AM.
Report comment to moderator  
Reply With Quote
Unread 04-18-07, 04:34 AM  
Tweeker
A Cobalt Mageweaver
 
Tweeker's Avatar
AddOn Compiler - Click to view compilations

Forum posts: 234
File comments: 153
Uploads: 5
error report

[2007/04/18 06:33:15-548-x1]: DemonTrainerFrame\DemonTrainerFrame.lua:372: attempt to call global 'message' (a table value)
<string>:"DemonTrainerFrame:OnShow":2: in function <[string "DemonTrainerFrame:OnShow"]:1>
<in C code>: in function `Show'
Interface\FrameXML\UIParent.lua:915: in function `SetLeftFrame':
Interface\FrameXML\UIParent.lua:815: in function `ShowUIPanel':
DemonTrainerFrame\DemonTrainerFrame.lua:94: in function `MerchantFrame_OnEvent'
<string>:"MerchantFrame:OnEvent":2: in function <[string "MerchantFrame:OnEvent"]:1>
__________________
Report comment to moderator  
Reply With Quote
Unread 04-17-07, 02:54 PM  
Malkom
A Kobold Labourer

Forum posts: 0
File comments: 29
Uploads: 0
I've got the same error with this new version


[2007/04/17 22:58:10-488-x1]: DemonTrainerFrame\DemonTrainerFrame.lua:372: attempt to call global 'message' (a table value)
<string>:"DemonTrainerFrame:OnShow":2: in function <[string "DemonTrainerFrame:OnShow"]:1>
<in C code>: in function `Show'
Interface\FrameXML\UIParent.lua:915: in function <Interface\FrameXML\UIParent.lua:903>:
<in C code>: in function `SetLeftFrame'
Interface\FrameXML\UIParent.lua:815: in function `ShowUIPanel':
DemonTrainerFrame\DemonTrainerFrame.lua:94: in function `MerchantFrame_OnEvent'
<string>:"MerchantFrame:OnEvent":2: in function <[string "MerchantFrame:OnEvent"]:1>

---
Last edited by Malkom : 04-17-07 at 02:59 PM.
Report comment to moderator  
Reply With Quote
Unread 04-16-07, 07:01 PM  
Toran
A Nerdscale Dorkin
 
Toran's Avatar
Premium Member

Forum posts: 143
File comments: 433
Uploads: 0
Originally posted by Darkspell
Hrmm... pretty standby message when things do not work I agree.

I just commented out line 359:

message(DTF_WARNING);


No more errors and it seems to work.
That fixed me too.
Report comment to moderator  
Reply With Quote
Unread 04-14-07, 07:03 AM  
Eries
A Kobold Labourer

Forum posts: 0
File comments: 3
Uploads: 0
Fehler: attempt to call global 'message' (a table value)
AddOn: DemonTrainerFrame
Datei: DemonTrainerFrame.lua
Zeile:359
Fehler Nummer:1
--------------------------------------------------
Fehler: attempt to index field '?' (a nil value)
AddOn: DemonTrainerFrame
Datei: DemonTrainerFrame.lua
Zeile:837
Fehler Nummer:2
Report comment to moderator  
Reply With Quote
Unread 04-08-07, 02:49 PM  
Darkspell
A Deviate Faerie Dragon

Forum posts: 17
File comments: 47
Uploads: 0
Hrmm... pretty standby message when things do not work I agree.

I just commented out line 359:

message(DTF_WARNING);


No more errors and it seems to work.
Report comment to moderator  
Reply With Quote
Unread 04-07-07, 08:08 PM  
quel
A Kobold Labourer
AddOn Author - Click to view AddOns

Forum posts: 0
File comments: 14
Uploads: 3
Re: Does not work

Originally posted by Darkspell
Either fix it or remove it.. really. Would love if this mod DID work, but all I get are:

Attempt to call global 'message' error in line 359

and

'for' limit must be a number in line 518
Shrug. It works fine on my system. I can't reproduce either of those errors or the blank frame mentioned below. The usual "disable the rest of the addons and see what happens" rule applies. WIthout something more useful to go, there's not likely to be any kind of fix.
Report comment to moderator  
Reply With Quote
Post A Reply



Category Jump: