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
1=== modified file 'openerp/addons/base/migrations/7.0.1.3/post-migration.py'
2--- openerp/addons/base/migrations/7.0.1.3/post-migration.py 2013-09-13 20:11:15 +0000
3+++ openerp/addons/base/migrations/7.0.1.3/post-migration.py 2013-10-16 21:36:14 +0000
4@@ -194,6 +194,23 @@
5 cr, SUPERUSER_ID, currency_ids,
6 {'company_id': False})
7
8+def migrate_res_company_logo(cr, pool):
9+ """
10+ Transfert logo from res_company to res_partner linked to the res_company,
11+ according to the new behaviour of V7 :
12+ 'res_company.logo' is now a field reladed to 'res_partner.image'
13+ """
14+ partner_obj = pool.get('res.partner')
15+ cr.execute("""
16+ SELECT partner_id, %s
17+ FROM res_company
18+ WHERE %s is not null""" %(
19+ openupgrade.get_legacy_name('logo'),
20+ openupgrade.get_legacy_name('logo')))
21+ for row in cr.fetchall():
22+ vals = {'image': row[1]}
23+ partner_obj.write(cr, SUPERUSER_ID, row[0], vals)
24+
25 @openupgrade.migrate()
26 def migrate(cr, version):
27 pool = pooler.get_pool(cr.dbname)
28@@ -207,6 +224,7 @@
29 migrate_partner_address(cr, pool)
30 update_users_partner(cr, pool)
31 reset_currency_companies(cr, pool)
32+ migrate_res_company_logo(cr, pool)
33 openupgrade.load_xml(
34 cr, 'base',
35 'migrations/7.0.1.3/data.xml')
36
37=== modified file 'openerp/addons/base/migrations/7.0.1.3/pre-migration.py'
38--- openerp/addons/base/migrations/7.0.1.3/pre-migration.py 2013-07-25 06:41:59 +0000
39+++ openerp/addons/base/migrations/7.0.1.3/pre-migration.py 2013-10-16 21:36:14 +0000
40@@ -38,8 +38,11 @@
41 # orm can map timestamp fields to date field
42 'res_users': [
43 ('date', 'login_date'),
44- ('user_email', openupgrade.get_legacy_name('user_email')),
45- ]
46+ ('user_email', None),
47+ ],
48+ 'res_company': [
49+ ('logo', None),
50+ ]
51 }
52
53 xmlid_renames = []

Subscribers

People subscribed via source and target branches