confirm quotation - move update

Bug #1239102 reported by Milan Tribuson
28
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Odoo Addons (MOVED TO GITHUB)
Confirmed
Medium
OpenERP R&D Addons Team 2

Bug Description

There is a bug when you try to confirm a Quotation:
If a quotation is created from a sale order (when a product in sale order is configured as "procurement method": "make to order" and "supply method": "buy") then you are unable to confirm the quotation when not logged in as Administrator.
This happens because there is a part of code in purchase.py around line 663:

                if order_line.move_dest_id:
                    order_line.move_dest_id.write({'location_id': order.location_id.id})

And this will produce an error because you cannot update significant data on stock.move when they are in state "done".

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

Hello Milan,

I have checked your issue with latest stable-7 at my end but all are working fine.
I have prepared a video and attached with this bug which shows that its working fine.

Would you please check and let me know your response, or you can notify me If I missed something.

Thank you!

Revision history for this message
Amit Parik (amit-parik) wrote :
Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Milan Tribuson (milan-infokom) wrote :

Hi Amit!

You have to process your outgoing shipment first before you go to purchase order. This code will produce an error if you try to do this on a stock.move which is in state done. You have created and confirmed your sale order but you haven't processed your outgoing shipment before going to purchase order...

Changed in openobject-addons:
status: Incomplete → New
Revision history for this message
Amit Parik (amit-parik) wrote :

Hello Milan,

Thanks for the replied!

I got this problem
Operation Forbidden!

Quantities, Units of Measure, Products and Locations cannot be modified on stock moves that have already been processed (except by the Administrator).

Let me confirm the issue!
Thank you!

Amit Parik (amit-parik)
Changed in openobject-addons:
importance: Undecided → Medium
status: New → Confirmed
Revision history for this message
Milan Tribuson (milan-infokom) wrote :

No problem Amit, hope you can solve this quickly. I suppose that they should delete that part of code or make a workaround. They could raise an error saying why this is forbidden but that wouldn't help so much because outgoing shipping order is done and you can't do nothing about it (you'll never be able to confirm this purchase order).

Cheers!

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

Hello,

Let me assign this to our R&D team they will find out quick and precious solution for this .

Cheers and thanks for the quick response every-time!

Changed in openobject-addons:
assignee: nobody → OpenERP R&D Addons Team 2 (openerp-dev-addons2)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Bug attachments

Remote bug watches

Bug watches keep track of this bug in other bug trackers.