[trunk]sale:in sale order "history" tab refund the invoice

[Kirti Savalia(OpenERP)]

Steps to recreate:
[1]create the sale order.
[2]confirm the sale order.
[3]create final invoice.
[4]Goto the "history" tab on sale order validate the invoice and refund that invoice.

Got the view error.

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello,

It has been Fixed in http://bazaar.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-872686-ksa/
revision-id: <email address hidden>
revno: 5350
It will be available in trunk soon.

Thanks.

[qdp (OpenERP) (qdp)]

not a good fix. Set the bug back to confirmed status

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello,

I checked this issue.I think it can be fix from server team.

Thanks.

[Rucha (Open ERP) (rpa-openerp)]

Hello Kirti,
Can you please be more specific to state the reason or proper workaround for this problem that led to decide this problem is from server side,
this may help them and us,
thanks

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello,

In the orm fields_view_get() method get proper view id but in apply_view_inheritance() method that is does not get the proper model for the particular view passed into the context.
IMHO this bug is tends to the Framework team, not for R&D team.

Thanks.

[Amit Parik (amit-parik)]

Hello,

I have checked this issue, but the problem is that, we can not passed the from view reference in context.

So I am assigning this to Framework team.

Thanks.

[Naresh(OpenERP) (nch-openerp)]

Hello,

Well the problem is the context key *form_view_ref* which is set on the invoice M2M. As this is set on the field so it will be available for each record and in turn will be available for each subset of that record i.e buttons,actions etc so when you click on refund button this key is also passed in the context . Now you may be wondering why this worked in 6.0 ? the reason is in 6.0 we were allowing views to be returned by model and view_id and now we improved in trunk to support view inheritance by copy, means there are cases where several objects would like to use a single view, but by default they cannot because a view belongs to only one model. With this change, it becomes possible to do inheritance "by copy", reusing a view from another model without having to redefine it, just by specifying its view_id in the relevant actions. If the original view needs to be modified, it is also possible to explicitly inherit the parent view from the other model, and the effect will be what is expected: the child view can be specified in an action, and will be the result of the inheritance applied on the parent view; but on the other hande the parent view itself is unaffected when used in the context of its own model. This change was done at 3358 <email address hidden>

So the possible solution is to remove this key from the context before calling fields_view_get from account_invoice_refund

Correct me if I missed something !

Regards,

[Naresh(OpenERP) (nch-openerp)]

so the bug should go back to addons :)

[Kirti Savalia(OpenERP) (ksa-openerp)]

Hello,

Thanks For the nice explanation.
It has been Fixed in http://bazaar.launchpad.net/~openerp-dev/openobject-addons/trunk-bug-872686-ksa/
Revision ID: <email address hidden>
Revision NO:5354
It will be available in trunk soon.

Thanks.

[Raphael Collet (OpenERP) (rco-openerp)]

landed in trunk
revno: 6023
revision-id: <email address hidden>