[NSFW] Conflict with WooCommerce DYMO Print

Hi @webmatthieu,

Apologies for the delay in response.

Could you please share the steps to replicate the issue so that it would be easy for us to check the issue in detail?

Regards,
Kenil Shah

Hi,

i have the same problem. When I deactivate your plugin, the URL when bulk printing labels with WooCommerce DYMO Print looks like this and the labels get printed:
https://www.MYWEBSITE.nl/wp-admin/edit.php?post_type=shop_order&paged=1&printID=169119%2C169118%2C169117%2C169116&type=order&label=shipping&bulk_printed=4

However, with your plugin active, I get a white screen, and nothing happens. The URL is as follows:
https://www.MYWEBSITE.nl/wp-admin/edit.php?s=&post_status=all&post_type=shop_order&_wpnonce=e9b59295fc&_wp_http_referer=%2Fwp-admin%2Fedit.php%3Fpost_type%3Dshop_order&action=wc_dymo_order_shipping&m=0&_customer_user=&paged=1&post%5B%5D=169291&action2=-1

Any solution?

Thanks in advance,

Paul

Hi @nlnieuw7,

The URL which you have shared is not getting opened for us. So, could you please share the steps to replicate the issue and also the screenshots of the issue which you are facing so that we can check it?

Regards,
Kenil Shah

Hi Kenil,

I provided the URL to show you the effect of your plugin on WooCommerce DYMO Print.

As you can see, the URL that should activate the Dymo printer to print labels in bulk gets screwed up when your plugin is active.

Regards,

Paul

Hello all,

As developer of WooCommerce DYMO Print we’ve get into this issue.

As soon as your plugin “Print Invoice & Delivery Notes for WooCommerce” is installed the BULK printing options are changed and the BULK printing options of our plugin WooCommerce DYMO Print (PRO version) are not working anymore.

We’ve get into the code and found that in /wp-content/plugins/woocommerce-delivery-notes/includes/class-wcdn-writepanel.php filter / function my_bulk_action_handler the returned redirect_to url is changed by your plugin EVEN if this is not necessary.

We advise to check if the correct args are send.
For example to add a if statement like below on line 140:
if(!in_array($_REQUEST['action'],array('wcdn_print_invoice','wcdn_print_delivery-note','wcdn_print_receipt'))) return $redirect_to;

Hopefully this will help you solving this issue.

Best regards,
Bart

Exactly, @pepbc is right.
I have a simple code to allow some csv export of metadata in orders. I had the same problem, once I added conditional statement in your plugin like @pepbc proposed, both functions started to function properly, my code and your plugin.

I’ve been waiting for a response from Kenil for almost 4 weeks now. Bart (@pepbc) is indeed right. On lines 139-141 I replaced
if ( ! isset( $_REQUEST[‘post’] ) ) { return; }
with Bart’s suggestion:
if(!in_array($_REQUEST[‘action’],array(‘wcdn_print_invoice’,’wcdn_print_delivery-note’,’wcdn_print_receipt’))) return $redirect_to;
and everything works fine again.
Thanks a lot Bart!

Kenil,

Can you incorporate this in the next version? Thanks.

Paul

Hi @webmatthieu,

Apologies for the delay in response.

We will check that code and make the necessary changes in the next update.

Regards,
Kenil Shah

Hi Kenil,

Your last update didn’t do the trick. I had to manually edit the file again. It doesn’t work without the “$redirect_to” after “return”.

It does work like this:
if(!in_array($_REQUEST[‘action’],array(‘wcdn_print_invoice’,’wcdn_print_delivery-note’,’wcdn_print_receipt’))) return $redirect_to;

Can you fix that in the next update please?

Thanks!