Impedire la pubblicazione di un post se i campi personalizzati non sono compilati

Come ha sottolineato m0r7if3r, non esiste un modo per impedire la pubblicazione di un post utilizzando l'hook save_post, poiché al momento in cui questo hook viene attivato, il post è già stato salvato. Tuttavia, il seguente codice ti permetterà di ripristinare lo stato del post senza utilizzare wp_insert_post_data e senza causare un loop infinito.

Il seguente codice non è stato testato, ma dovrebbe funzionare.

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

    // L'utente ha il permesso di modificare il post o la pagina?
     if ( !current_user_can( 'edit_post', $post->ID ) )
         return;

   // Ora esegui i controlli per validare i tuoi dati. 
   // Nota che i campi personalizzati (diversi dai dati nelle metabox personalizzate!) 
   // saranno già stati salvati.
    $prevent_publish= false;//Imposta su true se i dati non sono validi.
    if ($prevent_publish) {
        // Rimuovi l'hook per prevenire un loop infinito
        remove_action('save_post', 'my_save_post');

        // Aggiorna il post per cambiare lo stato
        wp_update_post(array('ID' => $post_id, 'post_status' => 'draft'));

        // Ripristina l'hook
        add_action('save_post', 'my_save_post');
    }
}
?>

Non ho verificato, ma osservando il codice, il messaggio di feedback mostrerà erroneamente che il post è stato pubblicato. Questo perché WordPress ci reindirizza a un URL in cui la variabile message ora non è corretta.

Per modificarlo, possiamo utilizzare il filtro redirect_post_location:

add_filter('redirect_post_location','my_redirect_location',10,2);
function my_redirect_location($location,$post_id){
    // Se il post è stato pubblicato...
    if (isset($_POST['publish'])){
        // Ottieni lo stato corrente del post
        $status = get_post_status( $post_id );

        // Il post è stato "pubblicato", ma se è ancora una bozza, mostra il messaggio di bozza (10).
        if($status=='draft')
            $location = add_query_arg('message', 10, $location);
    }

    return $location;
}

Per riassumere il filtro di reindirizzamento sopra: se un post è impostato per essere pubblicato, ma è ancora una bozza, allora modifichiamo il messaggio di conseguenza (che è message=10). Ancora una volta, questo non è stato testato, ma dovrebbe funzionare. Il Codex di add_query_arg suggerisce che quando una variabile è già impostata, la funzione la sostituisce (ma, come ho detto, non ho testato questo).

Penso che il modo migliore per affrontare questo problema non sia tanto PREVENIRE il cambiamento di stato quanto REVERTIRLO se dovesse verificarsi. Ad esempio: puoi agganciarti a save_post con una priorità molto alta (in modo che l'hook venga eseguito molto tardi, cioè dopo aver inserito i tuoi meta), quindi verificare lo post_status del post appena salvato e aggiornarlo a "pending" (o "draft" o altro) se non soddisfa i tuoi criteri.

Una strategia alternativa sarebbe agganciarsi a wp_insert_post_data per impostare direttamente il post_status. Lo svantaggio di questo metodo, per quanto mi riguarda, è che non avrai ancora inserito i postmeta nel database, quindi dovrai elaborarli, ecc. sul momento per fare i tuoi controlli, poi elaborarli nuovamente per inserirli nel database... il che potrebbe comportare un sovraccarico significativo, sia in termini di prestazioni che di codice.

Il metodo migliore potrebbe essere JAVASCRIPT:

<script type="text/javascript">
var field_id =  "My_field_div__ID";    // <----------------- CAMBIA QUESTO

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("Non riesco a trovare quell'ID di campo !!"); }
  else {
      var Enabled_Disabled= document.getElementById(field_id).value;
      if (Enabled_Disabled == "" ) { alert("Il campo è vuoto");   }  else{passed=true;}
  }
  if (!passed) { e.preventDefault();  return false;  }
}
</script>

Mi dispiace non poterti dare una risposta diretta, ma ricordo di aver fatto qualcosa di simile molto recentemente, solo che non ricordo esattamente come. Penso di averlo fatto in modo indiretto - qualcosa come impostarlo come valore predefinito e se l'utente non l'avesse cambiato, lo rilevavo in un'istruzione if quindi -> if(categoria==categoria predefinita) {echo "Non hai selezionato una categoria!"; return reindirizza alla pagina di creazione articolo; } scusa se non è una risposta diretta ma spero possa aiutare un po'.