如果未填写自定义字段,则阻止发布帖子


17

我有一个自定义帖子类型Event,其中包含开始和结束日期/时间自定义字段(作为帖子编辑屏幕中的元框)。

我想确保没有填写日期就无法发布(或安排)活动,因为这将导致显示事件数据的模板出现问题(除了这是必不可少的事实!)。但是,我希望能够在准备中的“草稿”事件中包含无效的日期。

我本来想通过挂钩save_post进行检查,但是如何防止状态更改发生呢?

EDIT1:这是我现在用来保存post_meta的钩子。

// Save the Metabox Data
function ep_eventposts_save_meta( $post_id, $post ) {

if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
    return;

if ( !isset( $_POST['ep_eventposts_nonce'] ) )
    return;

if ( !wp_verify_nonce( $_POST['ep_eventposts_nonce'], plugin_basename( __FILE__ ) ) )
    return;

// Is the user allowed to edit the post or page?
if ( !current_user_can( 'edit_post', $post->ID ) )
    return;

// OK, we're authenticated: we need to find and save the data
// We'll put it into an array to make it easier to loop though

//debug
//print_r($_POST);

$metabox_ids = array( '_start', '_end' );

foreach ($metabox_ids as $key ) {
    $events_meta[$key . '_date'] = $_POST[$key . '_date'];
    $events_meta[$key . '_time'] = $_POST[$key . '_time'];
    $events_meta[$key . '_timestamp'] = $events_meta[$key . '_date'] . ' ' . $events_meta[$key . '_time'];
}

$events_meta['_location'] = $_POST['_location'];

if (array_key_exists('_end_timestamp', $_POST))
    $events_meta['_all_day'] = $_POST['_all_day'];

// Add values of $events_meta as custom fields

foreach ( $events_meta as $key => $value ) { // Cycle through the $events_meta array!
    if ( $post->post_type == 'revision' ) return; // Don't store custom data twice
    $value = implode( ',', (array)$value ); // If $value is an array, make it a CSV (unlikely)
    if ( get_post_meta( $post->ID, $key, FALSE ) ) { // If the custom field already has a value
        update_post_meta( $post->ID, $key, $value );
    } else { // If the custom field doesn't have a value
        add_post_meta( $post->ID, $key, $value );
    }
    if ( !$value ) 
                delete_post_meta( $post->ID, $key ); // Delete if blank
}

}

add_action( 'save_post', 'ep_eventposts_save_meta', 1, 2 );

EDIT2:这是保存到数据库后我要用来检查发布数据的内容。

add_action( 'save_post', 'ep_eventposts_check_meta', 99, 2 );
function ep_eventposts_check_meta( $post_id, $post ) {
//check that metadata is complete when a post is published
//print_r($_POST);

if ( $_POST['post_status'] == 'publish' ) {

    $custom = get_post_custom($post_id);

    //make sure both dates are filled
    if ( !array_key_exists('_start_timestamp', $custom ) || !array_key_exists('_end_timestamp', $custom )) {
        $post->post_status = 'draft';
        wp_update_post($post);

    }
    //make sure start < end
    elseif ( $custom['_start_timestamp'] > $custom['_end_timestamp'] ) {
        $post->post_status = 'draft';
        wp_update_post($post);
    }
    else {
        return;
    }
}
}

与此相关的主要问题是另一个问题中实际描述的一个问题wp_update_post()save_post挂钩中使用会触发无限循环。

EDIT3:我想出了一种方法,通过钩子wp_insert_post_data代替save_post。唯一的问题是,现在post_status还原了,但是现在出现了一条误导性消息,指出“发布后”(通过添加&message=6到重定向的URL),但是状态设置为草稿。

add_filter( 'wp_insert_post_data', 'ep_eventposts_check_meta', 99, 2 );
function ep_eventposts_check_meta( $data, $postarr ) {
//check that metadata is complete when a post is published, otherwise revert to draft
if ( $data['post_type'] != 'event' ) {
    return $data;
}
if ( $postarr['post_status'] == 'publish' ) {
    $custom = get_post_custom($postarr['ID']);

    //make sure both dates are filled
    if ( !array_key_exists('_start_timestamp', $custom ) || !array_key_exists('_end_timestamp', $custom )) {
        $data['post_status'] = 'draft';
    }
    //make sure start < end
    elseif ( $custom['_start_timestamp'] > $custom['_end_timestamp'] ) {
        $data['post_status'] = 'draft';
    }
    //everything fine!
    else {
        return $data;
    }
}

return $data;
}

Answers:


16

正如m0r7if3r所指出的那样,无法阻止使用save_post挂钩发布帖子,因为在触发挂钩时,帖子已经保存。但是,以下内容将允许您在不使用wp_insert_post_data且不会引起无限循环的情况下还原状态。

