[wp-trac] [WordPress Trac] #16434: Give site admin ability to upload favicon in Settings, General

WordPress Trac wp-trac at lists.automattic.com
Thu Mar 1 19:51:03 UTC 2012


#16434: Give site admin ability to upload favicon in Settings, General
-------------------------------------------------+------------------
 Reporter:  jane                                 |       Owner:
     Type:  feature request                      |      Status:  new
 Priority:  normal                               |   Milestone:  3.4
Component:  Administration                       |     Version:  3.1
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch needs-testing ux-feedback  |
-------------------------------------------------+------------------

Comment (by ocean90):

 I just saw, that you use the basename as the value for `sitefavicon`,
 can't we just use the attachment ID or better, the complete URL?

 I asked because of this line

 {{{
 $posts = get_posts( array( 'name' => $favicon_fullname, 'post_type' =>
 'attachment' ) );
 }}}

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16434#comment:105>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list