[buddypress-trac] [BuddyPress Trac] #8179: Improved default value for site avatar

[buddypress-trac]

#8179: Improved default value for site avatar
-----------------------------------+---------------------
 Reporter:  boonebgorges           |       Owner:  (none)
     Type:  defect (bug)           |      Status:  new
 Priority:  normal                 |   Milestone:  7.0.0
Component:  Blogs                  |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  2nd-opinion has-patch  |
-----------------------------------+---------------------
Changes (by imath):

 * keywords:  2nd-opinion needs-patch => 2nd-opinion has-patch


Comment:

 [https://buddypress.trac.wordpress.org/attachment/ticket/8179/8179.patch
 8179.patch] is my suggestion to fix this ticket during 7.0.0 milestone.

 It adds a default avatar for blogs and use it if the blog do not have a
 site_icon and no `admin_user_id` was provided to the list of arguments of
 `bp_blog_avatar()`. It removes a filter that is deprecated since 1.5.

 Here's a screenshot of the list of sites with the patch applied

 [[Image(https://cldup.com/UV6s4rDS21.png)]]

 If you wan to test the patch, you'll need to make sure to get the first 2
 images from [https://cloudup.com/cKzfGLtFRXg there] and put them into the
 `src/bp-core/images` folder.

 @vapvarun I've quickly made these images, but feel free to add an improved
 versions of them to the ticket 😉

 If we commit this, the blog avatar will be:

 1. The blog's site icon if available.
 2. The Blog's admin avatar if `$args['admin_user_id']` is provided (just
 like into the BP REST API Blog avatars endpoint).
 3. The mystery-blog image by default

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/8179#comment:14>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac