Renvoyer $ post_id quand DOING_AUTOSAVE?

Question

Renvoyer $ post_id quand DOING_AUTOSAVE?

- 8
- 4700 2011-04-07
Je vois lemodèle suivantencoreet encore, sur ce site et à d'autresendroits:

add_action( 'save_post', 'wpse14169_save_post' ); function wpse14169_save_post( $post_id ) { if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) { return $post_id; } // Other code... }

Pourquoi devrais-je renvoyer $post_id? save_postest une action,et la valeur de retour d'ungestionnaire d'actionestignorée . Lenoyau WordPress lui-même ne faitpasnonplus .

L'exemple Codex renvoie le $post_id ,mais cene seraitpas lapremière ligneincorrecte (ou obsolète) du Codex.

Est-ce queje manque quelque chose? Dois-je renvoyer $post_id? Y a-t-ileu unmoment où cela étaitnécessaire?
I see the following pattern over and over, on this site and on other places:

add_action( 'save_post', 'wpse14169_save_post' ); function wpse14169_save_post( $post_id ) { if ( defined( 'DOING_AUTOSAVE' ) && DOING_AUTOSAVE ) { return $post_id; } // Other code... }

Why should I return $post_id? save_post is an action, and the return value of an action handler is ignored. The WordPress core itself doesn't do it either.

The Codex example does return the $post_id, but it would not be the first incorrect (or outdated) line in the Codex.

Am I missing something? Do I need to return $post_id? Was there a there a time when this was needed?
code customization save-post actions

- Pour référence,voici [modifier qui aintroduit lapremière version de ceci dans le Codex] (http://codex.wordpress.org/index.php?title=Function_Reference/add_meta_box&diff=77958&oldid=74247).Jene reconnaispas la connexion,vouspouvezessayer de rechercher unepersonneet demander (si cela vousintéresse).
  
  For reference here is [edit that introduced early version of this to Codex](http://codex.wordpress.org/index.php?title=Function_Reference/add_meta_box&diff=77958&oldid=74247). I do not recognize the login, you could try to look person up and ask (if that interested).
  - 0
  - 2011-04-07
  - Rarst

2 réponses

votes

7

2011-05-03

Puisque rienn'estfait avec la valeur de retour,renvoyer l'ID depublicationestinutileet ne doitpas êtrefait.Celane laisseplace qu'à la confusion.

Je viens de l'essayer,l'action save_post suivantefonctionnebien.

 function my_save_post($post_id)
{
    // Stop WP from clearing custom fields on autosave
    if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE)
        return;

    // Prevent quick edit from clearing custom fields
    if (defined('DOING_AJAX') && DOING_AJAX)
        return;

    // Sanitize, validate and save ...

}

Since nothing is being done with the return value, returning the post ID is pointless and should not be done. It only provides room for confusion.

Just tried it out, the following save_post action works fine.

function my_save_post($post_id)
{
    // Stop WP from clearing custom fields on autosave
    if (defined('DOING_AUTOSAVE') && DOING_AUTOSAVE)
        return;

    // Prevent quick edit from clearing custom fields
    if (defined('DOING_AJAX') && DOING_AJAX)
        return;

    // Sanitize, validate and save ...

}

John P Bloch · Accepted Answer · 2011-04-07

5

2011-04-07

L'action 'save_post' était ajouté aunoyau dans la version 2.0 ,et atoujours été une action.Enparcourant lesprocédures d'enregistrement automatiqueen cours,ilne semblepas appeler l'action 'save_post' directement àtoutmoment.

La réponse courteest doncnon.Iln'y a aucune raison,et n'ajamais été aucune raison,de renvoyer une valeur sur cette action.Bien sûr,celane faitpas demal de renvoyer l'identifiant dumessage.