Comment 6 for bug 526930

Revision history for this message
Borja López Soilán (NeoPolus) (borjals) wrote :

Err... Jay, please, read my post. I think that your commit is not the right thing to do:

Now we have to links that do mostly the same (the only difference is that the "All account entries" one shows reconciled moves too):

        <act_window domain="[('partner_id', '=', active_id), ('account_id.type', 'in', ['receivable', 'payable']), ('reconcile_id','=',False)]" id="act_account_partner_account_move_unreconciled" name="Receivables &amp; Payables" res_model="account.move.line" src_model="res.partner"/>

        <act_window domain="[('partner_id', '=', active_id), ('account_id.type', 'in', ['receivable', 'payable'])]" id="act_account_partner_account_move" name="All account entries" res_model="account.move.line" src_model="res.partner"/>

I think we shouldn't change the "All account entries" (if it doesn't display all the entries related to the partner... why do we call it like that?), but (if we really need it) add a new link and rename the current "Receivables & Payables" instead:

        <act_window domain="[('partner_id', '=', active_id), ('account_id.type', 'in', ['receivable', 'payable']), ('reconcile_id','=',False)]" id="act_account_partner_account_move_unreconciled" name="Unreconciled Receivables &amp; Payables" res_model="account.move.line" src_model="res.partner"/>

        <act_window domain="[('partner_id', '=', active_id), ('account_id.type', 'in', ['receivable', 'payable'])]" id="act_account_partner_account_move_unreconciled" name="All Receivables &amp; Payables" res_model="account.move.line" src_model="res.partner"/>

        <act_window domain="[('partner_id', '=', active_id)]" id="act_account_partner_account_move" name="All account entries" res_model="account.move.line" src_model="res.partner"/>