diff options
author | einhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46> | 2007-10-11 06:11:07 +0000 |
---|---|---|
committer | einhverfr <einhverfr@4979c152-3d1c-0410-bac9-87ea11338e46> | 2007-10-11 06:11:07 +0000 |
commit | daee124a79fdece2d3044cc9147a152ff725c1a0 (patch) | |
tree | 44f4da86a4250d2435c96820c7acd1010a44e13f /LedgerSMB/IS.pm | |
parent | 46ef05c89fadbf9e3e5c8ce5df806a97fdd565bb (diff) |
Fixed bug 1765161, post button duplicates invoices.
Adding a minor fixes to COGS edge cases for reversed invoices.
git-svn-id: https://ledger-smb.svn.sourceforge.net/svnroot/ledger-smb/trunk@1740 4979c152-3d1c-0410-bac9-87ea11338e46
Diffstat (limited to 'LedgerSMB/IS.pm')
-rw-r--r-- | LedgerSMB/IS.pm | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/LedgerSMB/IS.pm b/LedgerSMB/IS.pm index d494eb48..b3b54fca 100644 --- a/LedgerSMB/IS.pm +++ b/LedgerSMB/IS.pm @@ -1611,6 +1611,11 @@ sub cogs { # If there are unallocated items for the current invoice at the end, we # will throw an error until we have an understanding of other workflows # that need to be supported. -- CT + # + # Note: Victor's original patch selected items to reverse based on + # sell price. This causes issues with restocking fees and the like so + # I am removing that restriction. This should be discussed more fully + # however. -- CT $query = qq| SELECT i.id, i.qty, i.allocated, a.transdate, -1 * (i.allocated + i.qty) AS available, @@ -1619,11 +1624,10 @@ sub cogs { JOIN parts p ON (p.id = i.parts_id) JOIN ar a ON (a.id = i.trans_id) WHERE i.parts_id = ? AND (i.qty + i.allocated) > 0 - AND i.sellprice = ? ORDER BY transdate |; $sth = $dbh->prepare($query); - $sth->execute($id, $sellprice) || $form->dberror($query); + $sth->execute($id) || $form->dberror($query); my $qty; while ( my $ref = $sth->fetchrow_hashref(NAME_lc) ) { $form->db_parse_numeric(sth=>$sth, hashref => $ref); @@ -1662,7 +1666,7 @@ sub cogs { FROM invoice i JOIN parts p ON (i.parts_id = p.id) JOIN ap a ON (i.trans_id = a.id) - WHERE (i.allocated + i.qty) < 0 + WHERE allocated > 0 AND i.parts_id = ? ORDER BY a.transdate DESC, a.id DESC |; |