[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: reposting invoices



BTW you are correct on the patch you submitted :-)  I am applying the
patch now and putting through some final testing.

Is it OK to include your email in the CONTRIBUTORS file?

Best Wishes,
Chris Travers

On 7/7/07, Victor Sterpu <..hidden..> wrote:
I'm sure that a change at line 322 in IR.pm can't solve the bug 1748895.
To solve this bug a extra update is needed.
I added this : "$form->update_balance( $dbh, "invoice", "allocated",
qq|id = $invoice_id|, $qty  );" at line 393.

The problem is that when a ap invoice is posted and COGS are allocated
on ar invoices, the "allocated" field is not updated on the ap invoice.
And this way they can be reallocated.

Chris Travers wrote:
> On 7/7/07, Victor Sterpu <..hidden..> wrote:
>
>> It's true I reported a bug at COGS, but it was a diffrent one. :)
>> I also send the patch for that bug.
>>
>
> I am still evaluating what exactly is going on with that one.  I just
> want to understand the patch before I apply it (and this code isn't
> always the easiest code to understand what a change will do...   But
> that is why we are moving to the new architecture...).
>
>
>> As you suspected this bugs are also in sqlledger.
>>
>
> Almost certainly.
>
> It actually looks like this may be corrected by a a change to the SQL
> query starting at LedgerSMB/IR.pm: 322
>
>
>> Then I will fill in another bug report for what we discussed.
>> I will send a patch for the solution using GAAP(I prefer respecting GAAP
>> also).
>> And this solution is more simple. :)
>>
>> Chris Travers wrote:
>>
>>> On 7/7/07, Victor Sterpu <..hidden..> wrote:
>>>
>>>
>>>> You are right, my solution is not GAAP-compliant.
>>>>
>>>> I'll try then to define why COGS is wrong in my opinion.
>>>> 1) I create a new product.
>>>> 2) I insert a ap invoice witch contains a quantity of 1 from that
>>>> product at a price X.
>>>> 3) I insert a ap invoice witch contains a quantity of -1 from that
>>>> product at a price X(resulting price -X).
>>>> 4) I insert a ar invoice witch contains a quantity of 1 from that
>>>> product. This invoice should have no COGS, but it does. The COGS is X.
>>>> The problem is that the "allocated" field is not updated at step 3), for
>>>> the ap invoices. And this way the product will be allocated on the ar
>>>> invoice.
>>>>
>>>> I must find a solution as fast as posible to this problem, so I'm
>>>> willing to make the necesary modifications if you agree.
>>>>
>>>>
>>> You are of course welcome to make modifications to your system, and
>>> you can turn off transaction reversal enforcement.  This however, has
>>> known issues with COGS (particularly when prices are volitile) and so
>>> we are working on a more comprehensive solution.
>>>
>>> Also I suspect your problem is related to a COGS bug recently filed
>>> (you filed it, I assume)?  I expect to have that fixed shortly.  When
>>> I do, I will be sending you a hotfix and that may even address your
>>> issue.
>>>
>>> Also, if you are interested, the work on debit/credit invoices for 1.3
>>> may also address your issue.  We could work on this as well and it
>>> should be backward-compatible with 1.2 so backporting shouldn't be a
>>> problem as long as you are willing to make db schema changes.
>>>
>>>
>>>
>>>> Can you please detail about how you see the solution for this?
>>>> I don't know exactly what you mean here: "This means: that one
>>>> essentially creates a reversing invoice and then creates a new one
>>>> automatically."
>>>>
>>>> Following GAAP-compliant I see it like this:
>>>> 1) A ap invoice is posted with a negative value for a product.
>>>> 2) The post_invoice function automaticaly searches for unallocated
>>>> entries(qty!=allocated) in the "invoice" table for the same product, at
>>>> the same price. The "allocated" field is updated to match that 2 entryes.
>>>>
>>>>
>>> That is the idea of the debit/credit invoice structure, except that
>>> this allows one to display positive values for reversing invoices by
>>> setting a flag in the record.
>>>
>>> As I say, it might be a good idea to create a solution for this that
>>> could be an add-on patch for 1.2 and a part of the solution for 1.3
>>> :-)  I would be glad to help.
>>>
>>> Best Wishes,
>>> Chris Travers
>>>
>>> -------------------------------------------------------------------------
>>> This SF.net email is sponsored by DB2 Express
>>> Download DB2 Express C - the FREE version of DB2 express and take
>>> control of your XML. No limits. Just data. Click to get it now.
>>> http://sourceforge.net/powerbar/db2/
>>> _______________________________________________
>>> Ledger-smb-devel mailing list
>>> ..hidden..
>>> https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel
>>>
>>>
>>>
>> -------------------------------------------------------------------------
>> This SF.net email is sponsored by DB2 Express
>> Download DB2 Express C - the FREE version of DB2 express and take
>> control of your XML. No limits. Just data. Click to get it now.
>> http://sourceforge.net/powerbar/db2/
>> _______________________________________________
>> Ledger-smb-devel mailing list
>> ..hidden..
>> https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel
>>
>>
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by DB2 Express
> Download DB2 Express C - the FREE version of DB2 express and take
> control of your XML. No limits. Just data. Click to get it now.
> http://sourceforge.net/powerbar/db2/
> _______________________________________________
> Ledger-smb-devel mailing list
> ..hidden..
> https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel
>
>


-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Ledger-smb-devel mailing list
..hidden..
https://lists.sourceforge.net/lists/listinfo/ledger-smb-devel