Comment 10 for bug 531507

The assumption that only move.lines from the year to be closed would need to be closed seems a right one.

However, as specified in the initial report, that's not what this patch or bug is about, and your patch would not /seem/ to prevent the bug from happening.

The first bug relates to the fact that an invoice could be made in a previous year, and/or be paid in the/a next year. Neither will reconcile correctly.
The second bug relates to invoices that are made in the current (or, after my patch, previous years) and be paid in several parts in the next year(s).

The current code would generate a move.line for each payment, resulting in a horrible mess.