[wp-trac] [WordPress Trac] #36332: Custom Logo: PHP warnings in add_theme_support

WordPress Trac noreply at wordpress.org
Fri Mar 25 17:56:42 UTC 2016


#36332: Custom Logo: PHP warnings in add_theme_support
--------------------------+-------------------
 Reporter:  obenland      |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  4.5
Component:  Themes        |    Version:  trunk
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-------------------
 From the support forums: https://wordpress.org/support/topic/custom-logo-
 with-no-args-causing-php-warnings

 Hi,

 I'm testing the new WP4.5 development releases. Since current
 4.5-RC1-37079 update I get these PHP warnings:

 {{{ #!php
 Warning: array_intersect_key(): Argument #1 is not an array in /srv/www
 /wordpress-default/wp-includes/theme.php on line 1570`

 Warning: Cannot use a scalar value as an array in /srv/www/wordpress-
 default/wp-includes/theme.php on line 1570

 Warning: Cannot use a scalar value as an array in /srv/www/wordpress-
 default/wp-includes/theme.php on line 1574

 Warning: Cannot use a scalar value as an array in /srv/www/wordpress-
 default/wp-includes/theme.php on line 1575
 }}}
 I was able to trace these down to `add_theme_support( 'custom-logo' );`.
 When I use it in plain with no additional args, the PHP warnings above are
 thrown.

 If I set custom logo with args, it works fine:
 {{{ #!php
 add_theme_support( 'custom-logo', array(
         'height'      => 160,
         'width'       => 160,
         'flex-height' => true,
         'flex-width'  => true,
 ) );
 }}}
 Is it required to set the args? Shouldn't it work without the args? What
 happens if no args are passed, will it take the full size of the uploaded
 image?

 Thanks for addressing this issue!

--
Ticket URL: <https://core.trac.wordpress.org/ticket/36332>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list