[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:25:20 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 tomAuger):

 Replying to [comment:103 ocean90]:
 > Replying to [comment:102 tomAuger]:
 >
 > Ah, sorry, I mean `$title` for `<title></title>`:
 >
 > {{{
 > if (! current_user_can( 'manage_options' ) )
 >     wp_die( __( 'Cheatin&#8217; uh?' ) );
 >
 > check_admin_referer( 'favicon_upload-options' );
 >
 > $title = __( 'Crop Favicon' );
 > $parent_file = 'options-general.php';
 >
 > if ( isset( $_POST['CROP_AND_SAVE'] ) ) {
 > }}}

 Right on. Done.

 >
 > > What's wrong with the heredoc approach?
 > What Nacin said.
 Hmmm. I find popping in and out of `<?php ?>` way more distracting than
 <<< but that's my Perl backgrouond speaking no doubt. Cool, we'll do it
 that way.

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


More information about the wp-trac mailing list