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

[swguy]

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.

[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';

[carlwhat]

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?

[swguy]

The code you have duplicates the sitename. It yields

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

[carlwhat]

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...-not-supported

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

best.

[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.

[carlwhat]

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.

[carlwhat]

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.

[swguy]

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".

[carlwhat]

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.

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.