Corrected displayed money value in the split editor.

Commit c1a40f3 changed the way the money amount is displayed in the split editor. If you entered a decimal value in a split transaction, saved and then edited the split again, the values now seemed to be rounded and you would have to enter the correct values again to make changes. -> Deleted toShortString method in Money and changed it to toLocaleString in AccountType.displayBalance(). codinguser#876
Thumas · Aug 7, 2020 · 6e98dfa · 6e98dfa
1 parent 163b9d2
commit 6e98dfa
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 11 deletions.
diff --git a/app/src/main/java/org/gnucash/android/model/AccountType.java b/app/src/main/java/org/gnucash/android/model/AccountType.java
@@ -154,8 +154,8 @@ public void displayBalance(final TextView balanceTextView,
                 // Display value without currency and without decimals
                 balanceTextView.setText(!shallDisplayNegativeSignum
                                         ? balanceToDisplay.abs()
-                                                          .toShortString()
-                                        : balanceToDisplay.toShortString());
+                                                          .toLocaleString()
+                                        : balanceToDisplay.toLocaleString());
 
             } else {
                 // Shall not display balance

diff --git a/app/src/main/java/org/gnucash/android/model/Money.java b/app/src/main/java/org/gnucash/android/model/Money.java
@@ -448,15 +448,6 @@ public String toPlainString(){
 		return mAmount.setScale(mCommodity.getSmallestFractionDigits(), ROUNDING_MODE).toPlainString();
 	}
 
-	/**
-	 * Returns a locale-specific representation of the amount of the Money object (excluding the currency)
-	 *
-	 * @return String representation of the amount (without currency) of the Money object
-	 */
-	public String toShortString(){
-		return String.format(Locale.getDefault(), "%.0f", asDouble());
-	}
-
 	/**
 	 * Returns a locale-specific representation of the amount of the Money object (excluding the currency)
 	 *