Skip to:
Content

BuddyPress.org

Opened 15 years ago

Closed 14 years ago

#759 closed enhancement (fixed)

Patch for Gravatar/Avatar/Mystery Man

Reported by: johnjamesjacoby's profile johnjamesjacoby Owned by:
Milestone: 1.5 Priority: minor
Severity: Version:
Component: Core 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 15 years ago.

Download all attachments as: .zip

Change History (5)

#1 @buzz_lightyear
15 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)
15 years ago

  • Milestone Core 1.1 deleted

Milestone Core 1.1 deleted

#3 @DJPaul
14 years ago

  • Component set to Core
  • Keywords has-patch removed
  • Milestone set to 1.3

JJJ please would you take a look at this one thanks

#4 @johnjamesjacoby
14 years ago

  • Resolution set to fixed
  • Status changed from new to 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.