以下内容未经测试,但可以正常工作。

<?php
add_action('save_post', 'my_save_post');
function my_save_post($post_id) {
    if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE )
         return;

    if ( !isset( $_POST['ep_eventposts_nonce'] ) )
         return;

    if ( !wp_verify_nonce( $_POST['ep_eventposts_nonce'], plugin_basename( __FILE__ ) ) )
         return;

    // Is the user allowed to edit the post or page?
     if ( !current_user_can( 'edit_post', $post->ID ) )
         return;

   // Now perform checks to validate your data. 
   // Note custom fields (different from data in custom metaboxes!) 
   // will already have been saved.
    $prevent_publish= false;//Set to true if data was invalid.
    if ($prevent_publish) {
        // unhook this function to prevent indefinite loop
        remove_action('save_post', 'my_save_post');

        // update the post to change post status
        wp_update_post(array('ID' => $post_id, 'post_status' => 'draft'));

        // re-hook this function again
        add_action('save_post', 'my_save_post');
    }
}
?>

我没有检查,但是查看代码,反馈消息将显示帖子已发布的错误消息。这是因为WordPress将我们重定向到该message变量现在不正确的网址。

要更改它,我们可以使用redirect_post_location过滤器:

add_filter('redirect_post_location','my_redirect_location',10,2);
function my_redirect_location($location,$post_id){
    //If post was published...
    if (isset($_POST['publish'])){
        //obtain current post status
        $status = get_post_status( $post_id );

        //The post was 'published', but if it is still a draft, display draft message (10).
        if($status=='draft')
            $location = add_query_arg('message', 10, $location);
    }

    return $location;
}

总结一下上面的重定向过滤器:如果某个帖子设置为要发布,但仍然是草稿,那么我们将相应地更改消息(即message=10)。再次,这未经测试,但应该可以。add_query_arg建议书的法典建议,当已经设置了变量时,该函数将其替换(但正如我所说,我尚未对此进行测试)。


除了失踪; 在您的add_query_arg行上,这个redirect_post_location过滤器技巧正是我所需要的。谢谢!
MadtownLems 2014年

@MadtownLems已修复:)
Stephen Harris

9

好的,这就是我最终要完成的方法:对PHP函数进行检查的Ajax调用,某种程度上受此答案的启发,并使用了我在StackOverflow上提出问题中的巧妙技巧。重要的是,我确保仅当我们要发布检查时,这样才能始终保存草稿而不检查。最终这是实际上阻止发布该帖子的更简单的解决方案。这可能会帮助别人,所以我在这里写下了。

首先,添加必要的Javascript:

//AJAX to validate event before publishing
//adapted from /wordpress/15546/dont-publish-custom-post-type-post-if-a-meta-data-field-isnt-valid
add_action('admin_enqueue_scripts-post.php', 'ep_load_jquery_js');   
add_action('admin_enqueue_scripts-post-new.php', 'ep_load_jquery_js');   
function ep_load_jquery_js(){
global $post;
if ( $post->post_type == 'event' ) {
    wp_enqueue_script('jquery');
}
}

add_action('admin_head-post.php','ep_publish_admin_hook');
add_action('admin_head-post-new.php','ep_publish_admin_hook');
function ep_publish_admin_hook(){
global $post;
if ( is_admin() && $post->post_type == 'event' ){
    ?>
    <script language="javascript" type="text/javascript">
        jQuery(document).ready(function() {
            jQuery('#publish').click(function() {
                if(jQuery(this).data("valid")) {
                    return true;
                }
                var form_data = jQuery('#post').serializeArray();
                var data = {
                    action: 'ep_pre_submit_validation',
                    security: '<?php echo wp_create_nonce( 'pre_publish_validation' ); ?>',
                    form_data: jQuery.param(form_data),
                };
                jQuery.post(ajaxurl, data, function(response) {
                    if (response.indexOf('true') > -1 || response == true) {
                        jQuery("#post").data("valid", true).submit();
                    } else {
                        alert("Error: " + response);
                        jQuery("#post").data("valid", false);

                    }
                    //hide loading icon, return Publish button to normal
                    jQuery('#ajax-loading').hide();
                    jQuery('#publish').removeClass('button-primary-disabled');
                    jQuery('#save-post').removeClass('button-disabled');
                });
                return false;
            });
        });
    </script>
    <?php
}
}

然后,处理检查的函数:

