search

ccavanaugh / jgnash

jGnash Personal Finance
http://ccavanaugh.github.io/jgnash/
Other
139 stars 80 forks source link

Fix sign when aggregating different account types #10

Closed t-pa closed 8 years ago

t-pa commented 8 years ago

Income accounts in budgets work with a reversed sign, but this was not considered in the aggregation of child accounts.

The recurring check account.getAccountType() == AccountType.INCOME should probably be factored out into its own method, but I was not sure where to put it. It is used in BudgetResultsModel.java and BudgetFactory.java.

t-pa commented 8 years ago

... and two changes to the AbstractExpandingTableModel fixing a deadlock and a race condition.