Display issue

  • Resolved barbuse95

    (@barbuse95)


    6 years, 10 months ago

    Hi,
    I’m using the Product addon along with PDF Invoice addon, as many people and it seems that the Product addon causes problems for the Invoice to display correctly. We are several people having the same problem and after discussing it with the author of the PDF Invoice, here’s what he said:

    Unfortunately this is not something that I can change from our end. I see that the “WooCommerce Personalized Product Option Manager” has had a big update recently. On our end, nothing has changed with the way item meta is treated since the very first release ??

    The “WooCommerce Personalized Product Option Manager” overrides that echo parameter (or any other parameters for that matter!) and sets it to ‘true’, thus displaying instead of returning the meta:
    https://plugins.trac.www.ads-software.com/browser/woocommerce-product-addon/trunk/inc/woocommerce.php#L689

    This is a bug in that plugin that we cannot do anything about because the PDF invoice plugin needs to collect all data first before it can put it in the invoice… Luckily this is very easy to fix for the developers of that plugin, so I recommend contacting their support, you can refer them to this thread for more information.

    Is there anyway to correct that quickly ?

    Thanks in advance ??

Viewing 4 replies - 1 through 4 (of 4 total)

  • Hi,

    I see, well i have added a hook/filter to change args, hooks name is

    	$args = apply_filters( 'ppom_woocommerce_display_item_meta_args', $args, $item );

    We will push this change in repo soon. Please ask them it’s enough to fix issue with PDF Invoice?

    Thread Starter barbuse95

    (@barbuse95)

    Thanks for your quick reply !
    I’ll them and I’ll get back to you asap!
    In the mean time, where can add the code you mentionned so I can alreadry try to correct the problem ?

    Thanks for your help.

    well PDF team will use above hook and they can turn echo on/off as per their need. we have pushed new changes in same version 4.1

    Thread Starter barbuse95

    (@barbuse95)

    Well they said your first solution wouldn’t change anything. Now, I don’t know if it comes from your update or their, but the problem has been solved with the today’s ones ??

Viewing 4 replies - 1 through 4 (of 4 total)
  • The topic ‘Display issue’ is closed to new replies.