Merge lp:~therp-nl/openupgrade-server/6.0-lp1025797-dangling_references into lp:openupgrade-server/6.0

[Stefan Rijnhart (Opener)]

As reported in the associated issue, OpenERP 5 leaves dangling references to removed resources in ir_model_data. This branch checks for them at model initialization time and removes them.

[Holger Brunn (Therp) (hbrunn)]

review: Approve

[Don Kirkby (donkirkby)]

This proposal would solve the error, as well as bug 1025819, along with any other modules where deleted records cause problems.

However, I don't like the fact that records magically reappear during the migration, perhaps months after some administrator deleted them. That's why I recreated them as inactive records in my merge proposal.

How about a third option? Create a working table and copy all the dangling ir_model_data records there, then delete them. At the end of the migration, go through the working table and deactivate all the records that got recreated. That way, it's a general solution that doesn't require custom scripting for each module, you don't have to make up bogus values for the recreated records, but the deleted records still aren't visible.

If you agree in principle, I'll make a merge proposal for you to review. If you still prefer the current proposal, I can work around it by recreating the records before I start the migration.

review: Disapprove

[Stefan Rijnhart (Opener) (stefan-opener)]

Hi Don,

thank you for your opinion. I understand you really want to see lp:1023615 fixed but it must be fixed in the OpenERP server, not in this project.

Stefan.

=== modified file 'bin/osv/orm.py'
--- bin/osv/orm.py	2012-05-27 12:34:50 +0000
+++ bin/osv/orm.py	2012-07-18 10:49:22 +0000
@@ -2739,6 +2739,19 @@
             for order, f, k in todo_update_store:
                 todo_end.append((order, self._update_store, (f, k)))
 
+            # OpenUpgrade: remove dangling references from ir_model_data
+            cr.execute("""
+                DELETE FROM ir_model_data
+                WHERE model = \'%s\'
+                AND res_id not in (
+                    SELECT id FROM %s
+                )
+                """ % (self._name, self._table))
+            if cr.rowcount:
+                self.__logger.debug(
+                    "Removed %s dangling reference(s) to resources of model %s "
+                    "in ir_model_data", cr.rowcount, self._name)
+
         else:
             cr.execute("SELECT relname FROM pg_class WHERE relkind IN ('r','v') AND relname=%s", (self._table,))
             create = not bool(cr.fetchone())