Bug 787813 - Price change from editing a transaction not reflected...

in pricedb. Allow PRICE_SOURCE_SPLIT_REG to overwrite PRICE_SOURCE_XFER_DLG_VAL.
Gnucash · Jun 10, 2021 · 80d6749 · 80d6749
1 parent 9546c05
commit 80d6749
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/gnucash/register/ledger-core/split-register.c b/gnucash/register/ledger-core/split-register.c
@@ -2223,7 +2223,6 @@ gnc_split_register_auto_calc (SplitRegister* reg, Split* split)
     Account* account;
     int denom;
     int choice;
-    PriceSource source = PRICE_SOURCE_USER_PRICE;
 
     if (STOCK_REGISTER    != reg->type &&
         CURRENCY_REGISTER != reg->type &&
@@ -2372,7 +2371,6 @@ gnc_split_register_auto_calc (SplitRegister* reg, Split* split)
     {
         recalculate_price (split, reg, value, amount);
         price_changed = TRUE;
-        source = PRICE_SOURCE_SPLIT_REG;
     }
     if (recalc_value)
         recalculate_value (split, reg, price, amount, shares_changed);

diff --git a/libgnucash/engine/Transaction.c b/libgnucash/engine/Transaction.c
@@ -3009,14 +3009,17 @@ record_price (Split *split,
 
     if (price)
     {
+        PriceSource oldsource = gnc_price_get_source (price);
         price_value = gnc_price_get_value (price);
         if (gnc_numeric_equal (swap ? gnc_numeric_invert (value) : value,
                                price_value))
         {
             gnc_price_unref (price);
             return;
         }
-        if (gnc_price_get_source (price) < source)
+        if (oldsource < source &&
+            !(oldsource == PRICE_SOURCE_XFER_DLG_VAL &&
+             source == PRICE_SOURCE_SPLIT_REG))
         {
             /* Existing price is preferred over this one. */
             gnc_price_unref (price);