account_voucher - class res_company must be defined first

[Ferdinand]

expense_currency_exchange_account_id never gets created , because module fails to install

[Vishal Parmar(Open ERP) (vpa-openerp)]

Hello Dr.Ferdinand,

I have checked your issue with trunk but I didn't face any problem as you have. would you please try again
with updated code and provide your end revision if you have still faced the problem.

Thanks and waiting for your reply.

[Ferdinand (office-chricar)]

well I could not load account_voucher because

this is at the end

class res_company(osv.osv):
    _inherit = "res.company"
    _columns = {
        'income_currency_exchange_account_id': fields.many2one(
            'account.account',
            string="Income Currency Rate",
            domain="[('type', '=', 'other')]",),
        'expense_currency_exchange_account_id': fields.many2one(
            'account.account',
            string="Expense Currency Rate",
            domain="[('type', '=', 'other')]",),
    }

res_company()

and

 921 if amount_residual > 0:
 922 account_id = line.voucher_id.company_id.expense_currency_exchange_account_id

fails

[Jignesh Rathod(OpenERP) (jir-openerp)]

Hello,

I did not face any trace back but a better coding convention I have
confirming this issue.

Thank you!

[DJ Patel (OpenERP) (mdi-openerp)]

Hello Ferdinand,

Thanks for reporting. The solution for this bug is proposed in the branch : https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-908704-mdi/

with following Revision ID and Number.

Revision ID : <email address hidden>
Revision Number : 6193

Thanks and Regards,

Divyesh Makwana(MDI)

[qdp (OpenERP) (qdp)]

for "a better coding convention"? seriously who decided that it was "a better coding convention" to redefine first the class res_company before any other? i do not like working for nothing and confirming this issue and making a merge proposal is pretending that it is really an issue whereas you (and i) didn't face any problem.

Worst, it can shadow a real problem Ferdinand found.

So, i'm rejecting this bug and the merge proposal. Ferdinand, can you try again with a latest version of trunk and let us know?

Thanks,
Quentin

[Ferdinand (office-chricar)]

loading of account_voucher works now

[Ferdinand (office-chricar)]

sorry I have tested this with "my" modification
have to get a trunk version to make a new test

[Ferdinand (office-chricar)]

loading of account_voucher works now - with code from today