Updated Braintree App

[frankl]

# Braintree App for osCommerce Online Merchant v3.x

## Changelog

**v3.000 01-Aug-2021**

* Remove all TABLE_ FILENAME_ references and hardcode tables and filenames

* Replace $HTTP_GET_VARS and $HTTP_POST_VARS with $_GET and $_POST

* Update classes to be compatible with PHP 7.3.0

* Update Braintree PHP Client to v6.3.0.

* Update SSL test to go to https://www.howsmyssl.com/a/check

* Checking for updates now goes to my website rather than the osCommerce website

Now in the apps section

[Mort-lemur]

Hi is this app still supported?

Having a few issues....

Thanks

[frankl]

Not using osC any more so not really. Sorry.

[Mort-lemur]

 

8 hours ago, frankl said:

Not using osC any more so not really. Sorry.

Ok thank you

[Mort-lemur]

This is puzzling me - I have tried the installation 3 times now with the same result ie white screen on the orders.php page and under modules - payment - add modules it only lists 2 modules, not the usual 20 or so.

Also when clicking on either of the two options in the new Admin Braintree box again it is white screen.

There are no errors coming up in the error log

When I remove the files the admin again works as it should.

Site is a 2.3.4 build

Anyone have this mod up and running?

Thank You

[greasemonkey]

@frankl Not sure Frank is maintaining this so please ignore should you wish.

There is an error in includes/modules/payment/braintree_cc.php around lines 95

      if ( ($this->enabled == true) && ((int)OSCOM_APP_PAYPAL_DP_ZONE > 0) ) {
        $check_flag = false;
        $check_query = tep_db_query("select zone_id from " . TABLE_ZONES_TO_GEO_ZONES . " where geo_zone_id = '" . OSCOM_APP_PAYPAL_DP_ZONE . "' and zone_country_id = '" . $order->delivery['country']['id'] . "' order by zone_id");

Are referring to the wrong wrong zone. It should be

      if ( ($this->enabled == true) && ((int)OSCOM_APP_PAYPAL_BRAINTREE_DP_ZONE > 0) ) {
        $check_flag = false;
        $check_query = tep_db_query("select zone_id from " . TABLE_ZONES_TO_GEO_ZONES . " where geo_zone_id = '" . OSCOM_APP_PAYPAL_BRAINTREE_DP_ZONE . "' and zone_country_id = '" . $order->delivery['country']['id'] . "' order by zone_id");

 

Edited March 28, 2023 by greasemonkey

[greasemonkey]

On 3/28/2023 at 12:20 PM, greasemonkey said:

@frankl Not sure Frank is maintaining this so please ignore should you wish.

There is an error in includes/modules/payment/braintree_cc.php around lines 95

      if ( ($this->enabled == true) && ((int)OSCOM_APP_PAYPAL_DP_ZONE > 0) ) {
        $check_flag = false;
        $check_query = tep_db_query("select zone_id from " . TABLE_ZONES_TO_GEO_ZONES . " where geo_zone_id = '" . OSCOM_APP_PAYPAL_DP_ZONE . "' and zone_country_id = '" . $order->delivery['country']['id'] . "' order by zone_id");

Are referring to the wrong wrong zone. It should be

 

Sorry - the above post correction was not correct. The correct change is

      if ( ($this->enabled == true) && ((int)OSCOM_APP_PAYPAL_BRAINTREE_CC_ZONE > 0) ) {
        $check_flag = false;
        $check_query = tep_db_query("select zone_id from " . TABLE_ZONES_TO_GEO_ZONES . " where geo_zone_id = '" . OSCOM_APP_PAYPAL_BRAINTREE_CC_ZONE . "' and zone_country_id = '" . $order->delivery['country']['id'] . "' order by zone_id");