Сохранять мета настраиваемого сообщения, не сохраняя данные

Я создал собственный тип сообщения с датой метабокса от и датой до.

Создание пользовательского типа сообщения с функцией обратного вызова add_events_metaboxes

function event_list_init(){

    $labels = array(
        'name'                  => _x( 'Events', 'post type general name' ),
        'singular_name'         => _x( 'Event', 'post type singular name' ),
        'menu_name'             => _x( 'Events List', 'admin menu' ),
        'name_admin_bar'        => _x( 'Events List', 'add new on admin bar' ),
        'add_new_item'          => __( 'Add New Event' ),
        'new_item'              => __( 'New Event' ),
        'edit_item'             => __( 'Edit Event' ),
        'view_item'             => __( 'View Event' ),
        'all_items'             => __( 'All Events' ),
        'search_items'          => __( 'Search Events' ),
        'not_found'             => __( 'No Events found.' ),
        'not_found_in_trash'    => __( 'No Events found in Trash.' )
    );

    $args   = array(
        'labels'                => $labels,
        'description'           => __( 'Create Events' ),
        'public'                => true,
        'publicly_queryable'    => true,
        'show_ui'               => true,
        'show_in_menu'          => true,
        'query_var'             => true,
        'rewrite'               => array( 'slug' => 'event' ),
        'capability_type'       => 'post',
        'has_archive'           => true,
        'hierarchical'          => true,
        'menu_position'         => 6,
        'register_meta_box_cb'  => 'add_events_metaboxes',
        'menu_icon'             => 'dashicons-calendar-alt',
        'supports'              => array( 'title', 'editor', 'author', 'thumbnail', 'excerpt', 'comments' )
    );

    register_post_type('events',$args);

}

add_action('init','event_list_init');

вот функция обратного вызова, которая создает экземпляр класса для создания метабокса и сохраняет данные публикации с помощью хука действия save_post

function add_events_metaboxes(){
   new eventsListMetaBox();
}

class eventsListMetaBox{
    /*
     * Constructor that creates the meta box
     */
    public  function  __construct(){
        /**
         * Render and Add form meta box
         */
        add_meta_box('wpt_events_date', 'Events Date', array($this, 'fisa_events_date'), 'events', 'side', 'high');

        /**
         * Save Date from and to as meta key
         */
        add_action('save_post',array($this, 'fisa_events_date_save'),1,2);
    }

    /**
     * Render Form for Events date
     */
    function fisa_events_date() {

        global $post;

        // Add an nonce field so we can check for it later.
        wp_nonce_field( 'events_date_fromto', 'events_datefromto_nonce' );

        // Echo out the field
        echo '<label for="_fisa_date_from">Date From</label>';
        echo '<input id="fisa-event-datefrom" type="text" name="_fisa_date_from"  class="widefat" />';
        echo '<br/><br/>';
        echo '<label for="_fisa_date_to">Date To</label>';
        echo '<input id="fisa-event-dateto" type="text" name="_fisa_date_to" class="widefat" />';

    }

    /**
     * Meta key actual database insertion
     */
    function fisa_events_date_save($post_id){

        /**
         * Check if nonce is not set
         */
//        if (!isset($_POST['events_datefromto_nonce']))
//            return $post_id;
//
//        $nonce = $_POST['events_datefromto_nonce'];
//        /**
//         * Verify that the request came from our screen with the proper authorization
//         */
//        if(!wp_verify_nonce($nonce,'events_date_fromto'))
//            return $post_id;
//
//        //Check the user's permission
//
//        if(!current_user_can('edit_post',$post_id) )
//            return $post_id;

        //Prepare and sanitize the data before saving it
        $events_date =  array(
                            sanitize_text_field( $_POST['_fisa_date_from']),
                            sanitize_text_field($_POST['_fisa_date_to'])
                        );

        update_post_meta($post_id, '_fisa_events_date', $events_date);
    }
}

Моя проблема в том, что я не вижу мета-ключ _fisa_events_date в postmeta таблице wordpress. Может ли кто-нибудь указать, что я пропустил, или что мне делать, чтобы сохранить его?


