カスタムフィールドが入力されていない場合、投稿が公開されないようにします


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;
}

回答:


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変数が正しくないURLにリダイレクトされるためです。

変更するには、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 14

@MadtownLems修正:)
スティーブンハリス14

9

OK、これが最終的に私がそれをやった方法です:チェックを行う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を返したときに、投稿を「公開」ではなく「下書き」として保存しました。それを修正する方法がわからない!!!
マフムドゥル

1
私はこのソリューションを少し異なる方法で適用しました。まず、成功した場合に備えて、javascriptで以下のコードを使用しdelayed_autosave(); //get data from textarea/tinymce field jQuery('#publish').data("valid", true).trigger('click'); //publish postました。
マフムドゥール

3

これを実行する最善の方法は、ステータスの変更が発生するのを防ぐのではなく、それが発生した場合にそれを後回しにすることです。たとえばsave_post、フックを非常に高い優先度でフックし(つまり、フックを非常に遅く、つまりメタ挿入を実行した後)、post_status保存されたばかりの投稿を確認し、保留中(またはドラフトまたはそれがあなたの基準を満たさない場合)。

別の方法は、フックwp_insert_post_dataしてpost_statusを直接設定することです。この方法の欠点は、私が知る限り、まだデータベースにポストメタを挿入していないため、チェックを行うために所定の場所でそれを処理する必要があることです。データベースにそれを...それはパフォーマンスまたはコードのいずれかで、多くのオーバーヘッドになる可能性があります。


私は現在save_post、優先度1でフックして、メタボックスからメタフィールドを保存しています。あなたが提案しているのは、save_post優先度、たとえば99の2番目のフックを持つことです?これにより整合性が確保されますか?何らかの理由で最初のフックが起動し、メタデータが挿入されて投稿が公開され、2番目のフックが公開されないため、無効なフィールドになった場合はどうなりますか?
englebip

最初のフックが起動し、2番目のフックは起動しない状況を考えることはできません...それを引き起こす可能性のあるシナリオは何ですか?心配な場合は、ポストメタを挿入し、ポストメタを確認し、必要にpost_status応じて、フックへの1回の呼び出しで実行される1つの関数ですべてを更新できます。
mor7ifer

質問の編集としてコードを投稿しました。私は2番目のフックを使用しようとしましたsave_postが、それは無限ループを引き起こします。
englebip

問題は、作成した投稿を確認する必要があることです。だから、if( get_post_status( $post_id ) == 'publish' )あなたはあなたがデータを再定義されるため、使用したいものです$wpdb->postsでは、データではなく$_POST[]
mor7ifer

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; }申し訳ありませんが、これはまっすぐな答えではなく、希望ですそれは少し役立ちます。

弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.