Opened 9 years ago
Last modified 6 years ago
#36030 new enhancement
Expose site icon on wp.getUsersBlogs
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Future Release | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | XML-RPC | Keywords: | has-patch needs-unit-tests 2nd-opinion |
| Focuses: | Cc: |
Description
WordPress 4.3 has added the ability for site owners to manage their site’s favicon, but never exposed it over the XML-RPC protocol.
It's useful for XML-RPC clients to receive it back in the response of wp.getUsersBlogs, so they can show the proper icon beside the name of the site.
In the patch I've provided an empty value is returned if site_icon is not set on the blog. I've avoided returning a default value, since it's better to leave this responsibility to clients. If siteIcon is empty, the client should show their default icon, or nothing.
Attachments (1)
Change History (3)
Note: See
TracTickets for help on using
tickets.
Hi @daniloercoli,
Thanks for this! My apologies that it took so long to receive a response.
The patch still applies cleanly for me. XML-RPC has been in somewhat of a holding pattern not receiving much attention lately. I don't think it would hurt to add this, but I'd love to get more thoughts from other committers/contributors more familiar with the component.
It would also be great to add unit tests for these methods in the process.