Admin actions will now log their action based on the owner of the admin datum, rather than the client attached to the usr. This means that if the admin action changes the mob's client then it will not report null.
This is safe because the usr.client must always be src.owner.
This issue affects all admin actions that sleep.
Why It's Good For The Game
This will prevent incorrect reports of null performing an admin action for actions that mutate the state of usr.client.
Testing Photographs and Procedure
Changelog
:cl:
fix: Fixes some admin actions erronously showing a null user.
/:cl:
About The Pull Request
Admin actions will now log their action based on the owner of the admin datum, rather than the client attached to the usr. This means that if the admin action changes the mob's client then it will not report null.
This is safe because the usr.client must always be src.owner.
This issue affects all admin actions that sleep.
Why It's Good For The Game
This will prevent incorrect reports of null performing an admin action for actions that mutate the state of usr.client.
Testing Photographs and Procedure
Changelog
:cl: fix: Fixes some admin actions erronously showing a null user. /:cl: