Square Payment Module for Zen Cart [Support Thread]

[dbltoe]

Those files are from the latest download of the plugin which is newer than the release of 1.5.6c

We have had dual problems with Square. First, Square updated the API in late March.

Second, The latest download for Square is WAY different (over 100) than stock 1.5.6c

[carlwhat]

Those files are from the latest download of the plugin which is newer than the release of 1.5.6c

We have had dual problems with Square. First, Square updated the API in late March.

Second, The latest download for Square is WAY different (over 100) than stock 1.5.6c

i would still find out what is in $errors_object first before modifying the code.

in addition, looking at v157, if you have logging turned on, there should be a log file that can tell you exactly what the problem is. it looks like you are doing a refund. correct? what does the log say about the refund?

again, i have not used square before, but i know refunds are tricky. was the transaction settled? does square do voids (the function exists there).

i suppose you could look make this change that you are talking about. you are already at the end of the function, and this is just adding the error to the messageStack. but the error is not in the expected format. why? the log file can give you an indication...

hope that helps.

[RixStix]

[QUOTE=dbltoe;1367411]PHP 7.2 is causing an error in line 1110 of includes/modules/payment/square.php.

[07-Apr-2020 17:35:49 UTC] Request URI: /Ud;reW/orders.php?origin=index&page=1&oID=1718&action=doRefund, IP address: 123.231.145.199#1  square->_doRefund() called at [/home/allpecans/public_html/Ud;reW/orders.php:254]
--> PHP Warning: count(): Parameter must be an array or an object that implements Countable in /home/Ud;reW/public_html/includes/modules/payment/square.php on line 1110.

Just removed and reinstalled SQUARE 1.0.
1. Resolved my editing issue above.

2. Receive the EXACT same myDEBUG file when refunded the 1st test transaction.

AFAIK, Square does not do voids. Transaction was refunded within a minute of being authorized and captured.

[dbltoe]

Are we starting to channel RodG or are you still upset about RixStix not including you in the "Good Guy List"?

PHP states that ANY use of count() with 7.2 will generate the warning listed. At https://www.php.net/manual/en/function.count.php you will need to scroll down to Change Log to see their statement.

I did not say there was any error in operation of the cart.

PHP 7.2 is causing an error in line 1110 of includes/modules/payment/square.php.

Maybe I should have said warning.

I simply wanted to know if the replacement code was correct in order to stop the warning, which stops the customer from texting, which stops me from having to spend time on a warning, which causes me to...............................

[carlwhat]

again, where is the log file?

the process that you just talked about, would be a void. there is a _doVoid function in the square payment class.

my guess is that you are trying to refund a transaction that has not been settled and your only option is to void it. but the square log file would tell you that.

now why the square class allows you to refund this transaction is a different story. but i'm guessing that is where the problem is.

[dbltoe]

Used a tester that says the code is correct, so I'll just take my mole hill on down the road.

[carlwhat]

Are we starting to channel RodG or are you still upset about RixStix not including you in the "Good Guy List"?

PHP states that ANY use of count() with 7.2 will generate the warning listed. At https://www.php.net/manual/en/function.count.php you will need to scroll down to Change Log to see their statement.

I did not say there was any error in operation of the cart. Maybe I should have said warning.

I simply wanted to know if the replacement code was correct in order to stop the warning, which stops the customer from texting, which stops me from having to spend time on a warning, which causes me to...............................

last time, and i'm really done....

i am not at all upset. at all... the only thing that makes me upset is a comparison to rodG....

i'm just trying to help with the code. that is all....

WARNINGS will eventually turn to ERRORS which is a good reason to address them now; prior to a newer version of php which will say they are errors.

if the refund succeeded without problem, i do not understand the code. my concern is to help with the code.... and to not be compared to.... others.... that is all...

your code will address that problem, but if the result of return errors is NOT either an EMPTY array or a filled ARRAY (lines 1100 or 1104) the code needs to be fixed. IMO.

now get back out there and #stayathome.

best.

[simon1066]

New v1.1 of this mod just released

https://www.zen-cart.com/downloads.php?do=file&id=156

[dbltoe]

Couple of things not addressed or gone over a little lightly in either readme

This addresses UPGRADES

Up until just a couple of days ago, Square defaulted to sandbox when you entered your app. You could really get lost if you did not see that. Our current pages from Square show that they have defaulted to Production which should help. Just think the lack of sandbox for ZC should be more than line 129 in the MD

Couldn't find information in either readme regarding the changing of the API version for existing apps (some have not updated since their original creation. The latest version is 2020-03-25. The files with this new ZC version imply that this is the version needed for 1.1 to work. Version 1.0 API files listed 2020-02-26 yet nothing in the docs mentioned whether the user had to upgrade the API in square. We had several customers who were having problems that wound up being old API versions. It appears that 1.0 needed 2020-02-26. Nothing in the docs would have notified the user to change from an older version.

Since the API setting is "below the fold" on the app's landing page, it looks like most may have missed this.

Also, since the 2020-03-25 marks the start of Square's V2 of the API, I fear that only the new installs will work as most of the old installs will have an outdated API version.


BOTTOM LINE
If you are upgrading from any version prior to 1.1:

1. (Login to https://squareup.com/dashboard/apps and click on My Apps.
2. Click the Manage App button next to your listed App.
3. Make sure you are in Production as Sandbox is not used with Zen Cart
4. Scroll down to the API Version settings:
5. For version 1.1 of this mod, make sure your Production API Version is at least version 2020-03-25 or later.
6. .................

[DrByte]

@dbltoe - thanks for posting this.

I would have expected it to automatically use the right API version, since the SDK (which this module uses since v1.0) passes the API version it's written for, so should override what's set in the merchant account. (This is the behavior that other systems use, hence the assumption.)