Merge lp://qastaging/~openerp-dev/openobject-addons/trunk-bug-1020882-amp into lp://qastaging/openobject-addons

Proposed by Amit Parik
Status: Needs review
Proposed branch: lp://qastaging/~openerp-dev/openobject-addons/trunk-bug-1020882-amp
Merge into: lp://qastaging/openobject-addons
Diff against target: 25 lines (+15/-0)
1 file modified
purchase/purchase_view.xml (+15/-0)
To merge this branch: bzr merge lp://qastaging/~openerp-dev/openobject-addons/trunk-bug-1020882-amp
Reviewer Review Type Date Requested Status
Amit Parik (community) Needs Resubmitting
Fabien (Open ERP) Disapprove
Review via email: mp+114165@code.qastaging.launchpad.net

Description of the change

Hello,

This branch fixed the problem of purchase act_window 's form view doesn't open at product object.

Here on code missed to create new action for this act_window and passed act_window reference. This fixes will solve this issue.

Otherwise following error is generated.
" Could not get field with name 'parent.pricelist_id' for onchange 'onchange_product_uom(parent.pricelist_id,product_id,product_qty,product_uom,parent.partner_id, parent.date_order,parent.fiscal_position,date_planned,name,price_unit,notes,context) "

Thanks

To post a comment you must log in.
7015. By Anto

[MERGE] trunk

Revision history for this message
Fabien (Open ERP) (fp-tinyerp) wrote :

hello,

it's not normal to have both new records to have the same sequence number. One should be before the other. Please review.

review: Disapprove
7016. By Amit Parik

[FIX] Improved the fix

Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Fabien,

I have improved the code as per your suggestion.

Please review it.

Thank you!

review: Needs Resubmitting

Unmerged revisions

7016. By Amit Parik

[FIX] Improved the fix

7015. By Anto

[MERGE] trunk

7014. By Amit Parik

[MERGE]: Merge with lp:openobject-addons

7013. By Amit Parik

[FIX] purchase : Fix the problem of purchase order line form view doesn't open from product act_wondow

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: