luzifer/accounting
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix: allow updating transfer transactions

Browse source 
Signed-off-by: Knut Ahlers <knut@ahlers.me>
This commit is contained in:
Knut Ahlers 2024-02-01 12:00:00 +01:00
parent 0b525f72ec
commit ded3444391
Signed by: luzifer
SSH key fingerprint: SHA256:/xtE5lCgiRDQr8SLxHMS92ZBlACmATUmF1crK16Ks4E
2 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
pkg/database/database.go
View file

@ -457,6 +457,7 @@ func (c *Client) UpdateTransaction(txID uuid.UUID, tx Transaction) (err error) {
tx.ID = txID
tx.Account = oldTX.Account // Changing that would create chaos
tx.PairKey = oldTX.PairKey // Updating a paired tx should not decouple it
if err = tx.Validate(c); err != nil {
return fmt.Errorf("validating transaction: %w", err)

4
pkg/database/schema.go
View file

@ -82,6 +82,8 @@ func (b *BaseModel) BeforeCreate(*gorm.DB) (err error) {
}
// Validate executes some basic checks on the transaction
//
//nolint:gocyclo
func (t Transaction) Validate(c *Client) (err error) {
var errs []error
@ -110,7 +112,7 @@ func (t Transaction) Validate(c *Client) (err error) {
}
}
if acc.Type == AccountTypeBudget && !t.Category.Valid {
if acc.Type == AccountTypeBudget && !t.Category.Valid && !t.PairKey.Valid {
errs = append(errs, fmt.Errorf("budget account transactions need a category"))
}