Support WP_User objects

[mattheu]

Description of the Change

It is expected that I can pass WP_User object to get_avatar, as is documented. But this does not work because get_user_id in Simple_Local_Avatars does not handle this case.

Closes #192

How to test the Change

$user = wp_get_current_user();
echo get_avatar( $user );

Changelog Entry

Fixed - Support passing WP_User to get_avatar.

Credits

Props @mattheu

Checklist:

• I agree to follow this project's Code of Conduct.
• I have updated the documentation accordingly.
• I have added tests to cover my change.
• All new and existing tests pass.

[mattheu]

I'm not so familiar with WP_Mock, but I've made some changes so that I can mock a WP_User object. Please advise if there is a better way to do this. Thanks.

[faisal-alvi]

Minor spacing fixes

[faisal-alvi]

@mattheu Awesome! thank you for working on this.

✅ Checked the following code and the avatar works well in the PR branch. Can you please just make those small spacing fixes and we will be good to go!

echo '<h1>WORKS</h1>';
$user = wp_get_current_user();
echo get_avatar( $user->ID );

echo '<h1>DOES NOT WORK</h1>';
$user = wp_get_current_user();
echo get_avatar( $user );

---

in develop:

in this PR:

[mattheu]

Thanks. Suggestions applied.

[faisal-alvi]

@mattheu thank you for your efforts! 🚀