credativ team mailing list archive

[Yann Papouin <yann.papouin@xxxxxxxxx>]

Yann Papouin has proposed merging lp:~yann-papouin/openupgrade-addons/7.0-bug-1318686-project into lp:openupgrade-addons.

Requested reviews:
  OpenUpgrade Committers (openupgrade-committers)
Related bugs:
  Bug #1318686 in OpenUpgrade Addons: "[7.0] ProgrammingError when migrating project"
  https://bugs.launchpad.net/openupgrade-addons/+bug/1318686

For more details, see:
https://code.launchpad.net/~yann-papouin/openupgrade-addons/7.0-bug-1318686-project/+merge/219214
-- 
https://code.launchpad.net/~yann-papouin/openupgrade-addons/7.0-bug-1318686-project/+merge/219214
Your team OpenUpgrade Committers is requested to review the proposed merge of lp:~yann-papouin/openupgrade-addons/7.0-bug-1318686-project into lp:openupgrade-addons.

=== modified file 'project/migrations/7.0.1.1/post-migration.py'
--- project/migrations/7.0.1.1/post-migration.py	2014-04-20 10:04:36 +0000
+++ project/migrations/7.0.1.1/post-migration.py	2014-05-12 14:43:48 +0000
@@ -24,12 +24,12 @@
 
 def copy_state_from_analytic_account(cr):
     openupgrade.logged_query(cr, """
-        UPDATE project_project
+        UPDATE project_project pp
         SET state = account_analytic_account.%s
         FROM project_project
         INNER JOIN account_analytic_account
         ON account_analytic_account.id = project_project.analytic_account_id
-        WHERE analytic_account_id is not NULL
+        WHERE pp.analytic_account_id is not NULL
         """ %(openupgrade.get_legacy_name('state')))
 
 def short_name(name):