Support » Plugin: Product Customer List for WooCommerce » JavaScript error with post types that don’t support titles

  • Resolved Alexandre Simard

    (@brocheafoin)


    Hey there! (Salut Québec!)

    Great plugin, obviously. I’m having a small issue with it though. Your JavaScript is enqueued on all post edit screens. It also starts straight away by grabbing the value of the post title’s form input and performing string operations on it. Unfortunately, this causes JavaScript errors if the title input is not present, which _can_ happen, as titles are optional for posts.

    Here’s my suggested fix:

    
      global $post;
      if ( 'post.php' != $hook || 'product' != get_post_type( $post ) ) {
    

    Use it to replace line 358 in wc-product-customer-list.php. Cheers!

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Author Kokomo

    (@kokomoweb)

    Merci Alex, étrangement j’étais sur d’avoir déja mis cette vérification. je ferai la correction dans mon prochain update (major update, pour WC 3). Tant qu’à refaire une grosse partie du plugin, est-ce que tu suggère une façon plus efficace pour que j’aille chercher le titre du produit pour générer le titre / filename du PDF?

    L’ajouter dans ton wp_localize_script? Ça te permettrait aussi de le sanitizer en PHP avec les fonctions de WP.

    Plugin Author Kokomo

    (@kokomoweb)

    Tellement! Merci pour le hint, je vais faire ça. Je te fais signe quand la nouvelle version est en ligne!

    Plugin Author Kokomo

    (@kokomoweb)

    Nouvelle version en ligne, avec wp_localize_script, compat WC 3.0+ pis des nains qui font des backflip. Enjoy!

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘JavaScript error with post types that don’t support titles’ is closed to new replies.