[trunk, 6.1, 6.0] [account] [wizard account.invoice.refund] Invoice refund wizard make error with other relational fields

[Moisés López - http://www.vauxoo.com]

1) Steps to reproduce the issue you have observed:
   Add a new many2one field to account.invoice.line model.
   Create a new invoice
   Press button refund to invoke account.invoice.refund from invoice
   Confirm operation
2) The result you observed
    Error with new field many2one.
3) The result you expected
    Create a new refund invoice
4) The platform your are using
    OpenERP-6 & OpenERP-Trunk
5) OpenERP-6 revno 4934
     OpenERP-Trunk revno 5791

This bug is result of a Hard-Code in function:
account/account_invoice.py -> class account_invoice -> def _refund_cleanup_lines
The relational fields is assing with the name of current fields (hard-code), but this not compatible with new functionallity and inherit.

I will create a proposal merge with the issue

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

Hello Moisés López,

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

with following Revision ID and Number.

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

Thanks and Regards,

Divyesh Makwana(MDI)

[Moisés López - http://www.vauxoo.com (moylop260)]

Hello Divyesh Makwana

I tested it Done!

Greats

[Moisés López - http://www.vauxoo.com (moylop260)]

Hello,
Excuse me

What is the state of this issue?

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

Hello Moisés López,

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

with following Revision ID and Number.

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

Thanks,
Divyesh

[Moisés López - http://www.vauxoo.com (moylop260)]

Hello Somesh,
You will need try it with data in new field.
If field is empty no error.
If field has data show error

I tested the merge proposal of Divyesh Makwana
https://code.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-896462-mdi
And this fix is fine.

Just we need this merge in OpenERP-6.1 and OpenERP-6.0

Regards

[Somesh Khare (somesh.khare)]

Hello Moy lop,

With reference to your maintenance Ticket, As we have discussed with our experts and came to a conclusion that we can not make such a major changes into the stable series because It may create regression on the systems those are already in production, specially those who are using the account related module.

As a fix has already proposed for the trunk version and once approved will be available with the next OpenERP version, So for stable we keep the status of this bug as Won't fix.

Thanks for your understanding.

Regards,
Somesh Khare

[Moisés López - http://www.vauxoo.com (moylop260)]

Hello Somesh
I understand, thank you.

Now, we have a the same problem with new version 7.0 & trunk

[Nhomar - Vauxoo (nhomar)]

Hello.

Again the video, please this is a big problem, because is silently passing:

http://www.youtube.com/watch?v=EmWcnFSk6fk&sns=em

In our OPW contract they dont want to merge (even already merged in 6.1) and they forgot merge to 7.0, IMHO it must be merged.

Regards.

[Moisés López - http://www.vauxoo.com (moylop260)]

I add the Somesh's conversation:
...so that our R & D team will find a more clean fix for the trunk version with reference to the lp:~openerp-dev/openobject-addons/trunk-bug-896462-mdi branch because it required lots of changes which is against the stable bug policy...