FrontAccounting forum — update journal entry has query issue

Re: update journal entry has query issue

null@example.com (kvvaradha) — Thu, 29 Mar 2018 13:40:22 +0000

The journal voucher customization I did it for a company. There it used. It won't be applied to core feature

Re: update journal entry has query issue

null@example.com (apmuthu) — Thu, 29 Mar 2018 05:44:30 +0000

The function write_journal_entries() defined in gl/includes/db/gl_journal.inc is now used to edit journal entries by first voiding it and then adding the modified entry it as a new one.

In FA 2.3.x, the said function write_journal_entries() is defined in gl/includes/db/gl_db_trans.inc.

Re: update journal entry has query issue

null@example.com (apmuthu) — Thu, 29 Mar 2018 02:40:17 +0000

@kvvaradha: Provide your patch to include in the core. During such editing, does the reference number of the entry change?

Re: update journal entry has query issue

null@example.com (kvvaradha) — Wed, 28 Mar 2018 17:20:50 +0000

This is created to use in edit journal. But unfortunately it's not used , but I am using it for editing the journal vouchers.

Re: update journal entry has query issue

null@example.com (apmuthu) — Tue, 27 Mar 2018 17:03:52 +0000

This is a new file in FA 2.4.x. Looks like it was made to be linked into an Edit Journal button on journal entries listing.

Re: update journal entry has query issue

null@example.com (joe) — Tue, 27 Mar 2018 16:47:04 +0000

@notrinos.

You are right. This is the reason why no errors are reported from MySql.

This is a redundant function at the moment. This should be removed. Will discuss this with Janusz.

Joe

Re: update journal entry has query issue

null@example.com (notrinos) — Tue, 27 Mar 2018 16:19:25 +0000

No one has caught this problem because this function update_journal() has never been used in any where.
Don't know whether to be happy or sad

Re: update journal entry has query issue

null@example.com (joe) — Tue, 27 Mar 2018 15:26:38 +0000

This has been committed. Strange that this passed the MySql server without warnings!

Thanks for catching this.

/Joe

Re: update journal entry has query issue

null@example.com (apmuthu) — Tue, 27 Mar 2018 15:07:06 +0000

@kvvaradha: Good catch.
Wonder how it slipped thru for so long.

@joe: can commit it by removing the last closing bracket in the doc_date line.

update journal entry has query issue

null@example.com (kvvaradha) — Tue, 27 Mar 2018 13:45:38 +0000

Hello @joe,

This post is for the @joe and @apmuthu to update their repo.

gl/includes/db/gl_journal.inc

Here there is a function for update_journal

function update_journal($type, $trans_no, $amount, $tran_date, $currency, $reference, $source_ref='', $rate=1,
     $event_date='', $doc_date='')
{
    $sql = "UPDATE ".TB_PREF."journal SET "
      ."`amount`=".db_escape($amount).","
      ."`currency`=".db_escape($currency).","
      ."`rate`=".db_escape($rate).","
      ."`reference`=".db_escape($reference).","
      ."`source_ref`=".db_escape($source_ref).","
      ."`tran_date`="."'".date2sql($tran_date)."',"
      ."`event_date`="."'".date2sql($event_date)."',"
      ."`doc_date`="."'".date2sql($doc_date)."')  // here the '(' should be removed. 
       WHERE `type`=".db_escape($type)." AND " ."`trans_no`=".db_escape($trans_no);

    return db_query($sql, 'cannot update journal entry');
}