Code review comment for lp:~account-report-core-editor/account-financial-report/fix-opening-period-search

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Thanks Niels for your explanation. You are correct.

>
> Perhaps we should add a check that sees if there are any move_lines which
> use the period or not and find the next period which actually has some
> entries.
> But using the date field is a very bad idea, OpenERP SA has also made this
> assumption many times and we had to report all of them.
>

I think this is the solution which would make everyone happy.

« Back to merge proposal