with empty 'show_on_front' option is_front_page always show false

[kookoooo]

maybe just me, I don't have a 'show_on_front' value in db, by default the latest posts shows, but function is_front_page() won't work.

orig

function is_front_page () {
// most likely case
if ( 'posts' == get_option('show_on_front') && is_home() )
return true;

should be

function is_front_page () {
// most likely case
if ( (!get_option('show_on_front') || 'posts' == get_option('show_on_front')) && is_home() )
return true;

[filosofo]

You should have a "show_on_front" option, as it's part of the default database schema, so it should be part of either a new install or an upgrade.

My attached patch is a more succinct way of expressing the idea.

[Denis-de-Bernardy]

closing this as invalid. it gets added in the installer/upgrader, and never gets deleted. if it contains an invalid value, it's a plugin broke it.

[danaronson]

turns out this bug DOES exist (fix included). There is some conditional code that sets option values in options-reading.php. In a VERY standard case there is no else clause which sets them to the default values so later on options.php sets them to null.

I fixed it one way, but perhaps the better and more global fix is options.php to ONLY update the option is the value was set in the form (I didn't want to do that fix, because I don't know what it would break).

The way to duplicate this bug is to go to the options reading panel if there are no pages, update the option and voila, show_on_front gets cleared...

Here's the diff of the fix:

*** options-reading.php	Fri Jan 15 12:02:03 2010
--- options-reading.php.save	Fri Jan 15 11:44:00 2010
***************
*** 53,63 ****
  <?php endif; ?>
  </fieldset></td>
  </tr>
- <?php else: ?>
- 
- <input type="hidden" name="show_on_front" value="<?php echo get_option("show_on_front") ?>">
- <input type="hidden" name="page_on_front" value="<?php echo get_option("page_on_front") ?>">
- <input type="hidden" name="page_for_posts" value="<?php echo get_option("page_for_posts") ?>">
  <?php endif; ?>
  <tr valign="top">
  <th scope="row"><label for="posts_per_page"><?php _e('Blog pages show at most') ?></label></th>
--- 53,58 ----

[sivel]

Dup of #12737

[nacin]

(In [14362]) When no pages are present add a hidden input field so that the show_on_front option is not blanked. Fixes is_front_page return value in an edge case. props sivel, fixes #12737, fixes #9105.

[nacin]

Shouldn't have referenced this ticket in the commit, but I wanted to cross reference. Forgot it would move the resolution to fixed after it was already closed.