forked from mirror/_s
_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
This commit is contained in:
Michael Fields
Ian Stewart
parent
2a0b4e7bf8
commit
2cd6317a4e
|
|
@ -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 );
|
||||
}
|
||||
|
|
|
|||
Reference in New Issue