I think that is one more reason to bring back player hate lists. Removing them solves a few minor issues, but makes certain other things considerably harder to code properly. All we need to do is allow player hate lists again, but add WipeHateList() to a few places like FD, Escape, and Fading Memories (and a few others).
If someone knows all cases that WipeHateList() would be required if we restore player hate lists, I think it will be considerably easy to get them back to how they were before and still have all hate list issues fixed. Or we could always restore it now and just fix the minor issues it causes later when they are seen.
I think the IsEngaged() check for players is just too valuable to not have it available to us to use. Heck, if possible, I would be happy if we could just make a second hate list that was for clients only. That would resolve both issues at the same time, but I don't really know what would be involved in making that happen. But, if we did, we could just have a new command like ClientEngaged() or something.
|