Jump to content
Latest News: (loading..)
Harald Ponce de Leon

PayPal App for osCommerce Online Merchant

Recommended Posts

6 hours ago, lucsangel said:

Hi!

To correct it you simply need quotes around the offending constant in the original line where (int)OSCOM_APP_PAYPAL_TRANSACTIONS_ORDER_STATUS_ID is, like this

line 37:

      $ppstatus_query = tep_db_query("select comments from " . TABLE_ORDERS_STATUS_HISTORY . " where orders_id = '" . (int)$oID . "' and orders_status_id = '" . (int)'OSCOM_APP_PAYPAL_TRANSACTIONS_ORDER_STATUS_ID' . "' and comments like 'Transaction ID:%' order by date_added desc limit 1");

 

Note the 'OSCOM_APP_PAYPAL_TRANSACTIONS_ORDER_STATUS_ID' quotes.

That way the module will still work and you won't have to think about anymore - it is functional should you one day want Paypal but forgot what you did to make the warning disappear.

Hope this helps.

 

Bobbee
 

sorry Bobbee this is wrong and won't help anyone who uses it.

If you do this then it will always use a value of 0 even when the setting is set. So you will get rid of the warning but you won't be able to see any paypal orders in admin because there's no matching status value. If you put quotes round it then it's a string and not a constant (defined or otherwise).

The post you quoted was correct.


For a new install or if your store isn't mobile-friendly, get the community-supported responsive osCommerce (2.3.4.1 CE) here: https://github.com/gburton/Responsive-osCommerce/archive/2341-Frozen.zip

Working on generalising bespoke solutions for Quickbooks integration, Easify integration and pay4later (DEKO) integration at 2.3.x

Share this post


Link to post
Share on other sites

Recently I'm seeing a lot of 0.0.0.0 in the PayPal App log for customers ip address. Doesn't seem to be causing any payment problems but I'm curious what changed. Anyone else seeing this?


The water in a vessel is sparkling; the water in the sea is dark. The small truth has words which are clear; the great truth has great silence.

- Rabindranath Tagore

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×