Odoo Addons (MOVED TO GITHUB)

Merge lp://qastaging/~numerigraphe/openobject-addons/6.0-extra-migration-populate-fields into lp://qastaging/openobject-addons/extra-6.0

6.0-extra-migration-populate-fields
Merge into extra-6.0

Proposed by Numérigraphe on 2012-01-17

Status:	Needs review
Proposed branch:	lp://qastaging/~numerigraphe/openobject-addons/6.0-extra-migration-populate-fields
Merge into:	lp://qastaging/openobject-addons/extra-6.0
Diff against target:	83 lines (+35/-15) 1 file modified server_migration/migration.py (+35/-15)
To merge this branch:	bzr merge lp://qastaging/~numerigraphe/openobject-addons/6.0-extra-migration-populate-fields
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Numérigraphe	target branch		Needs Information on 2012-03-19
Lorenzo Battistini		2012-01-17	Approve on 2012-03-19
Jordi Esteve (www.zikzakmedia.com)		2012-01-17	Pending
sraps (Alistek)		2012-01-17	Pending
Review via email: mp+88911@code.qastaging.launchpad.net

Description of the change

This branch make server_migration pre-load the old field when it has the same name as the new field, when you push the button "Get Fields".

lp://qastaging/~numerigraphe/openobject-addons/6.0-extra-migration-populate-fields updated on 2012-02-13

5650. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-01-18: [FIX] server_migration: don't crash when old_model is emptpty
5651. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-01-24: [IMP] server_migration: ignore function fields in button "get fields" ; more logging"
5652. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-02-13: [IMP] server_migration: remove print statements.

Revision history for this message

Lorenzo Battistini (elbati) wrote on 2012-03-19:

Thank you Lionel,

I pushed this patch to lp:~openerp-community/domsense-agilebg-addons/server_migration-6.1
See it at http://bazaar.launchpad.net/~openerp-community/domsense-agilebg-addons/server_migration-6.1/revision/17

I would avoid using the extra branch as it is hard to handle and maintain.
Anyway, I confirm this patch works ;-)

review: Approve

Revision history for this message

Numérigraphe (numerigraphe) wrote on 2012-03-19:

Dear Lorenzo Battistini,
Is this the main branch for this module? If so, I suggest we remove the module from the extra addons altogether.
Lionel.

review: Needs Information (target branch)

Revision history for this message

Lorenzo Battistini (elbati) wrote on 2012-03-19:

2012/3/19 Numérigraphe <email address hidden>:
> Is this the main branch for this module? If so, I suggest we remove the module from the extra addons altogether.

I consider lp:~openerp-community/domsense-agilebg-addons/datamig-6 as
the main branch for openerp 6.0
and lp:~openerp-community/domsense-agilebg-addons/server_migration-6.1 for 6.1

I agree to remove it from the extra branch.

Unmerged revisions

5652. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-02-13: [IMP] server_migration: remove print statements.
5651. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-01-24: [IMP] server_migration: ignore function fields in button "get fields" ; more logging"
5650. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-01-18: [FIX] server_migration: don't crash when old_model is emptpty
5649. By Numerigraphe - Lionel Sausin <email address hidden> on 2012-01-17: [IMP] server_migration: populate the fields from the old model when the field's name is the unchanged.