Problemas de validación y/o inserción en la BD. Setting API

[rogertm]

Estoy haciendo un setting para un theme, algo parecido al que tiene Twentyeleven, pero una de las funciones no inserta los datos en la base de datos.

Lo que tengo es lo siguiente, una función que me permite escoger si en el header del theme aparecerá una imagen de cabecera o no y otra que habilitará unos links a redes sociales, twitter, facebook, etc...

La de la cabecera funciona bien, pero la otra no, aquí parte del código:

Código PHP:

<?php
/**
 * Register setting options
 */
add_action( 'admin_init', 'theme_register_setting_options_init' );
function theme_register_setting_options_init(){
    // Based on Twentyeleven WordPress Theme
    register_setting( 'theme_options', 'theme_theme_options', 'theme_theme_options_validate' );
 
    // Register our settings field group
    add_settings_section( 'general', '', '__return_false', 'theme-options' );
 
    // Register our individual settings fields
    add_settings_field( 'theme_header_stuff',    __( 'Header Options', 't_em' ),            'theme_settings_field_header_stuff',            'theme-options',    'general' );
    add_settings_field( 'theme_social_stuff',    __( 'Social Network Options', 't_em' ),    'theme_settings_field_socialnetwork_stuff',    'theme-options',    'general' );
}
 
/**
 * Return an array of header options for Twenty'em
 */
function theme_header_options(){
    $header_options = array (
        'no-header-image' => array (
            'value' => 'no-header-image',
            'label' => __( 'No header image', 't_em' ),
            'extend' => '',
        ),
        'header-image' => array (
            'value' => 'header-image',
            'label' => __( 'Header image', 't_em' ),
            'extend' => theme_header_image_extend(),
        ),
    );
 
    return apply_filters( 'theme_header_options', $header_options );
}
 
/**
 * Return an array of social network options for Twenty'em
 */
function theme_socialnetwork_options(){
    $socialnetwork_options = array (
        'twitter-stuff' => array (
            'value' => '',
            'name' => 'twitter-stuff',
            'label' => __( 'Twitter URL', 't_em' ),
        ),
        'facebook-stuff' => array (
            'value' => '',
            'name' => 'facebook-stuff',
            'label' => __( 'Facebook URL', 't_em' ),
        ),
    );
 
    return apply_filters( 'theme_socialnetwork_options', $socialnetwork_options );
}
?>
 

Además, tengo una función que me registra opciones por defecto, otra para obtener las opciones de la BD, otra por cada setting para mostrarlos en el administrador del theme que se las muestro. Esta me imprime en pantalla tres campos de texto donde poner las URLs de las redes sociales:

Código PHP:

<?php
/**
 * Render the Socialnetwork setting field
 */
function theme_settings_field_socialnetwork_stuff(){
    $options = theme_get_theme_options();
    foreach ( theme_socialnetwork_options() as $social ) :
?>
    <div class="layout text-option social">
        <label>
            <span><?php echo $social['label'];?></span>
            <input type="text" name="theme_theme_options['<?php echo $social['name']; ?>']" value="<?php echo esc_attr( $options[$social['name']] ) ?>" />
        </label>
    </div>
<?php
    endforeach;
}
?>
 

Y por último una función para validar los datos.

Código PHP:

<?php
/**
 * Sanitize and validate input. Accepts an array, return a sanitized array.
 */
function theme_theme_options_validate( $input ){
    $output = $defaults = theme_get_default_theme_options();
 
    // Header stuff must be in our array of header stuff options
    if ( isset( $input['header-stuff'] ) && array_key_exists( $input['header-stuff'], theme_header_options() ) ) :
        $output['header-stuff'] = $input['header-stuff'];
    endif;
 
    // Archive stuff must be in our array of archive stuff options
    if ( isset( $input['archive-stuff'] ) && array_key_exists( $input['archive-stuff'], theme_archive_options() ) ) :
        $output['archive-stuff'] = $input['archive-stuff'];
    endif;
 
    return apply_filters( 'theme_theme_options_validate', $output, $input, $defaults );
}
?>
 

El código completo del archivo es mucho más largo, pero me he centrado en solo lo más importante, además tengo un par de cosas más que no me insertan los datos en la BD, pero si resuelvo uno, ya seguro resuelvo lo demás.

Algo más, si elimino el tercer parámetro de la función register_setting(), que es un callback a la función de validación, entonces me guarda los datos, pero por ejemplo, en los campos donde se insertan las URLs de las redes sociales, me manda el siguiente error:

