WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#24743 closed defect (bug) (fixed)

Revisions: Don't show 'false' in a tooltip when gravatars are disabled

Reported by: ocean90 Owned by: markjaquith
Priority: normal Milestone: 3.6
Component: Revisions Version: 3.6
Severity: normal Keywords: has-patch commit 2nd-opinion
Cc: Focuses:

Description


Attachments (2)

24743.patch​ (655 bytes ) - added by ocean90 13 years ago.
24743.diff​ (2.3 KB ) - added by nacin 13 years ago.

Download all attachments as: .zip

Change History (6)

@ocean90
13 years ago

#1 @ocean90
13 years ago

  • Keywords has-patch added; needs-patch removed

#2 @markjaquith
13 years ago

  • Keywords commit 2nd-opinion added

This is good to go. Can I get another sign-off on it?

@nacin
13 years ago

#3 @nacin
13 years ago

Looks good. Here's something to speed things up a bit, especially when the same author does repeated edits.

#4 @markjaquith
13 years ago

  • Owner set to markjaquith
  • Resolution → fixed
  • Status new → closed

In 24708:

Improvements to output of revisions author data.

  • Do not output "false" if avatars are disabled. props ocean90.
  • Cache the author data separately to cut down on calls to get_the_author_meta() and get_avatar(). props nacin.

Fixes #24743.

Note: See TracTickets for help on using tickets.