-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Control Companion #1818
base: master
Are you sure you want to change the base?
Control Companion #1818
Conversation
var associate = GetHenchman(user); | ||
if (target == associate) | ||
{ | ||
Enmity.ClearEnmityTables(associate); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is abusable. We shouldn't be letting players clear enmity tables of their droids/beasts.
Removed the part of the code that cleared enmity tables.
Undid the change to the Enmity service, since ClearEnmityTables is no longer used by the command.
I've updated it to not clear enmity tables. That was in place to ensure creatures were always receptive to orders but I suppose enmity on NPC companions getting in the way of them following orders may make enmity a more important mechanic. Maybe down the line a "Calm Pet" beastmaster command could be added. |
This chat command will target a given NPC, unlocked door or placeable.
-If the NPC is hostile, or the target is a pleaceable, the user's henchman will attack it.
-If the target is an unlocked door, the henchman will open it.