credativ team mailing list archive

[Matthieu Choplin <choplin.mat@xxxxxxxxx>]

Matthieu Choplin has proposed merging lp:~matthieuchoplin/credativ-openerp/OED-132-improve-stock-image-discrepancy-report into lp:credativ-openerp.

Requested reviews:
  credativ (credativ)

For more details, see:
https://code.launchpad.net/~matthieuchoplin/credativ-openerp/OED-132-improve-stock-image-discrepancy-report/+merge/216075

Hello,

could you please merge this branch please? It is a very small improvement that just adds 2 columns on the report of the stock image discrepancy to show the stock levels we have in the ERP and the stock levels we have in the external WMS.

Thanks,

Matt
-- 
https://code.launchpad.net/~matthieuchoplin/credativ-openerp/OED-132-improve-stock-image-discrepancy-report/+merge/216075
Your team credativ is requested to review the proposed merge of lp:~matthieuchoplin/credativ-openerp/OED-132-improve-stock-image-discrepancy-report into lp:credativ-openerp.

=== modified file 'wms_integration/stock.py'
--- wms_integration/stock.py	2014-01-24 12:17:12 +0000
+++ wms_integration/stock.py	2014-04-16 11:49:53 +0000
@@ -308,12 +308,16 @@
                     ext_qty = 0
                 if not internal_qty:
                     internal_qty = 0
-                diff = int(ext_qty) - int(internal_qty)
+                ext_qty = int(ext_qty)
+                internal_qty = int(internal_qty)
+                diff = ext_qty - internal_qty
                 if diff != 0:
                     message += "%s: %s \n" % (product, diff)
-                    lines_to_export.append([product, diff])
+                    lines_to_export.append([product, diff, internal_qty, ext_qty])
             # Create csv file
-            file_exported = get_csv_format([ (_('SKU'), 'string'), (_('Difference'), 'number')], lines_to_export)
+            file_exported = get_csv_format([(_('SKU'), 'string'), (_('Difference'), 'number'),
+                                            (_('ERP stock'), 'number'), (_('External WMS Stock'), 'number')],
+                                           lines_to_export)
             # Create attachment
             attachment_id = False
             if file_exported: