Merge lp://qastaging/~carlos-realsystems/openobject-server/trunk-bugfix-954189 into lp://qastaging/openobject-server

Proposed by Carlos Contreras
Status: Rejected
Rejected by: Fabien (Open ERP)
Proposed branch: lp://qastaging/~carlos-realsystems/openobject-server/trunk-bugfix-954189
Merge into: lp://qastaging/openobject-server
Diff against target: 4988936 lines (has conflicts)
Conflict adding file .bzrignore.  Moved existing file to .bzrignore.moved.
To merge this branch: bzr merge lp://qastaging/~carlos-realsystems/openobject-server/trunk-bugfix-954189
Reviewer Review Type Date Requested Status
Lorenzo Battistini (community) Needs Fixing
OpenERP Core Team Pending
Review via email: mp+127963@code.qastaging.launchpad.net

Commit message

[FIX] Solves the problem that when you make an purchase order select create and then create an purchase order line.
When you select a product in purchase orders got allways "Selected UOM does not belong to the same category as the product UOM " but the UOM are the same

This error rises because uom_id always have a default UoM PCE and never gets true.

Description of the change

Error #954189:

In 6.1 i have the Problem that when i make an purchase order select create and then create an purchase order line.

When i select then a product i got allways "Selected UOM does not belong to the same category as the product UOM "
but the UOM are the same ?

This error rises because uom_id always have a default UoM PCE and never gets true on line 765:

- if not uom_id:
- uom_id = product_uom_po_id

It solves adding one condition and it doesnt cause harm:

+ if not uom_id or uom_id != product_uom_po_id:
+ uom_id = product_uom_po_id

To post a comment you must log in.
Revision history for this message
Lorenzo Battistini (elbati) wrote :

Hello Carlos, proposal has conflicts.

review: Needs Fixing
Revision history for this message
Lorenzo Battistini (elbati) wrote :

On 10/08/2012 07:48 PM, Carlos Enrique Contreras Vara wrote:
> Dear Lorenzo
>
> How can I do a merge correctly without conflicts because its only one
> line o code of purchase/purchase.py that has been modified.
>

You changed a file from openobject-addons/6.1 branch and proposed to
merge it into openobject-server/trunk.

You should put your branch under the openobject-addons project
(something like
lp:~carlos-realsystems/openobject-addons/6.1-bugfix-954189) and propose
to merge into openobject-addons/6.1 branch.

Regards

--
Lorenzo Battistini <email address hidden>
Via Brüsighell - 6807 - Taverne - Switzerland
Agile Business Group sagl http://www.agilebg.com

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

reject it as 4m lines of diff.
Your merge proposal is probably not clean, you should make feature branches.

Unmerged revisions

7011. By Carlos Contreras

[FIX] fixed Selected UOM does not belong to the same category as the product UOM on purchase order

7010. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7009. By Xavier ALT

[MERGE] OPW 50562: stock: fix picking type for chained picking

  forward port of v6.0 revid: <email address hidden>

7008. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7007. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7006. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7005. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7004. By Olivier Dony (Odoo)

[FIX] OPW 574637: account_analytic_plans: remove the strange behavior of instance editing - a thing of the past

7003. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

7002. By Launchpad Translations on behalf of openerp

Launchpad automatic translations update.

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.