Merge lp:~therp-nl/openupgrade-addons/7.0-auth_crypt into lp:openupgrade-addons

Proposed by Stefan Rijnhart (Opener)
Status: Merged
Merged at revision: 8165
Proposed branch: lp:~therp-nl/openupgrade-addons/7.0-auth_crypt
Merge into: lp:openupgrade-addons
Diff against target: 46 lines (+37/-0)
2 files modified
auth_crypt/migrations/7.0.1.1/openupgrade_analysis_work.txt (+6/-0)
auth_crypt/migrations/7.0.1.1/pre-migration.py (+31/-0)
To merge this branch: bzr merge lp:~therp-nl/openupgrade-addons/7.0-auth_crypt
Reviewer Review Type Date Requested Status
Pedro Manuel Baeza code review Approve
Review via email: mp+219433@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

I had this one pending! Thanks for doing before.

Regards.

review: Approve (code review)
Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Release early, release often ;-)

Thanks for the review.

Revision history for this message
Pedro Manuel Baeza (pedro.baeza) wrote :

We can merge this branch due to long inactivity, and there's no risk in doing it.

Regards.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'auth_crypt/migrations/7.0.1.1/openupgrade_analysis_work.txt'
2--- auth_crypt/migrations/7.0.1.1/openupgrade_analysis_work.txt 1970-01-01 00:00:00 +0000
3+++ auth_crypt/migrations/7.0.1.1/openupgrade_analysis_work.txt 2014-05-13 21:48:22 +0000
4@@ -0,0 +1,6 @@
5+---Fields in module 'auth_crypt'---
6+# hashed password is now stored in a separate column. Assume that all
7+# passwords are hashed after installation of base_crypt in 6.1, so
8+# rename the column
9+auth_crypt / res.users / password_crypt (char) : NEW
10+---XML records in module 'auth_crypt'---
11
12=== added file 'auth_crypt/migrations/7.0.1.1/pre-migration.py'
13--- auth_crypt/migrations/7.0.1.1/pre-migration.py 1970-01-01 00:00:00 +0000
14+++ auth_crypt/migrations/7.0.1.1/pre-migration.py 2014-05-13 21:48:22 +0000
15@@ -0,0 +1,31 @@
16+# -*- coding: utf-8 -*-
17+##############################################################################
18+#
19+# OpenERP, Open Source Management Solution
20+# This module copyright (C) 2014 Therp BV (<http://therp.nl>).
21+#
22+# This program is free software: you can redistribute it and/or modify
23+# it under the terms of the GNU Affero General Public License as
24+# published by the Free Software Foundation, either version 3 of the
25+# License, or (at your option) any later version.
26+#
27+# This program is distributed in the hope that it will be useful,
28+# but WITHOUT ANY WARRANTY; without even the implied warranty of
29+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
30+# GNU Affero General Public License for more details.
31+#
32+# You should have received a copy of the GNU Affero General Public License
33+# along with this program. If not, see <http://www.gnu.org/licenses/>.
34+#
35+##############################################################################
36+
37+from openerp.openupgrade import openupgrade
38+
39+column_renames = {
40+ 'res_users': [('password', 'password_crypt')],
41+}
42+
43+
44+@openupgrade.migrate()
45+def migrate(cr, version):
46+ openupgrade.rename_columns(cr, column_renames)

Subscribers

People subscribed via source and target branches