Ceon Manual Card Payment Module v4

[Mattie C]

I upgraded to Zen Cart v1.5.6a (from v1.5.5f) and also upgraded my PHP to 7.2.19 (from 5.5.35).
The plugin still seems to work fine but I am getting an error in the logs now.

[06-Jun-2019 14:40:40 America/Los_Angeles] Request URI: /admin/orders.php?oID=170&action=edit, IP address: ***.***.**.***
#1 sizeof() called at [/home/server/public_html/includes/modules/payment/ceon_manual_card.php:2478]
#2 ceon_manual_card->_checkForMissingCatalogFiles() called at [/home/server/public_html/includes/modules/payment/ceon_manual_card.php:2153]
#3 ceon_manual_card->_checkModuleEnvironment() called at [/home/server/public_html/includes/modules/payment/ceon_manual_card.php:312]
#4 ceon_manual_card->ceon_manual_card() called at [/home/server/public_html/admin/orders.php:420]
--> PHP Warning: sizeof(): Parameter must be an array or an object that implements Countable in /home/server/public_html/includes/modules/payment/ceon_manual_card.php on line 2478.

[delia]

That sounds like a php problem in that this mod hasn't been updated and may not work on higher versions of php. I'm about to upgrade a site to 1.5.5f that uses it too.

[dbltoe]

Conor's submission was a helpful one in its time. Credit card processing was a LOT different and e-commerce was getting a double hit as merchant account provider demanded different accounts for on line and brick-n-mortar.

A lot of us had to resort to Conor's mod and calling in our numbers. Later, we were able to connect with a browser but, we still paid that extra percentage for "card not present" transactions.

Conor himself acknowledged that this method was a work around and less than secure for all concerned. Its process is not only NOT PCI compliant but, it is unsafe for both the customer and the store owner.

With the advent of PayPal, we had a viable option that was more secure. Yes, some folks despise PayPal but, there are many out there in e-commerce that don't feel comfortable using anything else. PayPal does not force folks to make an account as some will say. If the customer has a PayPal account, the system will ask them to login. Otherwise, they are immediately provided the opportunity to use their own card.

Next came Square. Another one with all sorts of rumors floating around. "Can't get my money" seems to be the big complaint but, I've never had that happen. Square has only given me one charge back (stolen card) and I've been with them from the start. Oddly enough, it was the Square business model that Conor mentioned often in our emails. Even though Square was not around at the time of Conor's passing, I bet he would have done something like it within a year.

Unlike other merchant accounts, Square has no sign-up or monthly fees. If you manage to get the right rep on the phone, they might give you a better rate if you mention Zen Cart.

Sorry for the length but, I felt it needed to explain that this is old and, potentially, VERY expensive/damaging software. Expensive that "call in" fees are always more than keyed in AND the risk to your business and the customer through non-compliant processing. Damaging in the potential loss of customer data.

Unless you have one of the prohibited businesses listed at http://tinyurl.com/y5ksco5b, I'd rethink PayPal and Square before using Conor's mod.

[delia]

I am looking at square now for an alternative for my customers. Since the customers using the manual card are doing a card not present transaction, they may be paying high rates now. Square is 2.75% - about what I pay for many of my transactions since I'm a business to business user. Business/corporate cards engender higher processing fees. Having zen include square in the default makes the switch a sounder decision now.

[swguy]

--> PHP Warning: sizeof(): Parameter must be an array or an object that implements Countable in /home/server/public_html/includes/modules/payment/ceon_manual_card.php on line 2478.

On line 2478 change

return sizeof($missing_store_files > 0);

to

return (sizeof($missing_store_files) > 0);

[Mattie C]

On line 2478 change

return sizeof($missing_store_files > 0);

to

return (sizeof($missing_store_files) > 0);

Thanks swguy,
That fixed the previous error but now it generates a new one.
I also upgraded to Zen Cart v1.5.6c this morning if that effects anything. The error only happens using PHP 7.2.19 not when I roll back to PHP 7.1.30

[18-Jul-2019 13:06:28 America/Los_Angeles] Request URI: /index.php?main_page=checkout_shipping, IP address: **.***.***.***
#1 ceon_manual_cardSessionCleaner->update() called at [/home/server/public_html/includes/classes/class.base.php:103]
#2 base->notify() called at [/home/server/public_html/includes/modules/pages/checkout_shipping/header_php.php:12]
#3 require(/home/server/public_html/includes/modules/pages/checkout_shipping/header_php.php) called at [/home/server/public_html/index.php:36]
--> PHP Warning: Use of undefined constant NOTIFY_HEADER_START_CHECKOUT_SUCCESS - assumed 'NOTIFY_HEADER_START_CHECKOUT_SUCCESS' (this will throw an Error in a future version of PHP) in /home/server/public_html/includes/classes/observers/class.ceon_manual_cardSessionCleaner.php on line 50.

[swguy]

Change includes/classes/observers/class.ceon_manual_cardSessionCleaner.php line 50 to

if ($notifier == 'NOTIFY_HEADER_START_CHECKOUT_SUCCESS' ||

(quotes around notifier name)

[Mattie C]

Change includes/classes/observers/class.ceon_manual_cardSessionCleaner.php line 50 to

if ($notifier == 'NOTIFY_HEADER_START_CHECKOUT_SUCCESS' ||

(quotes around notifier name)

Works great using Zen Cart 1.5.6c running PHP Version 7.2.20 now.
Thanks for your help swguy!