Thankyou page tooop quick

Thanks Alan, we do understand and appreciate your efforts.
It only fails when the email address is the same and just overwrites the previous line / booking. The other parameters don’t matter.

Thanks

OK I can replicate that issue on test so I will be able to debug

There is a small logic hole there.

In that if payment hasnt completed ‘yet’ then using the same email will replace the existing as it is assuming that some on eis trying to complete a paymnet – e.g. that started then got to paypal and hit cancel ( as per your original instructions) then go back and try again the same person trying to pay.

The issue is that pay by voucher is not a ‘Paid’ state so it thinks it is pending too.

I need to think about the right fix.

• This reply was modified 1 year, 2 months ago by Alan Fuller.

First many thanks again for working late and that now works perfectly. We noticed the update. Sorry but theres another BUT.
It used to work when they booked if there was more than one passenger a box opened for other names, that now doesn’t happen. In the registration form theres a line saying “Show box for more names” This wasn’t ticked as shown in earlier picture but did work. We’ve tried ticking it but still the same. I’m not sure how important that is to us.
But more important its stopped showing the total cost on the customers register page. It does show correctly when you get to paypal.
We could remove that “Total” line but its good for the customer to have total confirmation.
Sorry for the bad news, we realise changing one thing can upset others, hope you can help.

The total and the more names’ is related to a javascript error.

Unfortunately this is quite hard for me to debug as it works fine on my test system.

I’ll see if I can work a way around it.

Can you try something

Edit this file

( You can go to Tools > Plugin File Editor > [Quick Event Manager] )

Edit

legacy/qem-event-register.php

line 677 and change the word ‘after’ to ‘before’

And let me know ( save, clear browser cache and test )

Have you had a chance to try this?

Hi Alan, I was away for weekend but just tried it. Had to enable plugin file editor then did exactly as suggested. I assume you were fixing the total amount but it didn’t help at all? We have added temporary text as you can see but it seemed the same? https://floatourboat.co.uk/book-trips/ I tried in different browsers. If you would prefer I can add you as WP user for you to try?

Sorry but the forum guideline specifically forbid providing login info

Do not post, offer, or ask for login information, even test IDs and test passwords. And do not offer off-site contact for the collection of those either.

https://www.ads-software.com/support/guidelines/

Can you try changing that one line to these three

add_action( ‘wp_enqueue_scripts’, function() use ( $id, $products ) {
wp_add_inline_script( ‘event_script’, ‘qem_multi_’ . (int) $id . ‘ = ‘ . wp_json_encode( $products ), ‘before’ );
} ,99,1);

I pasted in on the line and it said error in 678. went to another php file then it said I would lose changes and I thought it would revert to old. But it lost all the page so need to reload plugin. What is best way?

I’ve reinstalled qem from zip. not quite sure how to install the 3 lines. Do i overwrite 677, 8 and 9?

Sorry, I’m now out until 1700

Just replace 677

Hi Alan. I’ve tried pasting entire thing into 677 and it then goes into 678 and 9. I think because of carriage returns? Tried removing the returns in notepad and got errors. Then tried paste line by line and got as in picture.

Not sure what I’m doing wrong? Thanks

Oh don’t worry – I’m make you change a change that was already there, my fault for not looking hard enough.

If the original change didn’t work I’m at a limit of my ability to debug your site and not sure what to suggest.

I don’t know if you have any javascript optimization cache plugins, maybe they are causing an issue?