quest::traindiscs

[Zeice]

Ok I'm pretty sure this was working before, but now it's not working correctly. The GM command for training discs works fine, however I have a merchant that trains people's discs for them. Warriors are only getting 4 of their spells, but if I use the GM command they earn 40. I've used several variations and tests to try and get it to work, but only the GM command seems to be working.

Things I've tried:


quest::traindiscs(65);
quest::traindiscs(65,51);
quest::traindiscs(65,1);



None appear to be working properly and I'm wondering if it may have been something to do with Andmetal's changes. Granted I'm not at the latest revision so that may be it, I'll have to update and check it out. I'm currently at rev736.

[AndMetal]

It wouldn't surprise me. I'm pretty sure I remember testing this, but I could be mistaken (initially I had a >= instead of <= for one of the checks, but corrected it before committing).

I'll be working on a few additional things, so if I can, I'll try to reproduce this.

However, you may also want to check the zone log files that begins with "Train disciplines request" to make sure it's showing the correct level range.

[Congdar]

AndMetal,
It looks like the min_level isn't being used... max_level is in there twice so must be one of those is supposed to be min_level?

[AndMetal]

Ah yup, that would be it :-)