- Status Closed
- Percent Complete
- Task Type Feature Request
- Category Engine → Combat
- Assigned To No-one
- Operating System
- Severity Low
- Priority
- Reported Version
- Due in Version Undecided
-
Due Date
Undecided
- Votes
- Private
Attached to Project: PlaneShift
Opened by Loyni - 23.10.2008
Last edited by Steven Schwartfeger - 10.02.2009
Opened by Loyni - 23.10.2008
Last edited by Steven Schwartfeger - 10.02.2009
FS#2402 - Change command /assist so it can't be used to determine enemy's target.
Command /assist can be used to determine enemy’s target.
I suggest /assist to be usable only on guild or group members,
except for PvP areas, where it should be freely used on other players (inside the PvP).
Closed by Steven Schwartfeger
10.02.2009 08:32
Reason for closing:
Additional comments about closing:
10.02.2009 08:32
Reason for closing:
Additional comments about closing:
Tested.
Feature request, bug reports are for when it's not working like how it seems the developer designed it, not for when you disagree
haha, you're right :p
First of all I don't agree with Vornne's comment. This seems like a very good catch of an oversight on the devs' part. Thanks for reporting it.
My suggestion to the prospect who fixes this is to add validation to the /assist command on the server (in src/server/usermanager.cpp) to make sure the target being assisted is grouped with the person doing the assisting, or in the same guild. If they fail this validation, the requester should get an informative error message. Else the current processing should continue as normal.
Created and tested this patch for trunk r2894
Committed to trunk (r2807) by Vengeance.