FrontAccounting forum — update journal entry has query issue

Re: update journal entry has query issue

2018-03-29T13:40:22Z

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

2018-03-29T05:44:30Z

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

2018-03-29T02:40:17Z

@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

2018-03-28T17:20:50Z

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

2018-03-27T17:03:52Z

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

2018-03-27T16:47:04Z

@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

2018-03-27T16:19:25Z

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

2018-03-27T15:26:38Z

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

2018-03-27T15:07:06Z

@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

2018-03-27T13:45:38Z

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');
}