WordPress.org

Make WordPress Core

#24743 closed defect (bug) (fixed)

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

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

Description


Attachments (2)

24743.patch (655 bytes) - added by ocean90 12 months ago.
24743.diff (2.3 KB) - added by nacin 12 months ago.

Download all attachments as: .zip

Change History (6)

ocean9012 months ago

comment:1 ocean9012 months ago

  • Keywords has-patch added; needs-patch removed

comment:2 markjaquith12 months ago

  • Keywords commit 2nd-opinion added

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

nacin12 months ago

comment:3 nacin12 months ago

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

comment:4 markjaquith12 months ago

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