WordPress.org
Get WordPress

Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#34411 closed defect (bug) (fixed)

Make get_site_icon_url() filterable

Reported by: swissspidy Owned by: swissspidy
Priority: normal Milestone: 4.4
Component: Media Version: 4.3
Severity: normal Keywords: good-first-bug has-patch
Cc: Focuses:

Description

[34903] introduces the new embeds feature, which displays the site icon in the bottom of the embed template.

In that template there's an embed_site_icon_url hook to allow filtering the site icon URL. It probably makes more sense to have a general site_icon_url filter in the get_site_icon_url().

See #32522

Attachments (1)

34411.diff​ (2.4 KB ) - added by swissspidy 11 years ago.

Download all attachments as: .zip

Change History (4)

@swissspidy
11 years ago

#1 @swissspidy
11 years ago

  • Keywords has-patch added; needs-patch removed
  • Owner set to swissspidy
  • Status new → assigned

34411.diff​ removes the two filters in the embed template with 1 filter in get_site_icon_url().

Note that the docs for embed_site_icon_url were wrongly declared twice anyway.

#2 @SergeyBiryukov
11 years ago

  • Resolution → fixed
  • Status assigned → closed

In 35379:

Site Icon: Introduce get_site_icon_url filter for the icon URL.

Remove oembed_site_icon_url filter added in [34903].

Props swissspidy.
Fixes #34411.

#3 @johnbillion
11 years ago

  • Version trunk → 4.3
Note: See TracTickets for help on using tickets.