borish1

Sure thing.

Should I submit a support ticket?

Hey @stevendebeer thanks for the fast response.

The end URL is this one:
/wp-admin/admin.php?page=wp-google-maps-menu&action=edit&map_id=7

So it gives proper map_id.

Additional thing to mention:

Each image has this css applied:
#sb_instagram .sbi_no_js img = display:None;

If we turn off display none all images except last 2 shows up.

Hey Ewout,

Thank you for the fast response.

I understand that but we would need to keep the pdf files inside the WordPress as we are mounting the ERP system with the store. The ERP system needs to have access to all the invoices so I thought just to generate them all inside the /attachments/ folder and keep the automatic cleanup turned off. Via htaccess IP can be whitelisted to access them as I see.

So the main problem would be how to handle manual orders where you generate a manual invoice that won’t get saved inside attachments?

Do you have a better approach with your plugin that I could achieve?

I could do a custom implementation if there is a solution for this I just need some keypoints where to look.

• This reply was modified 3 years, 9 months ago by borish1.
• This reply was modified 3 years, 9 months ago by borish1.

Any update on this one?

Hey @alexmigf,

First of all, thank you for the fast response, appreciate it. I’ve taken your proposition as a last resort but I managed to work it out.

I’ll give it a sort of explanation in case someone finds this useful.

I modified the existing simple template of the plugin by adding an additional page that I need to send to the third-party service. I’m doing some checkups there before I load that second page. To be more exact I have a custom field that is used to check if the woo emails are sent. At the first check, they are not so at that time I included that second page. (simple true/false field). I’m hooking in one of the “after_woo_email_sent” hooks (can’t remember the name and saving the field.

Then I hook into woocommerce_checkout_order_processed and do the API connection/sending of the PDF. Before I send the actual PDF what I do is, instead of just looking for the existing generated PDF I put the existing data that we can get through get_pdf() into already predefined $path.

`$invoice = wcpdf_get_invoice($order, true);
$pdf_data = $invoice->get_pdf();
$tmp_path = WPO_WCPDF()->main->get_tmp_path(‘attachments’);
$filename = $invoice->get_filename();
$pdf_path = $tmp_path . $filename;
file_put_contents($pdf_path, $pdf_data);
return realpath($pdf_path);`

Now, here the main problem was that I would get that same generated PDF 2 pager sent to customers but then I found 2 actions inside your plugin.
First one being wpo_wcpdf_email_attachment where I regenerate the existing PDF.

add_action("wpo_wcpdf_email_attachment","cr_regenerate_pdf",10,3);

function cr_regenerate_pdf($pdf_path, $document_type, $document) {
    $pdf_path = get_pdf_path($document->order);
}

Now I also found that I need to lower the time at which the plugin will “let” user regenerate the file by using wpo_wcpdf_reuse_attachment_age and lowering it to 1 second since both emails are being sent 1 after another. Otherwise plugins locks the file and that was the main reason I couldn’t regenerate it instantly without using this filter.

add_filter("wpo_wcpdf_reuse_attachment_age","cr_min");
function cr_min() {
    return 1;
}

After that API sends the regenerated PDF which is 1 pager this time to the customer.

• This reply was modified 4 years, 4 months ago by borish1.

Key difference is an additional page so if you look at it I could be using the same template but then I need to figure out a way to include that additional page in the template for third party service while giving the default one without it to the customers.

Some additional notes:

https://ibb.co/VDwCZ1Y

On the image, you can preview part of the invoice from the PayPal dashboard.
The part that is important is the CUSTOM row which is basically a “Note” by the PayPal API documentation.

That functions of yours custom() is passing only order_key but I need order_id as well.

I hope this helps :).

Hey Andreas,

Yep, I modified it but directly inside the files (only on my development server to test it). The file is located inside /src/payment/ folder by the name of patchProvider.php.

The key part of the functions is this one
->setValue($this->order->get_order_key());

Currently, your plugin, upon successful creation of the invoice is sending custom data which in this case is wc_order_key.

Now if we go back to 1.1.1 version of this plugin it actually sent both the wc_order_key and and order_id which is this part of code that I changed into:

->setValue(wp_json_encode([
                'order_id' => $this->order->get_id(),
                'order_key' => $this->order->get_order_key(),
            ]));

Now my client is using third party software which pulls data via PayPal API from his PayPal account to double-check the order_key and order_id from the invoices but in 2.0.4 version we only have now order_key.

That is the main reason why I needed that order_id.

It would be good if you can put filter there so we can then add additional data without need to edit the core files of the plugin.

Hey @perryrylance,

I tried switching version from 7.0 to 7.3 with no changes (don’t want to go with lower version of PHP because of one plugin, no point).

libxml is installed, I confirmed that via phpinfo(); but still no fix.
– I tried speaking to the support of the shared hosting but they told me that -enable-dom flag is not added at all.

I went back to the older version since this is the last resort I can rely to currently. I hope you fix this in the next version. The version in which the error doesn’t occur is 7.10.18.

I tried 7.10.19 and it gives back same error so in that version there were some changes regarding this issue. Hope it helps you.

Hey @aweissinpsyde,

Sorry for a little delay. I sent you the last message from here to [email protected] as requested. If you need any further information do not hesitate to reply.

Kind regards,
Boris

@aweissinpsyde, maybe we could, after all, fix this (regarding an issue with the order_key):

To answer your question Datev gets the data directly from PayPal. The user logs in to the Datev system and has access of the payments etc that are pulled directly from PayPal. There are no custom inputs available for me to change in the Datev System. (as far as my clients tell me).

So once the Datev system pulls the data from the PayPal it looks something like this (part of the system)
https://prntscr.com/nmq1yo

As you can see under “Text” it gives back those 2 parameters order_id and order_key.

This is the 1.1.1 version passing both order_id and order_key:
https://prntscr.com/nmpxt9

This is the 2.0.2 version passing only order_key
https://prntscr.com/nmpyi3

Can we somehow bring back passing of both again?

• This reply was modified 5 years, 6 months ago by borish1.

Hey @aweissinpsyde,

I’ve reverted the plugin to the older version 1.1.1 since we have little information as to how is Datev Zahlungsverkehr system setup because other company did that. I don’t have thorough input for you so we could inspect further what is missing where. I will need to find a way for this to work again.

Additionally, I wanted to ask you about the 404 error that triggered. While two payments went through with no problem one went with 404.

[09-05-2019 06:06:51] Inpsyde\Lib\PayPal\Core\PayPalHttpConnection : ERROR: Got Http response code 404 when accessing https://api.paypal.com/v1/payments/payment/PAYID-***********************. {"name":"INVALID_RESOURCE_ID","message":"The requested resource ID was not found","information_link":"https://developer.paypal.com/docs/api/payments/#errors","debug_id":"956070d36d1df"}

Any way I can debug this as could what could have caused it?

Having the same issue. Any help appreciated.

Had the same problem but I found a solution to it. Basically what UM does is if class_exists(“WooCoommerce”) it dequeues select2 both style and script. There is a hook called “um_dequeue_select2_scripts” to remove the select2 that UM adds but that works only partially as the case for this to work is actually this one:

$dequeue_select2_status = apply_filters(“um_dequeue_select2_scripts”, false);
if(class_exists(“WooCoommerce”) || $dequeue_select2_status)

This means that you cannot really force it to be removed if you have woocommerce active. What you can do is deregister and dequeue select2 and then re-enqueue one for the woocommerce as a solution for now. That will bring both functionality and style of the default woo. Also make sure that your “wp_enqueue_scripts” is above priority of 100 as theirs is 100.

Hope it will help someone.