Merge lp:~therp-nl/openupgrade-addons/6.1-lp769632-purge_ir_model_data_from_deleted_admin_employee into lp:openupgrade-addons/6.1

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 6606
Proposed branch: lp:~therp-nl/openupgrade-addons/6.1-lp769632-purge_ir_model_data_from_deleted_admin_employee
Merge into: lp:openupgrade-addons/6.1
Diff against target: 31 lines (+19/-0)
1 file modified
hr/migrations/6.1.1.1/pre-migration.py (+19/-0)
To merge this branch: bzr merge lp:~therp-nl/openupgrade-addons/6.1-lp769632-purge_ir_model_data_from_deleted_admin_employee
Reviewer Review Type Date Requested Status
OpenUpgrade Committers Pending
Review via email: mp+130095@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hr/migrations/6.1.1.1/pre-migration.py'
2--- hr/migrations/6.1.1.1/pre-migration.py 2012-06-19 09:34:23 +0000
3+++ hr/migrations/6.1.1.1/pre-migration.py 2012-10-17 12:01:41 +0000
4@@ -21,8 +21,27 @@
5
6 from openerp.openupgrade import openupgrade
7
8+def purge_resource_ref(cr):
9+ """
10+ Workaround for https://bugs.launchpad.net/openobject-addons/+bug/769632
11+ when the administrator user has been removed
12+ """
13+ cr.execute("""
14+ SELECT COUNT(id) FROM ir_model_data
15+ WHERE module = 'hr'
16+ AND name = 'employee'"""
17+ )
18+ if not cr.fetchone()[0]:
19+ openupgrade.logged_query(
20+ cr, """
21+ DELETE FROM ir_model_data
22+ WHERE module = 'hr'
23+ AND name = 'employee_resource_resource'"""
24+ )
25+
26 @openupgrade.migrate()
27 def migrate(cr, version):
28+ purge_resource_ref(cr)
29 openupgrade.rename_tables(cr, [('hr_employee_marital_status', openupgrade.get_legacy_name('hr_employee_marital_status'))])
30 openupgrade.rename_columns(cr,
31 {

Subscribers

People subscribed via source and target branches