account_move_line insufficient error message

[Ferdinand]

please add the account id/name to the error message
no possibility to debug this (except expensive trial and error), especially invoices etc with many move_lines

def create...

        if not ok:
            raise osv.except_osv(_('Bad account !'), _('You can not use this general account in this journal !'))

[Ferdinand (office-chricar)]

given the fact that multiple checks are done IMHO all failing test should be reported
* mismatch type_control : account type = .... , allowed types: .......
* mismatch account_control : account = ....., allowed accounts: ...........

it would be nice to have the possiblity to open the journal view directly from the error message (usability issue)

[Mustufa Rangwala (Open ERP) (mra-tinyerp)]

Hello Bharat,

I checked your proposal and it is not properly fixed.
I wrote comment on bug check and correct it.

thanks,
mra

[Bharat Devnani (Open ERP) (bde-openerp)]

Hello Ferdinand,

The warning message has been improved, which gives proper understanding to user when a wrong journal is selected. Revision ID and number of the bug are as follows:

# Revision ID: bde@tiny2-desktop-20110609095106-nxjzoobqqen3txmq
Revision Number : 4773

And it will be merged in the trunk soon. Thanks for your suggestions.

Thanks & Regards,
Devnani Bharat R.

[Mustufa Rangwala (Open ERP) (mra-tinyerp)]

Bharat,

I wrote comment on your merge proposal as it does not solve problem.
Can you please check the comment on Merge proposal and work on it.

Thanks,
Mustufa

[Bharat Devnani (Open ERP) (bde-openerp)]

Hello Ferdinand,

The Revision ID and Number of the bug are as follows:

Revision ID : bde@tiny2-desktop-20110713070832-ps1nt6x7d332se0i
Revision Number : 4777

It will be merged in addons soon.

Thanks & Regards,
Devnani Bharat R.