credativ team mailing list archive

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

Thank you for fixing this!

If someone got caught by this, they can fix it on an already migrated
database with:

    update account_invoice_line set name = openupgrade_legacy_7_0_name
where name is NULL and openupgrade_legacy_7_0_name is not NULL;


** Changed in: openupgrade-addons
       Status: New => Fix Committed

** Also affects: openupgrade-addons/7.0
   Importance: Medium
       Status: Fix Committed

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

Title:
  [7.0] Incorrect behaviour for account.invoice.line migration (note &
  name)

Status in OpenUpgrade Addons:
  Fix Committed
Status in OpenUpgrade Addons 7.0 series:
  Fix Committed

Bug description:
  Hi,

  I think that there is a little bug in account module. In the merging
  of 'name' and 'note' field in account.invoice.line model.

  Reproduce the bug.

  6.1 : 
  row #1 : {name : 'my name' ; note : 'my note'}
  row #2 : {name : 'my name' ; note : None}

  Expected datas after migration in 7.0 : 
  row #1 : {name : 'my name' + '\n' + 'my note'}
  row #2 : {name : 'my name'}

  Observed datas : 
  row #1 : {name : 'my name' + '\n' + 'my note'}
  row #2 : {name : None}

  Observed log : 
  WARNING : Table 'account_invoice_line': unable to set a NOT NULL constraint on column 'name' !

To manage notifications about this bug go to:
https://bugs.launchpad.net/openupgrade-addons/+bug/1227929/+subscriptions