Merge lp:~sylvain-legal/openupgrade-server/base-res_company-migrate-logo into lp:openupgrade-server

Proposed by Sylvain LE GAL (GRAP)
Status: Merged
Merged at revision: 4634
Proposed branch: lp:~sylvain-legal/openupgrade-server/base-res_company-migrate-logo
Merge into: lp:openupgrade-server
Diff against target: 53 lines (+23/-2)
2 files modified
openerp/addons/base/migrations/7.0.1.3/post-migration.py (+18/-0)
openerp/addons/base/migrations/7.0.1.3/pre-migration.py (+5/-2)
To merge this branch: bzr merge lp:~sylvain-legal/openupgrade-server/base-res_company-migrate-logo
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Stefan Rijnhart (Opener) Approve
Review via email: mp+191510@code.launchpad.net

Commit message

[ADD] migration of the logo of 'res_company'. (in V7 'res_company'.'logo' is now field related to 'res_partner'.'image')

Description of the change

Add migration script for res_company.logo.

To post a comment you must log in.
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Looks good to me!

review: Approve
Revision history for this message
Holger Brunn (Therp) (hbrunn) :
review: Approve (code review)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'openerp/addons/base/migrations/7.0.1.3/post-migration.py'
--- openerp/addons/base/migrations/7.0.1.3/post-migration.py 2013-09-13 20:11:15 +0000
+++ openerp/addons/base/migrations/7.0.1.3/post-migration.py 2013-10-16 21:36:14 +0000
@@ -194,6 +194,23 @@
194 cr, SUPERUSER_ID, currency_ids,194 cr, SUPERUSER_ID, currency_ids,
195 {'company_id': False})195 {'company_id': False})
196196
197def migrate_res_company_logo(cr, pool):
198 """
199 Transfert logo from res_company to res_partner linked to the res_company,
200 according to the new behaviour of V7 :
201 'res_company.logo' is now a field reladed to 'res_partner.image'
202 """
203 partner_obj = pool.get('res.partner')
204 cr.execute("""
205 SELECT partner_id, %s
206 FROM res_company
207 WHERE %s is not null""" %(
208 openupgrade.get_legacy_name('logo'),
209 openupgrade.get_legacy_name('logo')))
210 for row in cr.fetchall():
211 vals = {'image': row[1]}
212 partner_obj.write(cr, SUPERUSER_ID, row[0], vals)
213
197@openupgrade.migrate()214@openupgrade.migrate()
198def migrate(cr, version):215def migrate(cr, version):
199 pool = pooler.get_pool(cr.dbname)216 pool = pooler.get_pool(cr.dbname)
@@ -207,6 +224,7 @@
207 migrate_partner_address(cr, pool)224 migrate_partner_address(cr, pool)
208 update_users_partner(cr, pool)225 update_users_partner(cr, pool)
209 reset_currency_companies(cr, pool)226 reset_currency_companies(cr, pool)
227 migrate_res_company_logo(cr, pool)
210 openupgrade.load_xml(228 openupgrade.load_xml(
211 cr, 'base',229 cr, 'base',
212 'migrations/7.0.1.3/data.xml')230 'migrations/7.0.1.3/data.xml')
213231
=== modified file 'openerp/addons/base/migrations/7.0.1.3/pre-migration.py'
--- openerp/addons/base/migrations/7.0.1.3/pre-migration.py 2013-07-25 06:41:59 +0000
+++ openerp/addons/base/migrations/7.0.1.3/pre-migration.py 2013-10-16 21:36:14 +0000
@@ -38,8 +38,11 @@
38 # orm can map timestamp fields to date field38 # orm can map timestamp fields to date field
39 'res_users': [39 'res_users': [
40 ('date', 'login_date'),40 ('date', 'login_date'),
41 ('user_email', openupgrade.get_legacy_name('user_email')),41 ('user_email', None),
42 ]42 ],
43 'res_company': [
44 ('logo', None),
45 ]
43}46}
4447
45xmlid_renames = []48xmlid_renames = []

Subscribers

People subscribed via source and target branches