quest::traindiscs()

**trevius** · #1 09-04-2008, 03:04 AM

Well, I was going by the example of scribespells. Currently I just allow the use of #traindisc command on my server, but I would love to remove the need for that. Though, I do think it would be nice if there was an option to train up to a certain level. But, I think it should be set so that if you don't set anything it will go by current level as the max.

So_1337 · #2 09-04-2008, 08:49 AM

As AndMetal pointed out, I think that that's one of the failings of quest::scribespell, and it'd be nice to not repeat the same mistake here =P

I know I'd intended for every class to be able to have their spells scribed up to level 51 on my server, but I had to change it for mages; I didn't want them getting a free epic. Having more flexibility with the scribespell function (and the proposed traindisc) can only help.

Secrets · #3 09-04-2008, 11:29 AM

Quote:

Originally Posted by So_1337

I know I'd intended for every class to be able to have their spells scribed up to level 51 on my server, but I had to change it for mages; I didn't want them getting a free epic. Having more flexibility with the scribespell function (and the proposed traindisc) can only help.

Just a quick FYI, if you don't want people getting spells on scribespells, change the spell file SERVERSIDED ONLY to class level 255 on that spell that you don't want. This will disable the spell.

Provided you do not have spell scrolls, of course, then it's another issue altogether.

So_1337 · #4 09-04-2008, 11:35 AM

I want them to have the spell available to them, just not via a spell-scribing NPC. If they complete the magician epic, they sure deserve to use it =P

That's a good tip for some other spells that might be included in what are scribed, though, such as if I wanted to limit spells from the scriber by different eras. Then update the file once a new era is unlocked. (We're currently doing progression.)

Thanks for the tip, didn't consider that =)

**trevius** · #5 09-05-2008, 12:20 AM

Good point. I supposed that if scribespells worked the ideal way, you could just do quest::scribespells($ulevel); to achieve the same effect that it currently does, but you would still have the option to set any other level you wanted it to be.

And, if you don't want a class to have a certain spell, it is probably best to scribe them all and then unscribe the 1 spell with quest::unscribespell();. Though, you might need to use a timer to delay and make sure the scribing finishes before the unscribe happens.

So_1337 · #6 09-05-2008, 08:49 AM

Another interesting idea. Thank you so much. Sorry for having derailed the thread a bit, but I don't think you're in any danger. Your idea's quite a nice proposal, I think it's a no-brainer, even =P

**trevius** · #7 09-06-2008, 06:30 AM

I moved this post to Server Code Submissions. I tested out the code in my first post and had to change "t" to "initiator" (which I already edited into the post), and it works perfectly as intended. I think this is ready for the official source.

This is for the discipline version of the quest::scribespells() command. It works almost exactly the same way.

If someone wanted to make new versions for both scribespells and traindiscs that has an argument option, then please do. The idea for an argument that allows you to define how high to scribe/train to in level has already been mentioned, but that is a little above my coding level still lol. I don't actually code anything, I just steal code from elsewhere and use it from example.