Authorize.net API with Card on File transactions support thread...

carlwhat:

i am working on a new release of this most excellent, IMO (inherently biased), payment module. it should address all bugs documented by you and verified by me. including the verified bugs associated with guest checkout.

thanks for pointing them out.

best.

Glad to hear that! I'll get it installed and tested as soon as it's available.

v 2.2.0 now submitted for review.

many bug fixes as well as now having the ability to have multiple authorizations per order! all configurable.

problems associated with guest checkout should now be addressed.

best.

v 2.2.0 now available.

carlwhat:

v 2.2.0 now available.

Thank you! I'll report back on any issues.

Here's one typo to fix. Line 32 from \includes\languages\english\card_update.php

define('CARD_UPDATE_OK', 'Succesfully updated information.');
correct to:
define('CARD_UPDATE_OK', '**Successfully **updated information.');

v 2.2.1 is now available.

in addition to having multiple authorizations per order, admins can use any customer card on file for additional authorizations.

i have addressed all reported and verified bugs. including new ones found by me!

best.

Possibly useful change to the docs/index.html file: it currently says:

Go to Admin -> Modules -> Payment and you will see 2 new payment modules
PLEASE PLEASE PLEASE, even if you already have an authorize.net account, create a new one using the link here.

I assumed here was a link so I tried clicking it, and then tried scanning the page to see where the link was.
Maybe instead:

Go to Admin -> Modules -> Payment and you will see 2 new payment modules.
Start by clicking on New Credit Card Authorize.net (CIM). You will see a link under the Install Module button that says
Click Here to Sign Up for a NEW Merchant Account.
PLEASE PLEASE PLEASE, even if you already have an authorize.net account, create a new one using this link.

... or something like that.

swguy:

Possibly useful change to the docs/index.html file: it currently says:

Go to Admin -> Modules -> Payment and you will see 2 new payment modules
PLEASE PLEASE PLEASE, even if you already have an authorize.net account, create a new one using the link here.

I assumed here was a link so I tried clicking it, and then tried scanning the page to see where the link was.
Maybe instead:

Go to Admin -> Modules -> Payment and you will see 2 new payment modules.
Start by clicking on New Credit Card Authorize.net (CIM). You will see a link under the Install Module button that says
Click Here to Sign Up for a NEW Merchant Account.
PLEASE PLEASE PLEASE, even if you already have an authorize.net account, create a new one using this link.

... or something like that.

i will take a look. thanks for the input.

best.

v 2.2.2 is now available.

the key fix on this version, is the order status only gets changed after capture of the funds. if you have the authorization type set to authorization+capture, you will not notice any difference. if, on the other hand, you are set to authorize only; the order status will only get updated after capture of the funds on the admin side.

there are a few other bug fixes/improvements made behind the scenes.

best.

I'd like the Keep Card on File checkbox to default to unchecked. I would have thought changing the call to zen_draw_checkbox_field in includes/modules/payment/authorizenet_cim.php line 200 would do it, but it seems stuck on checked. Any thoughts?

swguy:

I'd like the Keep Card on File checkbox to default to unchecked. I would have thought changing the call to zen_draw_checkbox_field in includes/modules/payment/authorizenet_cim.php line 200 would do it, but it seems stuck on checked. Any thoughts?

hi @swguy,
yeah, that's what i would think as well. that code looks wrong there. not sure how it got that way.... have you tried:

if (!zen_in_guest_checkout()) {
	            $selection['fields'][] = array(
		            'title' => 'Keep Card on File',
		            'field' => zen_draw_checkbox_field('authorizenet_cim_save', '', false),
	//		            '' . ' id="' . $this->code . '-save"' . $onFocus),
		            'tag' => $this->code . '-save'
	            );
            }

that seems to work for me. i tested on a base157 install as well as on a customized 157a site.

it does not look like that other stuff is doing whatever it was i intended it to do there.

i will fix that in the next release. thanks for pointing it out!

best.

LOL I got suckered in by the incorrect bracket placement! That's why just switching to false didn't work. Thanks for taking a second look.

Small bug in admin/includes/classes/authnet_order.php line 28.

