tandem
/
numerus
2
0
Fork 0
Code Issues 39 Pull Requests Packages Projects 1 Releases 1 Wiki Activity

Avoid panic error when there is no expense to compute the sum of

This commit is contained in:
jordi fita mas 2023-11-06 13:18:02 +01:00
parent ef215f1e6e
commit 4e831d94db
1 changed files with 5 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
pkg/expenses.go
View File

@ -158,8 +158,9 @@ func (page *expensesIndexPage) mustComputeExpensesTotalAmount(ctx context.Contex
join currency using (currency_code) join currency using (currency_code)
group by decimal_digits group by decimal_digits
`, where), args...) `, where), args...)
if err := row.Scan(&page.SumAmount, &page.SumTotal); err != nil { if notFoundErrorOrPanic(row.Scan(&page.SumAmount, &page.SumTotal)) {
panic(err) page.SumAmount = "0.0"
page.SumTotal = "0.0"
} }
row = conn.QueryRow(ctx, fmt.Sprintf(` row = conn.QueryRow(ctx, fmt.Sprintf(`
@ -180,8 +181,8 @@ func (page *expensesIndexPage) mustComputeExpensesTotalAmount(ctx context.Contex
group by decimal_digits group by decimal_digits
`, where), args...) `, where), args...)
var taxes [][]string var taxes [][]string
if err := row.Scan(&taxes); err != nil { if notFoundErrorOrPanic(row.Scan(&taxes)) {
panic(err) // well, nothing to do
} }
page.SumTaxes = make(map[string]string) page.SumTaxes = make(map[string]string)
for _, tax := range taxes { for _, tax := range taxes {