add_action('wp_ajax_ep_pre_submit_validation', 'ep_pre_submit_validation');
function ep_pre_submit_validation() {
//simple Security check
check_ajax_referer( 'pre_publish_validation', 'security' );

//convert the string of data received to an array
//from /wordpress//a/26536/10406
parse_str( $_POST['form_data'], $vars );

//check that are actually trying to publish a post
if ( $vars['post_status'] == 'publish' || 
    (isset( $vars['original_publish'] ) && 
     in_array( $vars['original_publish'], array('Publish', 'Schedule', 'Update') ) ) ) {
    if ( empty( $vars['_start_date'] ) || empty( $vars['_end_date'] ) ) {
        _e('Both Start and End date need to be filled');
        die();
    }
    //make sure start < end
    elseif ( $vars['_start_date'] > $vars['_end_date'] ) {
        _e('Start date cannot be after End date');
        die();
    }
    //check time is also inputted in case of a non-all-day event
    elseif ( !isset($vars['_all_day'] ) ) {
        if ( empty($vars['_start_time'] ) || empty( $vars['_end_time'] ) ) {
            _e('Both Start time and End time need to be specified if the event is not an all-day event');
            die();              
        }
        elseif ( strtotime( $vars['_start_date']. ' ' .$vars['_start_time'] ) > strtotime( $vars['_end_date']. ' ' .$vars['_end_time'] ) ) {
            _e('Start date/time cannot be after End date/time');
            die();
        }
    }
}

//everything ok, allow submission
echo 'true';
die();
}

true如果一切正常,此函数将返回,并通过正常渠道提交表单以发布帖子。否则,函数将返回一条错误消息,显示为alert(),并且不提交表单。


我遵循相同的方法,并在验证函数返回true时将帖子另存为“草稿”而不是“发布”。不确定如何解决该问题!<br/>在ajax调用期间还无法获取textarea字段(例如post_content,任何其他文本区域自定义字段)的数据吗?
Mahmudur

1
我采用的解决方案略有不同:首先,在成功的情况下,我在javascript中使用了以下代码:delayed_autosave(); //get data from textarea/tinymce field jQuery('#publish').data("valid", true).trigger('click'); //publish post非常感谢。
Mahmudur

3

我认为解决此问题的最佳方法不是防止状态变化发生,而应该防止它发生。例如:您save_post具有很高的优先级的hook(这样,该挂钩将在很晚的时候触发,即在您执行元插入之后),然后检查post_status刚刚保存的帖子的,并将其更新为待处理(或草稿或(无论如何)是否不符合您的条件。

另一种策略是挂钩wp_insert_post_data直接设置post_status。就我而言,此方法的缺点是您尚未将postmeta插入数据库中,因此您将必须对其进行处理等以进行检查,然后再次对其进行处理以进行插入。将其存入数据库...可能会增加性能或代码开销。


我目前正在save_post使用优先级1来保存metabox中的meta字段;您现在建议的是save_post优先拥有第二个钩子,例如99?这样可以确保完整性吗?如果由于某种原因触发了第一个挂钩,插入了元数据并发布了帖子,但是第二个挂钩没有触发,那么您最终得到了无效的字段怎么办?
englebip 2012年

我想不出第一个钩会触发但第二个钩不会触发的情况……您认为哪种情况可能会导致这种情况?如果您对此感到担心,则可以插入post meta,检查post meta,然后根据需要更新post_status单个调用钩子的所有功能。
mor7ifer 2012年

我发布了代码,作为对问题的修改;我试图使用第二个钩子,save_post但是触发了无限循环。
englebip 2012年

您的问题是您应该检查创建的帖子。所以,if( get_post_status( $post_id ) == 'publish' )是要被使用,因为您也将重新定义的数据是什么$wpdb->posts,而不是在数据$_POST[]
mor7ifer 2012年

0

最好的方法可能是JAVASCRIPT:

<script type="text/javascript">
var field_id =  "My_field_div__ID";    // <----------------- CHANGE THIS

var SubmitButton = document.getElementById("save-post") || false;
var PublishButton = document.getElementById("publish")  || false; 
if (SubmitButton)   {SubmitButton.addEventListener("click", SubmCLICKED, false);}
if (PublishButton)  {PublishButton.addEventListener("click", SubmCLICKED, false);}
function SubmCLICKED(e){   
  var passed= false;
  if(!document.getElementById(field_id)) { alert("I cant find that field ID !!"); }
  else {
      var Enabled_Disabled= document.getElementById(field_id).value;
      if (Enabled_Disabled == "" ) { alert("Field is Empty");   }  else{passed=true;}
  }
  if (!passed) { e.preventDefault();  return false;  }
}
</script>

-1

抱歉,我无法给您一个直接的答案,但我确实记得最近做过类似的事情,只是不记得具体如何。我想我可能大概是这样做的-就像我将其作为默认值一样,如果此人没有更改,我会在if语句中选择它,所以-> if(category==default category) {echo "You didn't pick a category!"; return them to the post creation page; }对不起,这不是直接答案,但希望它会有所帮助。

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.