🐛 mob:facePlayer() causing crashes

[slashtangent]

I affirm:

• I understand that if I do not agree to the following points by completing the checkboxes my issue will be ignored.
• I have read and understood the Contributing Guide and the Code of Conduct.
• I have searched existing issues to see if the issue has already been opened, and I have checked the commit log to see if the issue has been resolved since my server was last updated.

OS / platform the server is running (if known)

Windows 10

Branch affected by issue

base

Steps to reproduce

Use mob:facePlayer(target, true) in your code, zone/logout and the server will crash when a mob attempts to call mob:facePlayer(target, true).

ai_container.cpp exception gets thrown if the targeted player is logged out when a mob attempts to mob:facePlayer(target, true)

Expected behavior

Server does not crash.

[WinterSolstice8]

I am making a PR to improve the safety of this so it doesnt happen but storing a player ref is prone to crashing and really not ideal in general

you should store the long-form player ID and look up that player in the future