[Not a bug] table structure problem

[clydejones]

While working on one of my sites that doesn't use a database prefix, I ran across the following error:

occurs from
admin -> customers -> paypal IPN

after displaying the customer information page, if you click on the "orders" button
the following error displays

1054 Unknown column 'zen_order_id' in 'where clause'
in:
[select * from paypal where zen_order_id = '1251' order by paypal_ipn_id DESC LIMIT 1]


As I said the database doesn't use a prefix and the paypal table contains a row named order_id

I fixed the problem by deleting the "zen_" from several places in
includes/modules/payment/paypal.php and paypalwwp.php

Is it possible to fix this for future versions in cases where a database doesn't use a prefix.

perhaps something like:

$sql = "select * from " . TABLE_PAYPAL . " where . DB_PREFIX . "order_id = '" . (int)$zf_order_id . "' order by paypal_ipn_id DESC LIMIT 1";

[Kim]

Clyde have you applied the most recent patch for the PayPal files?

[Ajeh]

Check your paypal table that is a real field ...

[clydejones]

Check your paypal table that is a real field ...

on the site that uses no prefix the paypal table contains order_id

on other sites I work on Those with a zen_ prefix the paypal table contains zen_order_id.

[clydejones]

Clyde have you applied the most recent patch for the PayPal files?

Yep, done that also. last patch was 21 March.

I need to point out that I didn't set up the site initially, I was asked to fix several problem that cropped up afterward.

This was one of them.

[a_berezin]

on the site that uses no prefix the paypal table contains order_id

You are mistaken. You can see origin zc_install/sql/mysql_zencart.sql:

...
CREATE TABLE paypal (
  paypal_ipn_id int(11) unsigned NOT NULL auto_increment,
  zen_order_id int(11) unsigned NOT NULL default '0',
...

[clydejones]

You are mistaken. You can see origin zc_install/sql/mysql_zencart.sql:

...
CREATE TABLE paypal (
  paypal_ipn_id int(11) unsigned NOT NULL auto_increment,
  zen_order_id int(11) unsigned NOT NULL default '0',
...

Andrew,

No, I checked the database table and it is order_id.

As I said I didn't do the initial set up of the site I only got involved to troubleshoot several problems that cropped up after the installation.

I suppose I can fix the table.

[clydejones]

Andrew,

No, I checked the database table and it is order_id.

As I said I didn't do the initial set up of the site I only got involved to troubleshoot several problems that cropped up after the installation.

I suppose I can fix the table.

Resolved.

I renamed order_id to zen_order_id re_installed the most recent paypal patch


evidently the original installer thought that "zen_" was a prefix to match the database prefix. Since a prefix hadn't been used for the database it was just deleted from order_id.

Anyway everything working fine now.

Thanks All

[Ajeh]

Thanks for the update ...

I have done that in a hasty global replace forgetting about the couple three fields that start with zen_ on the paypal tables ... then wondered why the code stopped working ...