Review, and move as many calls to declarative db mappings as possible. - Githubissues

onemoredata / bagger

Massive log storage in PostgreSQL

BSD 2-Clause "Simplified" License

9 stars 0 forks source link

Review, and move as many calls to declarative db mappings as possible. #64

Closed einhverfr closed 8 months ago

einhverfr commented 8 months ago

https://github.com/ledgersmb/LedgerSMB/blob/0b004fb5588e70b1645c2ffc61bd5d65c721867d/lib/LedgerSMB/Entity/Bank.pm#L101

shows we can do the argument mapping that I had not thought was supported yet (either Erik added it or I did and forgot).

This means we can move a lot of the existing functions that call db functions to declarative mappings, reducing the Perl code a bit more.

We should do this.