if (get_theme_mod()) returns false if 0

[karlikdesign]

Seems like the number type isn't returning 0 in the customizer

Here's my control code in functions.php

$wp_customize->add_setting('apw_header_cta_padding_bottom', array( 'default' => '20' ) );
	$wp_customize->add_control(
		new Customize_Number_Control(
			$wp_customize,
			'header_cta_padding_bottom',
			array(
				'label'      => 'Bottom Padding',
				'section'    => 'apw_header_cta',
				'settings'   => 'apw_header_cta_padding_bottom',
				'type'		 => 'number',
				'priority'	 => 30
			)
		)
	);	

Then, in my header.php I have

<?php if(get_theme_mod( 'apw_header_cta_padding_bottom')) : ?>

Then, when I preview the site in the customizer and set that value to zero, this if statement returns false when the value should be zero.

[westonruter]

@karlikdesign: Thanks for the report. Actually this is not a bug because in PHP the value of 0 is “false-y”. You can see the values in PHP that are considered false in a boolean context via the ​PHP Manual.

So all you have to do is change your conditional to check for the false value explicitly with a not-identical comparison:

<?php if ( false !== get_theme_mod( 'apw_header_cta_padding_bottom' ) ) : ?>

You probably want to report such future issues to the ​WordPress Support forums first before opening a Core Trac ticket. Thanks!

[karlikdesign]

oh whoops! You're right. That did work :)

Really appreciate your help!!!

Replying to westonruter:

@karlikdesign: Thanks for the report. Actually this is not a bug because in PHP the value of 0 is “false-y”. You can see the values in PHP that are considered false in a boolean context via the ​PHP Manual.

So all you have to do is change your conditional to check for the false value explicitly with a not-identical comparison:

<?php if ( false !== get_theme_mod( 'apw_header_cta_padding_bottom' ) ) : ?>

You probably want to report such future issues to the ​WordPress Support forums first before opening a Core Trac ticket. Thanks!

[celloexpressions]

I would also suggest setting the default return value in the 2nd argument of the get_theme_mod() call, which would also result in more predictable behavior:

get_theme_mod( 'apw_header_cta_padding_bottom', 0 );

[karlikdesign]

Good call!