Skip to:
Content

BuddyPress.org

Opened 17 years ago

Closed 16 years ago

#759 closed enhancement (fixed)

Patch for Gravatar/Avatar/Mystery Man

Reported by: johnjamesjacoby Owned by:
Priority: minor Milestone: 1.5
Component: Core Version:
Severity: Keywords: gravatar, mystery-man
Cc:

Description

This prevents BP from hitting gravatar.com with requests when the default avatar setting is mystery man.

This also adds a quick filter called 'bp_custom_mystery_man' so the URL to mystery-man.jpg can be switched by plugin developers or theme designers.

Attachments (1)

bp-core-avatars.patch​ (1.7 KB ) - added by johnjamesjacoby 17 years ago.

Download all attachments as: .zip

Change History (5)

@johnjamesjacoby
17 years ago

#1 @buzz_lightyear
17 years ago

this patch works great, but there's small bug in it.
Line 60 cannot be removed, but instead replaced as:

//$gravatar = 'http://www.gravatar.com/avatar/' . md5( $ud->user_email ) . '?d=' . $default_grav . '&s=';
$gravatar = 'http://www.gravatar.com/avatar/' . md5( $ud->user_email ) . '?d=' . $gravatar . '&s=';

Otherwise it doesn't work properly.

#2 @(none)
17 years ago

  • Milestone Core 1.1

Milestone Core 1.1 deleted

#3 @DJPaul
16 years ago

  • Component → Core
  • Keywords has-patch removed
  • Milestone → 1.3

JJJ please would you take a look at this one thanks

#4 @johnjamesjacoby
16 years ago

  • Resolution → fixed
  • Status new → closed

Patch is outdated, and avatar loading has been reprioritized now so Gravatar isn't always pinged.

There are filters and args now to actually bypass Gravatar all together, and a filter on the mystery-man src too.

Closing this since it's currently fixed.

Note: See TracTickets for help on using tickets.