Erreur dans la fonction LpcOrderQueries::getLpcOrdersIdsForPurge()

  • Resolved Yvan

    (@dugwood)


    4 months ago

    Bonjour,

    j’ai forcé les erreurs sur WordPress (qui en masque certaines), et il y a une grosse erreur dans le script LpcOrderQueries::getLpcOrdersIdsForPurge() :

    Ligne 342 : ???????????????$ordersId[] = $result->order_id;

    order_id n’est pas défini dans le cas où le test if (self::isHposActive()) { retourne false. La requête SQL ne retourne que des post_id.

    En attendant je vais ignorer l’info, puisqu’elle créait un tableau plein de NULL.

    Merci, Yvan.

Viewing 3 replies - 1 through 3 (of 3 total)
  • Thread Starter Yvan

    (@dugwood)

    Et il serait bon que vous activiez les NOTICE sur PHP, histoire de voir des bugs de ce type, qui sont critiques. Je sais que WordPress les dégage sans raison, mais il faut les forcer. Dans un environnement de debug, il suffit d’activer le mode WP_DEBUG et ?a force toutes les erreurs.

    à défaut il faut :

    • activer le filtre comme décrit dans la fonction wp_debug_mode() (dans le fichier load.php)
    • désactiver la fonction error_reporting() dans PHP, ou supprimer le code moisi de WP qui force sans autre forme de procès le masquage des NOTICE dans wp-load.php (ligne 36).
    Plugin Author Colissimo

    (@iscpcolissimo)

    Bonjour,

    Nous utilisons ceci dans wp-config.php :

    define('WP_DEBUG', true);
    define('WP_DEBUG_DISPLAY', true);

    Mais étant donné que ce code est exécuté en différé, on a pu rater l’erreur. Nous ajouterons define('WP_DEBUG_LOG', true); pour ce genre de cas.

    Merci d’avoir remonté ce bug, il sera corrigé dans la prochaine version ??
    En attendant le patch, vous pouvez remplacer la ligne 324 :

    $query    = 'SELECT DISTINCT isDelivered.post_id

    par :

    $query    = 'SELECT DISTINCT isDelivered.post_id AS order_id

    Bonne journée

    Thread Starter Yvan

    (@dugwood)

    Merci pour cette réponse rapide ??

    Effectivement, c’était en différé… je ne l’ai vu que parce que je coupe mes scripts après 100 erreurs ??

    C’est noté pour le patch, je me doutais que c’était ?a, mais je préférai ne rien faire en attendant (puisque justement ?a ne faisait rien avec le bug), des fois que j’empire la chose ??

    Bonne journée !

Viewing 3 replies - 1 through 3 (of 3 total)
  • You must be logged in to reply to this topic.