From 2cd6317a4e5596e899b58214221093f7e19edd9c Mon Sep 17 00:00:00 2001
From: Michael Fields <michael@mfields.org>
Date: Mon, 23 Apr 2012 21:35:30 +0000
Subject: [PATCH] _s: Only set theme options if necessary. fixes #1151 * Remove
 call to add_option() during admin_init. * Rework _s_get_theme_options() so
 that it merges default values into the stored settings. * Rework
 _s_theme_options_validate() to only return recognized values if they are
 present in the $_POST request - only store what is absolutely necessary. *
 Updated the docs in a few places. * Note: _s now requires PHP5. see L161
 array_intersect_key().

git-svn-id: https://wpcom-themes.svn.automattic.com/_s/@9483 d957f892-c61d-0410-b221-f235e6eecf30
---
 inc/theme-options/theme-options.php | 66 +++++++++++++----------------
 1 file changed, 29 insertions(+), 37 deletions(-)

diff --git a/inc/theme-options/theme-options.php b/inc/theme-options/theme-options.php
index 87b39c91..61b25f03 100644
--- a/inc/theme-options/theme-options.php
+++ b/inc/theme-options/theme-options.php
@@ -12,19 +12,12 @@
  * This function is attached to the admin_init action hook.
  *
  * This call to register_setting() registers a validation callback, _s_theme_options_validate(),
- * which is used when the option is saved, to ensure that our option values are complete, properly
+ * which is used when the option is saved, to ensure that our option values are properly
  * formatted, and safe.
  *
- * We also use this function to add our theme option if it doesn't already exist.
- *
  * @since _s 1.0
  */
 function _s_theme_options_init() {
-
-	// If we have no options in the database, let's add them now.
-	if ( false === _s_get_theme_options() )
-		add_option( '_s_theme_options', _s_get_default_theme_options() );
-
 	register_setting(
 		'_s_options',       // Options group, see settings_fields() call in _s_theme_options_render_page()
 		'_s_theme_options', // Database option, see _s_get_theme_options()
@@ -147,30 +140,27 @@ function _s_sample_radio_buttons() {
 	return apply_filters( '_s_sample_radio_buttons', $sample_radio_buttons );
 }
 
-/**
- * Returns the default options for _s.
- *
- * @since _s 1.0
- */
-function _s_get_default_theme_options() {
-	$default_theme_options = array(
-		'sample_checkbox' => 'off',
-		'sample_text_input' => '',
-		'sample_select_options' => '',
-		'sample_radio_buttons' => '',
-		'sample_textarea' => '',
-	);
-
-	return apply_filters( '_s_default_theme_options', $default_theme_options );
-}
-
 /**
  * Returns the options array for _s.
  *
  * @since _s 1.0
  */
 function _s_get_theme_options() {
-	return get_option( '_s_theme_options', _s_get_default_theme_options() );
+	$saved = (array) get_option( '_s_theme_options' );
+	$defaults = array(
+		'sample_checkbox'       => 'off',
+		'sample_text_input'     => '',
+		'sample_select_options' => '',
+		'sample_radio_buttons'  => '',
+		'sample_textarea'       => '',
+	);
+
+	$defaults = apply_filters( '_s_default_theme_options', $defaults );
+
+	$options = wp_parse_args( $saved, $defaults );
+	$options = array_intersect_key( $options, $defaults );
+
+	return $options;
 }
 
 /**
@@ -255,7 +245,7 @@ function _s_settings_field_sample_textarea() {
 }
 
 /**
- * Returns the options array for _s.
+ * Renders the Theme Options administration screen.
  *
  * @since _s 1.0
  */
@@ -283,22 +273,24 @@ function _s_theme_options_render_page() {
  * @see _s_theme_options_init()
  * @todo set up Reset Options action
  *
+ * @param array $input Unknown values.
+ * @return array Sanitized theme options ready to be stored in the database.
+ *
  * @since _s 1.0
  */
 function _s_theme_options_validate( $input ) {
-	$output = $defaults = _s_get_default_theme_options();
+	$output = array();
 
-	// The sample checkbox should either be on or off
-	if ( ! isset( $input['sample_checkbox'] ) )
-		$input['sample_checkbox'] = 'off';
-	$output['sample_checkbox'] = ( $input['sample_checkbox'] == 'on' ? 'on' : 'off' );
+	// Checkboxes will only be present if checked.
+	if ( isset( $input['sample_checkbox'] ) )
+		$output['sample_checkbox'] = 'on';
 
 	// The sample text input must be safe text with no HTML tags
-	if ( isset( $input['sample_text_input'] ) )
+	if ( isset( $input['sample_text_input'] ) && ! empty( $input['sample_text_input'] ) )
 		$output['sample_text_input'] = wp_filter_nohtml_kses( $input['sample_text_input'] );
 
 	// The sample select option must actually be in the array of select options
-	if ( array_key_exists( $input['sample_select_options'], _s_sample_select_options() ) )
+	if ( isset( $input['sample_select_options'] ) && array_key_exists( $input['sample_select_options'], _s_sample_select_options() ) )
 		$output['sample_select_options'] = $input['sample_select_options'];
 
 	// The sample radio button value must be in our array of radio button values
@@ -306,8 +298,8 @@ function _s_theme_options_validate( $input ) {
 		$output['sample_radio_buttons'] = $input['sample_radio_buttons'];
 
 	// The sample textarea must be safe text with the allowed tags for posts
-	if ( isset( $input['sample_textarea'] ) )
-		$output['sample_textarea'] = wp_filter_post_kses($input['sample_textarea'] );
+	if ( isset( $input['sample_textarea'] ) && ! empty( $input['sample_textarea'] ) )
+		$output['sample_textarea'] = wp_filter_post_kses( $input['sample_textarea'] );
 
-	return apply_filters( '_s_theme_options_validate', $output, $input, $defaults );
+	return apply_filters( '_s_theme_options_validate', $output, $input );
 }