• ericta
  • Senior Member
  • Offline
  • From: USA
  • Registered: 04/06/2011
  • Posts: 82

Topic: void a transaction error

Hi,

We just upgraded FA from 2.3.14 to 2.3.19 three days ago.
When we try to void a customer payment today we got following error

https://drive.google.com/file/d/0B2hEL1wolzPkSVcwVFZqT2tGRkE/edit?usp=sharing

Can anyone help?
Thank you
Eric

  • From: Singapore, India, Chennai
  • Registered: 05/02/2009
  • Posts: 6,150

Re: void a transaction error

The function get_bank_trans() in lines 66 to 91 in the file gl/includes/db/gl_db_bank_trans.inc has the stated SQL construct.

It is possible that some data may have not had the desired effect on being db_escaped.

Lines 77-79:

                 LEFT JOIN ".TB_PREF."suppliers supplier ON supplier.supplier_id = st.supplier_id,
             ".TB_PREF."bank_accounts act
        WHERE act.id=bt.bank_act ";

may be changed to:

                 LEFT JOIN ".TB_PREF."suppliers supplier ON supplier.supplier_id = st.supplier_id
                     LEFT JOIN ".TB_PREF."bank_accounts act ON act.id=bt.bank_act 
        WHERE 1 ";

Try and let us know if this solves your issue. It may be a better construct anyway.

apmuthu's Website

  • ericta
  • Senior Member
  • Offline
  • From: USA
  • Registered: 04/06/2011
  • Posts: 82

Re: void a transaction error

Hi apmuthu,

Thank you for your quick response.
I replaced the code, but same error exists.

Have a great holiday everyone.
Eric

4 Reply by apmuthu (edited by apmuthu 12/26/2013 04:07:28 pm)

  • From: Singapore, India, Chennai
  • Registered: 05/02/2009
  • Posts: 6,150

Re: void a transaction error

Why don't you try the latest codebase from the mercurial repo or just use the difference file set to overwrite the v2.3.19 files?

Attached is the changed fileset after v2.3.19 was released.

Only one sql command needs to be executed on the existing db (change 0_ to whatever your company # is):

ALTER TABLE `0_debtor_trans` ADD INDEX `Order` (`order_`);
Post's attachments

FA_Fixes_post_v2.3.19.zip 135.5 kb, 1 downloads since 2013-12-26 

You don't have the permssions to download the attachments of this post.

apmuthu's Website

  • ericta
  • Senior Member
  • Offline
  • From: USA
  • Registered: 04/06/2011
  • Posts: 82

Re: void a transaction error

I'll give a try.

Thank you very much.
Eric

apmuthu wrote:

Why don't you try the latest codebase from the mercurial repo or just use the difference file set to overwrite the v2.3.19 files?

Attached is the changed fileset after v2.3.19 was released.

Only one sql command needs to be executed on the existing db (change 0_ to whatever your company # is):

ALTER TABLE `0_debtor_trans` ADD INDEX `Order` (`order_`);