IntenseFinance/gnucash
3
0
Fork 0
mirror of https://github.com/Gnucash/gnucash.git synced 2025-02-20 11:48:30 -06:00
Code Issues Packages Projects Releases Wiki Activity

[stock-txn-asst] Fix sell-side stock amounts.

Browse Source 
It was setting a negative amount in one too many places.
This commit is contained in:
John Ralls 2023-08-10 16:58:49 -07:00
parent 502dcc03ce
commit 917b4b247c
1 changed files with 2 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
gnucash/gnome/assistant-stock-transaction.cpp
View File

@ -686,12 +686,7 @@ StockTransactionStockEntry::set_amount(gnc_numeric amount, Logger& logger)
return; return;
} }
bool neg{gnc_numeric_negative_p(amount) == TRUE}; m_amount = amount;
if ((m_debit_side && !neg) || (!m_debit_side && neg))
m_amount = amount;
else
m_amount = gnc_numeric_neg(amount);
PINFO("%s set amount %s", m_memo, print_amount(amount)); PINFO("%s set amount %s", m_memo, print_amount(amount));
} }
@ -704,7 +699,7 @@ StockTransactionStockEntry::calculate_price(bool new_balance)
gnc_numeric_zero_p(m_amount) || gnc_numeric_zero_p(m_value)) gnc_numeric_zero_p(m_amount) || gnc_numeric_zero_p(m_value))
return gnc_numeric_error(GNC_ERROR_ARG); return gnc_numeric_error(GNC_ERROR_ARG);
return gnc_numeric_div(m_value, m_amount, return gnc_numeric_div(m_debit_side ? m_value : gnc_numeric_neg(m_value), m_amount,
GNC_DENOM_AUTO, GNC_HOW_DENOM_EXACT); GNC_DENOM_AUTO, GNC_HOW_DENOM_EXACT);
} }