person jameshwart lopez    schedule 12.02.2016    source источник
comment
Удалось ли вам увидеть мета-поле, отображаемое при редактировании сообщения этого типа?   -  person Stiliyan    schedule 18.02.2016
comment
@Stiliyan Да. Я вижу это. Единственная проблема в том, что данные не сохраняются .. Вот почему в моем посте я сделал проверку nonce в качестве комментария, просто чтобы попытаться сохранить данные напрямую.   -  person jameshwart lopez    schedule 18.02.2016
comment
@jameswartlopez Возможно, вы не зарегистрировали свои метабоксы в add_action('admin_init', 'add_events_metaboxes');? Я успешно могу сохранить _fisa_events_date вместе с остальной частью вашего кода после добавления этого оператора.   -  person Stiliyan    schedule 18.02.2016
comment
@Stiliyan где вы добавили admin_init на основе моего кода?   -  person jameshwart lopez    schedule 18.02.2016
comment
Вы можете добавить его прямо под свой add_action('init', 'event_list_init') хук. Но я рекомендую помещать все настраиваемые почтовые индексы в один файл, например events.php и add_action крючки в самом верху файла. Затем в своем functions.php вы можете require_once('events.php'); См. Мой ответ ниже для более подробной информации.   -  person Stiliyan    schedule 18.02.2016
comment
Удалось ли вам решить проблему?   -  person Stiliyan    schedule 18.02.2016
comment
@Stiliyan Большое спасибо, это сохраняет данные, но я не решаюсь принять ваш ответ, потому что мне кажется неправильным подключаться к admin init, пока есть save_post и register_meta_box_cb кажется бесполезным в вашем ответе. В любом случае плюс один для вас: )   -  person jameshwart lopez    schedule 18.02.2016
comment
@Stiliyan прав! вы не должны использовать register_meta_box_cs для сохранения сообщения. admin init был бы гораздо лучшим вариантом и часто используется для инициализации классов!   -  person David    schedule 22.02.2016


Ответы (4)


Вы полагаетесь на register_meta_box_cb, чтобы отобразить ваше мета-поле и подключить логику сохранения к save_post. Проблема в том, что Wordpress запускает register_meta_box_cb (который подключается к add_meta_boxes) только тогда, когда нужно отобразить метабоксы, то есть когда посещаются страницы редактирования или добавления сообщений. Но когда Wordpress сохраняет сообщения, ему не нужно отображать метабоксы, поэтому register_meta_box_cb, add_events_metaboxes и ваш save_post хук никогда не вызываются.

Самое простое решение - удалить аргумент register_meta_box_cb и подключить функцию add_events_metaboxes к admin_init.

add_action('admin_init', 'add_events_metaboxes');

Вы можете сделать это прямо под вашим add_action('init', 'event_list_init') крючком.

Найдите суть здесь.

person Stiliyan    schedule 18.02.2016

Согласно документации Wordpress, register_meta_box_cb:

будет вызываться при настройке мета-боксов для формы редактирования.

Что слишком поздно для подключения save_post.

Поэтому я предлагаю вам отдельно подключиться к save_post где-нибудь в вашем файле functions.php:

Регистрация типа сообщения

$args   = array(
    ....
    'register_meta_box_cb'  => 'add_events_metaboxes',
    ....
);
register_post_type( 'events', $args );

Визуализируйте метабокс

function add_events_metaboxes(){
   new eventsListMetaBox();
}

class eventsListMetaBox{
    public  function  __construct(){
        add_meta_box('wpt_events_date', 'Events Date', array($this, 'fisa_events_date'), 'events', 'side', 'high');
    }

    function fisa_events_date() {
        ...
    }
}

Сохранение метабокса

add_action( 'save_post', 'fisa_events_date_save' );

function fisa_events_date_save(){
    global $post;
    $post_id = $post->ID;

    // Use wp_verify_nonce and other checks to verify everything is right

    $events_date =  array(
                        sanitize_text_field( $_POST['_fisa_date_from']),
                        sanitize_text_field($_POST['_fisa_date_to'])
                    );
    update_post_meta($post_id, '_fisa_events_date', $events_date);
}

Имейте в виду, что Wordpress не предназначен для полностью объектно-ориентированной работы, поэтому могут возникнуть проблемы с использованием его системы подключения. с концепциями ООП.

person Pmpr    schedule 19.02.2016

Как указано в других ответах, проблема в том, что обратный вызов register_meta_box_cb будет иметь дело только со следующим:

Сделайте remove_meta_box() и add_meta_box() вызывает обратный вызов.

save_post не покрывается обратным вызовом и должен быть независимым . Этот хук действия происходит:

всякий раз, когда сообщение или страница создается или обновляется, что может быть из импорта, формы редактирования публикации / страницы, xmlrpc или публикации по электронной почте

