/skill command

[John Adams]

Can we get a manual /skill command added that will list, add, and remove skills from players?

Code: Select all

/skill [ add | remove | list ] [skill_name] {target}

Where:

/skill list Dagger -- will show me a list of skill_name's related to daggers

/skill add dagger Scatman -- will add the Dagger skill to Scatman's character_skills record

/skill remove dagger Scatman -- will remove the Dagger skill from Scatman's character_skills record



If {target} is not a name, make it add / remove the skill to the current targeted character.


TIA

[LethalEncounter]

Yah good idea

[John Adams]

Scatman has started looking into this, but had to run for a while. Not sure if he started it, or is just thinking it over. If you have any ideas, let's lay them out.

One thing he considered that I didn't, /skill remove skill_name {Player} might be a detriment if a malicious player decided to strip someone's skills heh... so we've decided that the {Player} param will only work if you are a GM.

[Scatman]

I haven't started implementing it but I can get to it today. Also, what about a /skill list command or something similar to list all the commands from the DB?

Edit: Nvm, just saw you already mentioned that

[Scatman]

Hmm, looking at how we handle the required status for commands I don't think we'll be able to allow only clients with the required status to use the command differently. We check the required status before we even enter the command handler and if it's not enough, we exit.

I could tell the check to ignore checking the admin status up front if the command is 'skill'. Otherwise we only allow GMs to use the command.

[John Adams]

Nah Scat, let's just do it like a normal command, and make it self-only.

We can make a second command for GMs: /giveskill or something, if we need to.

[LethalEncounter]

Or just add a check in the command block that only allows flagged accts access to certain features.