Square WebPay support thread.

carlwhat:

on the site you are trying to fix, i have confirmed that we are getting a response back from square. see:

https://www.dropbox.com/s/ubo8m9jjwbill9k/Screenshot%20from%202022-04-20%2009-50-25.png?dl=0

i have determined a potential problem with this file.

includes/modules/pages/checkout_payment/jscript_square_webPay.php

there is a new version of said file here for anyone who wants to test it:

https://github.com/proseLA/square_webPay/blob/master/for_upload/includes/modules/pages/checkout_payment/jscript_square_webPay.php

this problem seems to be confined to OPC.

best.

I have replaced jscript_square_webPay.php with the RAW FILE and still getting a problem with OPC

carlwhat:

on the site you are trying to fix, i have confirmed that we are getting a response back from square. see:

https://www.dropbox.com/s/ubo8m9jjwbill9k/Screenshot%20from%202022-04-20%2009-50-25.png?dl=0

i have determined a potential problem with this file.

includes/modules/pages/checkout_payment/jscript_square_webPay.php

there is a new version of said file here for anyone who wants to test it:

https://github.com/proseLA/square_webPay/blob/master/for_upload/includes/modules/pages/checkout_payment/jscript_square_webPay.php

this problem seems to be confined to OPC.

best.

These are the settings for 'Order Total, JQUERY Selector' in OPC

Order Total, jQuery Selector
Please make any necessary changes

Order Total, jQuery Selector
Identify the CSS/jQuery selector used to locate an order's current total value. The default value, #ottotal > div:first-child applies to the Zen Cart responsive_classic template's format.

#ottotal .ot-text

These are in the admin to load the bootstrap template order total selector div. I remember having a similar issue to the issues I am having now when I first started using OPC. It was the same error until I applied the selector, as it is in the image, and given to me by Cindy.

So I am wondering if this setting might be part of the issue?

nick,
please go here:

admin -> configuration -> One-Page Checkout Settings -> Load Minified Script Files

and set to:

FALSE.

also on that page, what is the value of:

Payment-Block Action on Shipping Change

Key: CHECKOUT_ONE_PAYMENT_BLOCK_ACTION

carlwhat:

also on that page, what is the value of:

Payment-Block Action on Shipping Change

Key: CHECKOUT_ONE_PAYMENT_BLOCK_ACTION

It's set to 'Update'

carlwhat:

nick,
please go here:

admin -> configuration -> One-Page Checkout Settings -> Load Minified Script Files

and set to:

FALSE.

Still getting the same issue. Is this something that Cindy LAT9 can help with?

carlwhat:

also on that page, what is the value of:

Payment-Block Action on Shipping Change

Key: CHECKOUT_ONE_PAYMENT_BLOCK_ACTION

I've turned OPC off for now anyway as I know Square Works with the standard cart and I need to get some orders coming through.

Nick1973:

Still getting the same issue. Is this something that Cindy LAT9 can help with?

nick,
i only had you turn off minifying to help with my trouble shooting.

rest assured, i am close to figuring out the problem. unfortunately i am not able to reproduce it on my dev box as of yet, but as soon as i am able, i will have a solution.

rest assured, cindy has complete confidence in my trouble shooting. and this problem will get solved.

but paying customers first, and i got a bunch of stuff on my plate.

carlwhat:

nick,
i only had you turn off minifying to help with my trouble shooting.

rest assured, i am close to figuring out the problem. unfortunately i am not able to reproduce it on my dev box as of yet, but as soon as i am able, i will have a solution.

rest assured, cindy has complete confidence in my trouble shooting. and this problem will get solved.

but paying customers first, and i got a bunch of stuff on my plate.

Ok no problem Carl, and yes paying customers do come first, plus the marketing to actually get them in the first place.

So can you work with OPC off on my end? Or do you need it on?

It's just I would like some actual orders that pay money :-)

better if it is on... but do as you need to...

carlwhat:

better if it is on... but do as you need to...

Ok I have put OPC into Conditional Mode on ventureengravings.uk

You are the only customer that will be able to use it. Your Customer ID is 249

Nick1973:

Ok I have put OPC into Conditional Mode on ventureengravings.uk

You are the only customer that will be able to use it. Your Customer ID is 249

