FrontAccounting forum — Bug in gl/includes/db/gl_journal.inc

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-11T16:06:33Z

Actually the old query INSERT INTO table1 SELECT col.. FROM table2 is a right syntax.
But better to keep the new query because if the subquery SELECT return null the system will raise an error meanwhile the old query can run successfully even when the SELECT subquery returns null

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-08T15:46:24Z

Yes, it is amazing that this bug is first detected now after so many minor releases.

Maybe because the entrance is mostly done by the Sales routines.

Joe

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-08T12:07:36Z

Thanks. U have checked this bug has persisted since at least 2.4.2 and it is quite critical because data entry via journal entry feature that relate to account receivable will write the cust_trans using this method and it will fail because the query is wrong.

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-08T10:34:21Z

This has been fixed and committed to stable repo. Thanks @barbarian for finding this.

A fixed copy can be downloaded and replaced.

/Joe

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-08T07:07:58Z

@joe,
Can you update the code and commit it to stable version downloads.

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-07T23:18:00Z

Instead of

$sql = "INSERT INTO ".TB_PREF."debtor_trans (trans_no, type, debtor_no, branch_code, tran_date, reference, ov_amount, rate)
        SELECT "
        .db_escape($trans_no).", ".db_escape($trans_type).", debtor_no,"
        .db_escape($branch_id).",'".date2sql($date)."', ".db_escape($ref).","
        .db_escape($amount).", ".db_escape($rate)."
        FROM ".TB_PREF."cust_branch WHERE branch_code=".db_escape($branch_id);

It should be

$sql = "INSERT INTO ".TB_PREF."debtor_trans (trans_no, type, debtor_no, branch_code, tran_date, reference, ov_amount, rate)
        VALUES ("
        .db_escape($trans_no).", ".db_escape($trans_type).", (SELECT debtor_no FROM ".TB_PREF."cust_branch WHERE branch_code=".db_escape($branch_id). ") ,"
        .db_escape($branch_id).",'".date2sql($date)."', ".db_escape($ref).","
        .db_escape($amount).", ".db_escape($rate).")";

The sub query SELECT debtor_no ... result should bebecome the debtor_no column when inserting.

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-07T18:08:08Z

barbarian wrote:

I found this incorrect sql query:
Screenshot

The SQL Query is wrong. INSERT INTO .... SELECT FROM ...

Re: Bug in gl/includes/db/gl_journal.inc

2021-08-07T14:58:42Z

Can you elaborate the issue in text?

Bug in gl/includes/db/gl_journal.inc

2021-08-07T14:22:01Z

I found this incorrect sql query:

Screenshot