Notice: Undefined index: twitter-stuff in /path/to/file/theme-options.php on line 398

que es

Código PHP:

<input type="text" name="theme_theme_options['<?php echo $social['name']; ?>']" value="<?php echo esc_attr( $options[$social['name']] ) ?>" />
 

Para hacer todo esto me estoy guiando por la Settings API y el archivo theme-options.php que viene en Twentyeleven

Gracias mil de antemano a todos los que me puedan dar una ayuda...

[masterpuppet]

Que tal rogertm,

Yo de WP ni idea pero es correcto utilizar apply_filters de esa forma ?, tal cual lo tienes no estas generando recursividad ?

[rogertm]

En algún momento me hice esa misma pregunta, si dejo return $socialnetwork_options; entonces hace lo mismo, osea, no hay cambios. Pero como dije antes, me estoy fijando por el código que hay en Twentyeleven, y si los que saben lo tienen así, pues por algo será

Y acabo de hacer lo siguiente:

Código PHP:

/**
 * Return an array of header options for Twenty'em
 */
$header_options = array (
    'no-header-image' => array (
        'value' => 'no-header-image',
        'label' => __( 'No header image', 't_em' ),
        'extend' => '',
    ),
    'header-image' => array (
        'value' => 'header-image',
        'label' => __( 'Header image', 't_em' ),
        'extend' => theme_header_image_extend(),
    ),
    'slideshow' => array (
        'value' => 'slideshow',
        'label' => __( 'Slideshow', 't_em' ),
        'extend' => theme_slider_extend(),
    ),
);
function theme_header_options(){
    theme_ $header_options;
    return apply_filters( 't_em_header_options', $header_options );
}
 

Y funciona exactamente igual

[masterpuppet]

Buenas rogertm, lo pudiste resolver ?, en cualquier caso te hago una pregunta y comentario, la pregunta, en theme_get_default_theme_options tienes definidas las key twitter-stuff - facebook-stuff ? y el comentario, ten en cuenta que tienes '' de mas en el html del input

mal

Código HTML:

<input name="theme_theme_options['<?php echo $social['name']; ?>']" />
 

bien

Código HTML:

<input name="theme_theme_options[<?php echo $social['name']; ?>]" />
 

Saludos.

[rogertm]

Si si... las comillas de la MUERTE ... Me pasé tres madrugadas en eso

Ahora funciona bien, pero sin la validación, si activo la función de validar el input, algunas cosas se guardan y otras no, tengo que dedicarle un rato a eso a ver si lo termino de resolver, porque aunque no es obligatorio hacer la validación, no está de más...

Gracias...

[masterpuppet]

Por eso la pregunta

theme_get_default_theme_options tienes definidas las key twitter-stuff - facebook-stuff ?

si no están definidas las claves en las opciones por defecto tienes que hacer algo similar al if de header-stuff y archive-stuff para que quede "registrado" en $output

[rogertm]

Bueno, desde que publiqué este post hasta hoy han habido algunos cambios, ya me valida bien todo excepto un select, pero creo tener ubicado el problema así que será cuestión de dedicarle un rato...

Finalmente decidí validar según el tipo de input que sea, para así pasarlos todos por un foreach y esta es mi función de validación:

Código PHP:

