Commands to modify max-vitals and skills #2219
Conversation
dgarson
changed the title
| public static void HandleModifyVital(Session session, params string[] parameters) | ||
| { | ||
| var lastAppraised = CommandHandlerHelper.GetLastAppraisedObject(session); | ||
| if (lastAppraised == null || !(lastAppraised is Creature)) return; |
There was a problem hiding this comment.
Might be useful to output text to the admin letting them know why the command failed.
| [CommandHandler("modifyvital", AccessLevel.Admin, CommandHandlerFlag.None, 2, "Adjusts the maximum vital attribute for the last appraised mob/player and restores full vitals", "<Health|Stamina|Mana> <delta>")] | ||
| public static void HandleModifyVital(Session session, params string[] parameters) | ||
| { | ||
| var lastAppraised = CommandHandlerHelper.GetLastAppraisedObject(session); |
There was a problem hiding this comment.
my only concern about GetLastAppraisedObject() is how decal can automatically appraise other objects invisibly / behind-the-scenes, and unintentionally change the intended target of the player
there is also a field in ACE which points to the 'currently selected creature target' (anything with vitals), and this selection is not automatically changed by decal, as far as i know... perhaps it would be worth considering to use that variant?
There was a problem hiding this comment.
This seems like a good idea. I found the selectedTarget field in Player class but it's a Guid. What's the easiest way to convert that to a Creature instance?
There was a problem hiding this comment.
Something like this should work:
Creature targetCreature = CurrentLandblock?.GetObject(Guid) as Creature;
Adds two new admin commands for (1) modifying the max vitals for an NPC or player, (2) modifying the skill levels for an NPC or player
#2209 and #2208