TheAxelander / OpenBudgeteer

OpenBudgeteer is a budgeting app based on the Bucket Budgeting Principle
https://theaxelander.github.io
MIT License
745 stars 42 forks source link

Account closing potentially possible with non-zero balance #247

Open TheAxelander opened 2 months ago

TheAxelander commented 2 months ago

This should be .Where(i => i.AccountId == id) otherwise it just checks for primary key of BankTransaction which should never find anything

https://github.com/TheAxelander/OpenBudgeteer/blob/025ec09a994fe75b859e7f353417542ba68b0a2c/OpenBudgeteer.Core.Data.Services/AccountService.cs#L60-L63