View Single Post
  #13  
Old 09-08-2008, 08:00 AM
trevius's Avatar
trevius
Developer
 
Join Date: Aug 2006
Location: USA
Posts: 5,946
Default

Woot! Just confirmed that my last code post here for the new scribespells and traindiscs works perfectly as intended. I tried setting them both to level 10 and it only trained up to level 10. Then I tried using $ulevel and they both worked the way they used to by scribing/training up to my level!

Not a huge or complex code change, but I think it is something that has been needed for a while. And I think I am finally starting to learn a little by using examples in the source

If this makes it into the official code, then it would probably be a good idea to note that scribespells() won't work anymore unless it has the level argument. But a simple change to scribespells($ulevel) works perfectly as a replacement. So, any server using this command will need to make the appropriate changes for it to work again after the update.

Of course, the command name could be changed to add in 1 more command, but IMO this is the way that scribespells should work in the first place. And it really only requires a very minor script change for any server. I don't think it is more than asking them to add required SQL.
__________________
Trevazar/Trevius Owner of: Storm Haven
Everquest Emulator FAQ (Frequently Asked Questions) - Read It!
Reply With Quote