credativ team mailing list archive

["Stefan Rijnhart \(Therp\)" <1131653@xxxxxxxxxxxxxxxxxx>]

** Also affects: openupgrade-server
   Importance: Undecided
       Status: New

** Changed in: openupgrade-server
   Importance: Undecided => High

** Changed in: openupgrade-server
       Status: New => Triaged

** Also affects: openupgrade-server/7.0
   Importance: High
       Status: Triaged

** Changed in: openupgrade-server/7.0
     Assignee: (unassigned) => Stefan Rijnhart (Therp) (stefan-therp)

-- 
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 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