[18.0][FIX] account_financial_report: account selection by interval when handling NewId records

[Ricardoalso]

Fixing some unfixed on_change_account_range methods following the same approach as the ones done in #1416 and #1293

Useful explanation from the comments

Q: it is not clear to me why we no longer have to handle the NewId case here. Could you enlighten me?

A: Because this is an onchange method on an unsaved Transient Model record. Odoo wraps the assigned records with NewId objects. By calling the filtered on those records of NewId type, it fails.
And now we are only calling the search method once with the adapted domain, improving overall performances also.

[gurneyalex]

it is not clear to me why we no longer have to handle the NewId case here. Could you enlighten me?

[Ricardoalso]

Because this is an onchange method on an unsaved Transient Model record. Odoo wraps the assigned records with NewId objects. By calling the filtered on those records of NewId type, it fails.

And now we are only calling the search method once with the adapted domain, improving overall performances also.