Merge lp:~therp-nl/openupgrade-server/7.0-lp1111298-remove_currency_companies into lp:openupgrade-server

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 4614
Proposed branch: lp:~therp-nl/openupgrade-server/7.0-lp1111298-remove_currency_companies
Merge into: lp:openupgrade-server
Diff against target: 27 lines (+13/-0)
1 file modified
openerp/addons/base/migrations/7.0.1.3/post-migration.py (+13/-0)
To merge this branch: bzr merge lp:~therp-nl/openupgrade-server/7.0-lp1111298-remove_currency_companies
Reviewer Review Type Date Requested Status
Holger Brunn (Therp) code review Approve
Review via email: mp+166216@code.launchpad.net
To post a comment you must log in.
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-05-03 12:57:10 +0000
3+++ openerp/addons/base/migrations/7.0.1.3/post-migration.py 2013-05-29 10:54:27 +0000
4@@ -164,6 +164,18 @@
5 }
6 partner_obj.write(cr, SUPERUSER_ID, row[1], partner_vals)
7
8+def reset_currency_companies(cr, pool):
9+ """
10+ Having a company on currencies affects multicompany databases
11+ https://bugs.launchpad.net/openobject-server/+bug/1111298
12+ """
13+ currency_ids = pool.get('res.currency').search(
14+ cr, SUPERUSER_ID, [('company_id', '!=', False)],
15+ {'active_test': False})
16+ pool.get('res.currency').write(
17+ cr, SUPERUSER_ID, currency_ids,
18+ {'company_id': False})
19+
20 @openupgrade.migrate()
21 def migrate(cr, version):
22 pool = pooler.get_pool(cr.dbname)
23@@ -176,3 +188,4 @@
24 migrate_company(cr)
25 migrate_partner_address(cr, pool)
26 update_users_partner(cr, pool)
27+ reset_currency_companies(cr, pool)

Subscribers

People subscribed via source and target branches