Stock And Logistic Warehouse

Code review comment for lp:~numerigraphe-team/stock-logistic-warehouse/7.0-inventory-location

  1. 7.0-inventory-location
  2. Merge into 7.0
Revision history for this message
Laetitia Gangloff (Acsone) (laetitia-gangloff) wrote :

Hello,

I see some little mistake that I correct in :
http://bazaar.launchpad.net/~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga/revision/40
http://bazaar.launchpad.net/~acsone-openerp/stock-logistic-warehouse/7.0-inventory-hierarchical-fill2-lga/revision/41

It seems I forgot to report them on this branch :(.

The most important of them is the second one.

The change is the following:

l.699 ids_to_check = ids
should be : ids_to_check = list(ids)

review: Needs Fixing

« Back to merge proposal