ok, @Nick1973 and whoever else is paying attention, this is what i have found.- we have 2 event listeners competing for the same event.

• when, the square_webpay event listener gets registered first, there is a successful transaction.
• when the OPC event listener gets registered first, the square submittal does not happen, and hence there is no token that gets submitted as part of the form.
• i have no idea why on some sites, the event listener gets registered first in OPC and other times square_webpay.
• i am working on addressing that.

as a test, leave Minify set to FALSE - always... until the problem is resolved.

try commenting out these 2 lines:

https://github.com/lat9/one_page_checkout/blob/e484351e78d3cef19552f1b5203670140ef9d608/includes/modules/pages/checkout_one/jquery.checkout_one.js#L545
https://github.com/lat9/one_page_checkout/blob/e484351e78d3cef19552f1b5203670140ef9d608/includes/modules/pages/checkout_one/jquery.checkout_one.js#L568

and try placing an order.

carlwhat:

ok, @Nick1973 and whoever else is paying attention, this is what i have found.

• we have 2 event listeners competing for the same event.

• when, the square_webpay event listener gets registered first, there is a successful transaction.

• when the OPC event listener gets registered first, the square submittal does not happen, and hence there is no token that gets submitted as part of the form.

• i have no idea why on some sites, the event listener gets registered first in OPC and other times square_webpay.

• i am working on addressing that.

as a test, leave Minify set to FALSE - always... until the problem is resolved.

try commenting out these 2 lines:

https://github.com/lat9/one_page_checkout/blob/e484351e78d3cef19552f1b5203670140ef9d608/includes/modules/pages/checkout_one/jquery.checkout_one.js#L545
https://github.com/lat9/one_page_checkout/blob/e484351e78d3cef19552f1b5203670140ef9d608/includes/modules/pages/checkout_one/jquery.checkout_one.js#L568

and try placing an order.

Ok without those two lines, the order processes and completes. The order emails are also produced.

I get "your details have changed" BUT, I have the changes to the jscript_square_webPay.php file in place. Should I be testing without those?

dbltoe:

I get "your details have changed" BUT, I have the changes to the jscript_square_webPay.php file in place. Should I be testing without those?

Are you talking to me or @carlwhat?

carlwhat

With just the changes to the OPC file, I get a good sale without the "Your settings have changed"

the changes to the:

jscript_square_webPay.php

are fine and should be left in.

i am seeing both sites as now working without problem.

changing the minify to TRUE will, at this point, not work.

i will hopefully work on a solution with @lat9 to resolve this compatibility issue with OPC.

Just a side note:

Well, two

First is that I send all log files I get from the test sites just in case they might make a difference. I think one or two that turn out to be irrelevant is better than not sending the one that is.

And, because of that, I will mentiion that two upgraded sites (one with and one without OPC) will not display the details of the order (below the payment method line) without retaining the old includes/modules/payment/square_support and its files.

OTOH, several other sites work fine without that folder and files.

More info available on request. I have to break sites to get the log file for the support folder. Don't want to do that during business hours.

dbltoe:

Just a side note:

Well, two

First is that I send all log files I get from the test sites just in case they might make a difference. I think one or two that turn out to be irrelevant is better than not sending the one that is.

And, because of that, I will mentiion that two upgraded sites (one with and one without OPC) will not display the details of the order (below the payment method line) without retaining the old includes/modules/payment/square_support and its files.

OTOH, several other sites work fine without that folder and files.

More info available on request. I have to break sites to get the log file for the support folder. Don't want to do that during business hours.

just so we are clear

• unless i ask for log files, please do not send them.
• i NEVER said to delete or remove includes/modules/payment/square_support
• i am sure other sites work without those files being there due to a config setting. but, IMO those files need to remain there.
• for the next release i will include and package those files in the zip.
• the next release will have a little better error messaging. and the logs will not have what i now deem to be superfluous data. i'm a fan of logs having only the relevant data. as more feedback comes in, logs can be better streamlined as apache or nginx have the more complete and detailed logs.
• i'm really not sure what to make of your last point. you need to break a site in order to get to any log files? i do not want to insult the host :p, but needing to break a site to get to any log files is a problem. i look at log files all the time with the site still being up. i'm going to go with i do not understand what you are saying.
• i am happy we got this problem resolved.