Поскольку вы используете класс для обертывания создания метабокса, я бы посоветовал обернуть все внутри него.
PS: проверьте комментарии и добавьте необходимые проверки внутри save_post обратного вызова.

<?php
class MyEvents {
    public  function  __construct(){
        add_action( 'init', array( $this, 'init' ) );    
        add_action( 'save_post', array( $this, 'save_post' ), 10, 2 ); // no need to change priority to 1
    }

    public function init(){
        $labels = array(
            'name'                  => _x( 'Events', 'post type general name' ),
            'singular_name'         => _x( 'Event', 'post type singular name' ),
            'menu_name'             => _x( 'Events List', 'admin menu' ),
            'name_admin_bar'        => _x( 'Events List', 'add new on admin bar' ),
            'add_new_item'          => __( 'Add New Event' ),
            'new_item'              => __( 'New Event' ),
            'edit_item'             => __( 'Edit Event' ),
            'view_item'             => __( 'View Event' ),
            'all_items'             => __( 'All Events' ),
            'search_items'          => __( 'Search Events' ),
            'not_found'             => __( 'No Events found.' ),
            'not_found_in_trash'    => __( 'No Events found in Trash.' )
        );
        $args   = array(
            'labels'                => $labels,
            'description'           => __( 'Create Events' ),
            'public'                => true,
            'publicly_queryable'    => true,
            'show_ui'               => true,
            'show_in_menu'          => true,
            'query_var'             => true,
            'rewrite'               => array( 'slug' => 'event' ),
            'capability_type'       => 'post',
            'has_archive'           => true,
            'hierarchical'          => true,
            'menu_position'         => 6,
            'register_meta_box_cb'  => array( $this, 'add_metaboxes' ),
            'menu_icon'             => 'dashicons-calendar-alt',
            'supports'              => array( 'title', 'editor', 'author', 'thumbnail', 'excerpt', 'comments' )
        );
        register_post_type('events',$args);
    }

    public function add_metaboxes() {
        add_meta_box( 'wpt_events_date', 'Events Date', array( $this, 'the_metabox' ), 'events', 'side', 'high' );
    }

    public function the_metabox( $post ) { // No need for "global $post", it's passed as parameter
        wp_nonce_field( 'events_date_fromto', 'events_datefromto_nonce' );
        $dates = get_post_meta( $post->ID, '_fisa_events_date', true);
        $from = $dates ? $dates[0] : false;
        $to = $dates ? $dates[1] : false;
        echo '<label for="_fisa_date_from">Date From</label>';
        printf(
            '<input id="fisa-event-datefrom" type="text" name="_fisa_date_from"  class="widefat" value="%s" />',
            $from ? $from : ''
        );
        echo '';
        echo '<br/><br/>';
        echo '<label for="_fisa_date_to">Date To</label>';
        printf(
            '<input id="fisa-event-dateto" type="text" name="_fisa_date_to"  class="widefat" value="%s" />',
            $to ? $to : ''
        );
    }

    public function save_post( $post_id, $post_object ) { // second parameter has useful info about current post
        /* BRUTE FORCE debug */
        // wp_die( sprintf( '<pre>%s</pre>', print_r( $_POST, true ) ) );

        /**
         * ADD SECURITY AND CONTENT CHECKS, omitted for brevity
         */
        if( !empty( $_POST['_fisa_date_from'] ) ) {
            $events_date =  array(
                                sanitize_text_field( $_POST['_fisa_date_from']),
                                sanitize_text_field($_POST['_fisa_date_to'])
                            );
            update_post_meta($post_id, '_fisa_events_date', $events_date);
        }
    }
}
new MyEvents();
person brasofilo    schedule 20.02.2016

Самое простое и гибкое решение - использовать расширенные настраиваемые поля.

person TripsLeft    schedule 19.02.2016
comment
Привет, Trips, прошу прощения за неудовлетворительную оценку, но в вашем ответе есть пара проблем, и почти заманчиво пометить его для преобразования в комментарий ... Быть похожим на комментарий - это первая проблема; тогда OP запрашивает кодовое решение, а не подсказку простого плагина (да, ACF - это круто); за советы, пожалуйста, оставьте комментарий. Личное мнение: если бы я отвечал так, как вы, я бы добавил ссылку на плагин, добавил краткое описание (объективный материал, а не маркетинг) и показал пример того, как это сделать программно, как это позволяет ACF. Всего наилучшего! - person brasofilo; 20.02.2016