Getting the blog avatar should be possible outside a blogs loop

[imath]

Testing the new Blog REST API Endpoints, I've noticed the prepare_item_for_response() method was missing the blog avatar.

Looking at the Blogs component, I've then noticed the bp_get_blog_avatar() function was requiring the $blogs_template global to be set to fetch the blog avatar, which makes it unusable by the BP REST API.

The attached patch makes it possible to get the blog avatar outside the Blogs loop.

[espellcaste]

This is not the only Blog function that is attached to the blogs_template. I didn't add this because of this:

At the moment, blog avatars are simply the user avatars of the blog.

But I think it is an important change. :)

[imath]

Thanks for your feedback @espellcaste

It's a very good point about other functions. I'll also look at it.

[espellcaste]

@imath Just to confirm, did we decide to merge this one and update the other functions attachment from blogs_template?

[imath]

Just this one for 6.0.0 😊

[espellcaste]

k! This patch looks good! :)

[imath]

Nice, thanks for your feedback let's have it in!

[imath]

In 12496:

Make it possible to fetch a blog's avatar outside of the blogs loop

Passing 2 new parameters ($blog_id & $admin_user_id) to the bp_get_blog_avatar() function will return the corresponding blog's avatar without the need to run a blogs loop.

Props espellcaste

Fixes #8150