credativ team mailing list archive

[Mohammad Alhashash <1131653@xxxxxxxxxxxxxxxxxx>]

This bug prevents installing demo data as the module email_template yml
script triggers this bug when setting opt-out  on all partners.

I submitted a fix proposal using CTE directly in the database:
https://code.launchpad.net/~alhashash/openobject-
server/v7_fix_check_recursion_bug_1131653

Mohammad Alhashash

-- 
You received this bug notification because you are a member of
OpenUpgrade Committers, which is subscribed to OpenUpgrade Server.
https://bugs.launchpad.net/bugs/1131653

Title:
  [7.0] _check_recursion gives false positive if ids include a
  parent/child pair

Status in OpenERP Server:
  New
Status in OpenERP Server 6.0 series:
  New
Status in OpenUpgrade Server:
  Triaged
Status in OpenUpgrade Server 7.0 series:
  Triaged

Bug description:
  Here is my company parent structure:

      (Pdb) self.read(cr, uid, [3, 4], ['parent_id'])
      [{'id': 3, 'parent_id': False},
       {'id': 4, 'parent_id': (3, u'Acme 3')}]

  That is to say, record 3 is the parent of record 4. Record 3 does not
  have a parent itself.

  When I write the same value to all records, the constraint on
  _check_recursion signals a recursive structure incorrectly:

      (Pdb) self.check_recursion(cr, uid, [3, 4], context=None, parent=None)
      False

  Having each record checked individually, the method gives the right
  results.

      (Pdb) self._check_recursion(cr, uid, [3], context=None, parent=None)
      True
      (Pdb) self.check_recursion(cr, uid, [4], context=None, parent=None)
      True

  These results are inconsistent and as a result I cannot write the same
  field on these records simultaneously.

To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/1131653/+subscriptions