Remove unneeded eager loading for user show page

[marcusmoore]

This PR removes some unneeded related model loading for the user show page.

Most of the tables on the user show page are loaded via the api so we don't need to pass through assets, assets.model, or userlog.item.

When attempting to view a user with a lot of previous activity, the $user->userlog->load('item') line could potentially end up causing a memory exhaustion exception.

[what-the-diff]

PR Summary

• Improved Readability of User Model Relationships Loading

  • The changes add clarity to the way our software retrieves connected data. Instead of chaining commands, we now use an array-based approach. This enhances the tidiness and readability of our code, reducing the potential for confusion.

• Enhanced Simplicity in Returning User Data

  • The return statement's structure has been simplified by using an easy-to-understand array. Instead of compacting user and userlog into a single package, the new approach includes just user and settings. It makes processing the returned data more streamlined, resulting in better overall software performance.