You have:
include DIR_FS_CATALOG . DIR_WS_ADMIN . DIR_WS_LANGUAGES . $_SESSION['language'] . '/authnet_order.php';

You want:

     include DIR_FS_ADMIN . DIR_WS_LANGUAGES . $_SESSION['language'] . '/authnet_order.php';

swguy:

Small bug in admin/includes/classes/authnet_order.php line 28.

You have:
include DIR_FS_CATALOG . DIR_WS_ADMIN . DIR_WS_LANGUAGES . $_SESSION['language'] . '/authnet_order.php';

You want:

     include DIR_FS_ADMIN . DIR_WS_LANGUAGES . $_SESSION['language'] . '/authnet_order.php';

hey scott,
happy new year!

thanks for taking a look.

are you referring to the fact that the code as implemented has an extra slash '/' in there?

The code you have duplicates the sitename. It yields

/Users/scott/Sites/mysite//mysite/admin/includes/languages/english/authnet_order.php

swguy:

The code you have duplicates the sitename. It yields

/Users/scott/Sites/mysite//mysite/admin/includes/languages/english/authnet_order.php

i did the following:

<?php
	require 'includes/application_top.php';

	echo "-------->" . DIR_FS_CATALOG . DIR_WS_ADMIN . "<---------<br>";
	echo "-------->" . DIR_FS_ADMIN . "<---------<br>";

results in:

-------->/var/www/la155//mybloodyadmin/<---------
-------->/var/www/la155/mybloodyadmin/<---------

(names changed to protect the innocent...)

what server software are you running? i'm running:

HTTP Server: Apache/2.4.38 (Debian)
PHP Version: 7.4.13 (Zend: 3.4.0)

if you are running windows:

https://docs.zen-cart.com/user/first_steps/server_requirements/#windows-servers--iis-are-not-supported

if you are not running windows, by all means, i would love to explore further.

best.

I am in a subfolder of document root called demo_157a. Document root is /Users/scott/Sites/.

MacOS (BSD Unix) - Zen Cart 1.5.7a, PHP 7.2 under MAMP. (Just a test site.)

echo "-------->" . DIR_FS_CATALOG . DIR_WS_ADMIN . "<---------<br>";

gives

-------->/Users/scott/Sites/demo_157a//demo_157a/admin/<---------

DIR_WS_ADMIN starts with the subfolder when one is used.

swguy:

I am in a subfolder of document root called demo_157a. Document root is /Users/scott/Sites/.

MacOS (BSD Unix) - Zen Cart 1.5.7a, PHP 7.2 under MAMP. (Just a test site.)

echo "-------->" . DIR_FS_CATALOG . DIR_WS_ADMIN . "<---------<br>";

gives

-------->/Users/scott/Sites/demo_157a//demo_157a/admin/<---------

DIR_WS_ADMIN starts with the subfolder when one is used.

so the problem is when the store is installed in a subfolder?

great. i'll take a look and apply a fix as necessary. i am going to be doing an update shortly.

thanks.

**v2.2.3 **now available for download.

php 7.0 is now required for this version.

if your server is running less than php7.0, then please stay with 2.2.2.

addresses fixes documented in thread above. also continues code cleanup... addresses some issues on card deletion on the admin side as well as card update on the customer side.

best.

What causes the enabled flag in customers_cc table to be set to N?

Small suggested tweak: in
./includes/classes/observers/class.cim_admin_observer.php
you could use BUTTON_DELETE_CARDS
instead of the string "Delete Credit Cards".

swguy:

What causes the enabled flag in customers_cc table to be set to N?
@swguy, this code is an evolution over many years work for some clients. during the pandemic, i decided to refactor it (it desperately needed it), and make it available to the community. this flag was part of the evolution, and in this module's current form is not used.

swguy:

Small suggested tweak: in
./includes/classes/observers/class.cim_admin_observer.php
you could use BUTTON_DELETE_CARDS
instead of the string "Delete Credit Cards".

agreed. however that constant is not loaded at that point in the observer. i will look at where i am loading the payment modules and potentially move them so that the constant is available. little more complex, but i'm in agreement.

thanks.