function t_em_theme_options_validate( $input ){
    global $excerpt_options, $slider_layout;
 
    // All the checkbox are either 0 or 1
    foreach ( array(
        'header-featured-image',
    ) as $checkbox ) :
        if ( !isset( $input[$checkbox] ) )
            $input[$checkbox] = null;
        $input[$checkbox] = ( $input[$checkbox] == 1 ? 1 : 0 );
    endforeach;
 
    // Validate all radio options
    $radio_options = array(
        'header-options'    => array (
            'set'        => 'header-set',
            'callback'    => t_em_header_options(),
        ),
        'slider-options'    => array (
            'set'        => 'slider-thumbnail',
            'callback'    => $slider_layout,
        ),
        'archive-options'    => array (
            'set'        => 'archive-set',
            'callback'    => t_em_archive_options(),
        ),
        'excerpt-options'    => array (
            'set'        => 'excerpt-set',
            'callback'    => $excerpt_options,
        ),
        'layout-options'    => array (
            'set'        => 'layout-set',
            'callback'    => t_em_layout_options(),
        ),
    );
    foreach ( $radio_options as $radio ) :
        if ( ! isset( $input[$radio['set']] ) )
            $input[$radio['set']] = null;
        if ( ! array_key_exists( $input[$radio['set']], $radio['callback'] ) )
            $input[$radio['set']] = null;
    endforeach;
 
    // Validate all int (input[type="number"]) options
    foreach( array (
        'slider-thumbnail-width',
        'slider-thumbnail-height',
        'slider-number',
        'excerpt-thumbnail-width',
        'excerpt-thumbnail-height',
        'layout-width',
    ) as $int ) :
        $input[$int] = wp_filter_nohtml_kses( $input[$int] );
    endforeach;
 
    // Validate all url (input[type="url"]) options
    foreach ( array (
        'twitter-set',
        'facebook-set',
        'googlepluss-set',
        'rss-set',
    ) as $url ) :
        $input[$url] = esc_url_raw( $input[$url] );
    endforeach;
 
    // Validate all select list options
    $select_options = array ( // No pincha
        'slider-cat'        => array (
            'set'        => 'slider-category',
            'callback'    => t_em_slider_callback(),
        ),
    );
    foreach ( $select_options as $select ) :
        if ( ! array_key_exists( $input[$select['set']], $select['callback'] ) )
            $input[$select['set']] = null;
    endforeach;
 
    return $input;
}
?>
 

Ahora, $excerpt_options y $slider_layout son un par de variables que me retornan el array del slider-set y el excerpt-set, que a su vez pertenecen a las funciones que llamo como callback en t_em_archive_options() y t_em_header_options(), por ejemplo:

Código PHP:

function t_em_header_options(){
    $header_options = array (
        'no-header-image' => array (
            'value' => 'no-header-image',
            'label' => __( 'No header image', 't_em' ),
            'extend' => '',
        ),
        'header-image' => array (
            'value' => 'header-image',
            'label' => __( 'Header image', 't_em' ),
            'extend' => t_em_header_image_callback(),
        ),
        'slider' => array (
            'value' => 'slider',
            'label' => __( 'Slider', 't_em' ),
            'extend' => t_em_slider_callback(),
        ),
    );
 
    return apply_filters( 't_em_header_options', $header_options );
}
 

Y aquí la función callback:

Código PHP:

function t_em_slider_callback(){
    global $slider_layout;
    $slider_layout = array (
        'slider-thumbnail-left' => array (
            'value' => 'slider-thumbnail-left',
            'label' => __( 'Slider thumbnail on left', 't_em' ),
            'thumbnail' => T_EM_FUNCTIONS_DIR_IMG . '/slider-thumbnail-left.png',
        ),
        'slider-thumbnail-right' => array (
            'value' => 'slider-thumbnail-right',
            'label' => __( 'Slider thumbnail on right', 't_em' ),
            'thumbnail' => T_EM_FUNCTIONS_DIR_IMG . '/slider-thumbnail-right.png',
        ),
        'slider-thumbnail-full' => array (
            'value' => 'slider-thumbnail-full',
            'label' => __( 'Slider thumbnail on full', 't_em' ),
            'thumbnail' => T_EM_FUNCTIONS_DIR_IMG . '/slider-thumbnail-full.png',
        ),
    );
// aquí más código
}
 

Entonces, lo que me falta, como dije antes, es validar un select, de donde escojo una categoría determinada para mostrar en el slider del theme, pero creo tener claro qué debo hacer, es que me devuelva el valor mediante un array (como los anteriores), esto es lo que tengo:

Código PHP:

function t_em_slider_callback(){
// Aquí hay más código
    // Display a select list of categories
    $list_categories = get_categories();
    $extend_slider .= '<div class="sub-extend">';
    $extend_slider .=    '<label class="description">';
    $extend_slider .=         '<p>'. __( 'Select the category you want to be displayed in the slider section', 't_em' ) .'</p>';
    $extend_slider .=         '<select name="t_em_theme_options[slider-category]">';
    foreach ( $list_categories as $slider_category ) :
        $selected_option = ( $options['slider-category'] == $slider_category->cat_ID ) ? 'selected="selected"' : '';
        $extend_slider .=     '<option value="'.$slider_category->cat_ID.'" '.$selected_option.'>'.$slider_category->name.'</option>';
    endforeach;
    $extend_slider .=         '</select>';
    $extend_slider .=    '</label>';
    $extend_slider .= '</div>';
}
 

En fin, va funcionando, en cuanto lo termine publico la solución completa...

Gracias.