Comment 1 for bug 526930

If I understood you correctly and when you say "All moves" you refer to "All account entries" then I agree with you that it makes more sense to have a listing of all moves related to the partner accounts ('receivable' and 'payable', maybe also when reconcile is True) when clicking the "All moves" rather than a listing of all types of accounts.
However the account type is not a functional field...
You can change the domain of "All moves" action from [('partner_id', '=', active_id)] to [('partner_id', '=', active_id),('account_id.type','in',('receivable','payable'))] and you would get the behaviour you desire...
You can test this on a working data base, search the "All Moves" (or "All account entries" in my case) action in "Administration/Low Level objects/Actions/Window Actions" and editing the domain field.
So I think changing this in the account/account_invoice_view.xml would be a better solution than changing the code that generates the account move for the invoice.
Regards,
Hernán