Запретить публикацию записи, если не заполнены пользовательские поля

Как отметил 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;

    // Имеет ли пользователь право редактировать запись или страницу?
     if ( !current_user_can( 'edit_post', $post->ID ) )
         return;

   // Теперь выполняем проверки для валидации данных. 
   // Обратите внимание, что произвольные поля (отличные от данных в произвольных метабоксах!) 
   // уже будут сохранены.
    $prevent_publish= false;//Установите true, если данные невалидны.
    if ($prevent_publish) {
        // отключаем эту функцию, чтобы избежать бесконечного цикла
        remove_action('save_post', 'my_save_post');

        // обновляем запись, чтобы изменить её статус
        wp_update_post(array('ID' => $post_id, 'post_status' => 'draft'));

        // снова подключаем эту функцию
        add_action('save_post', 'my_save_post');
    }
}
?>

Я не проверял, но, глядя на код, сообщение обратной связи будет отображать некорректную информацию о том, что запись была опубликована. Это происходит потому, что WordPress перенаправляет нас на URL, где переменная message теперь неверна.

Чтобы изменить это, мы можем использовать фильтр redirect_post_location:

add_filter('redirect_post_location','my_redirect_location',10,2);
function my_redirect_location($location,$post_id){
    //Если запись была опубликована...
    if (isset($_POST['publish'])){
        //получаем текущий статус записи
        $status = get_post_status( $post_id );

        //Запись была 'опубликована', но если она всё ещё черновик, показываем соответствующее сообщение (10).
        if($status=='draft')
            $location = add_query_arg('message', 10, $location);
    }

    return $location;
}

Подводя итог по фильтру перенаправления: если запись должна быть опубликована, но остаётся черновиком, мы изменяем сообщение соответственно (это message=10). Опять же, код не тестировался, но должен работать. Согласно Codex функции add_query_arg, если переменная уже установлена, функция заменяет её (но, как я уже сказал, я не тестировал это).

Я считаю, что лучший подход заключается не в том, чтобы ПРЕДОТВРАЩАТЬ изменение статуса, а в том, чтобы ВОССТАНАВЛИВАТЬ его, если оно произошло. Например: можно использовать хук save_post с очень высоким приоритетом (чтобы хук срабатывал как можно позже, уже после вставки метаданных), затем проверить post_status только что сохранённой записи и изменить его на "pending" (или "draft", или другой нужный статус), если он не соответствует заданным критериям.

Альтернативная стратегия — использовать хук wp_insert_post_data для непосредственной установки статуса записи. Недостаток этого метода, на мой взгляд, в том, что на момент срабатывания хука метаданные ещё не будут сохранены в базу данных, поэтому придётся обрабатывать их дважды: сначала для проверки условий, а затем для вставки в базу... что может привести к значительным накладным расходам — как по производительности, так и по объёму кода.

Лучший метод может быть на JAVASCRIPT:

<script type="text/javascript">
var field_id =  "My_field_div__ID";    // <----------------- ИЗМЕНИТЕ ЭТО

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("Я не могу найти поле с таким ID !!"); }
  else {
      var Enabled_Disabled= document.getElementById(field_id).value;
      if (Enabled_Disabled == "" ) { alert("Поле пустое");   }  else{passed=true;}
  }
  if (!passed) { e.preventDefault();  return false;  }
}
</script>

Извините, я не могу дать вам прямой ответ, но я помню, что совсем недавно делал что-то подобное, просто не могу вспомнить точно как. Кажется, я сделал это обходным путем - что-то вроде установки значения по умолчанию, и если пользователь его не изменил, я отлавливал это в условии if, например: if(category==default category) {echo "Вы не выбрали категорию!"; return them to the post creation page; }. Извините, что это не прямой ответ, но надеюсь, хоть немного поможет.