Merge lp:~camptocamp/openerp-connector/7.0-e-commerce-addons-onchange-lines-commands into lp:openerp-connector/7.0-e-commerce-addons

[Guewen Baconnier @ Camptocamp]

Actually, the onchange assume that we have only new lines in a sales order.
If we want to use it on an existing sales order with existing lines, we use the command "(1, ID, {values})".

This change aims to support all the command types, not only "(0, 0, {values})".

[lp:~camptocamp/openerp-connector/7.0-e-commerce-addons-onchange-lines-commands]

340. By Guewen Baconnier @ Camptocamp on 2013-11-12

[FIX] simplify last change

=== modified file 'connector_ecommerce/unit/sale_order_onchange.py'
--- connector_ecommerce/unit/sale_order_onchange.py	2013-08-06 16:15:11 +0000
+++ connector_ecommerce/unit/sale_order_onchange.py	2013-11-12 13:24:17 +0000
@@ -210,14 +210,15 @@
             # shipping fees with an OpenERP Product
             line_lists.append(order['order_line'])
         for line_list in line_lists:
-            for idx, line in enumerate(line_list):
+            for idx, command_line in enumerate(line_list):
                 # line_list format:[(0, 0, {...}), (0, 0, {...})]
-                old_line_data = line[2]
-                new_line_data = self._play_line_onchange(old_line_data,
-                                                         processed_order_lines,
-                                                         order)
-                new_line = (0, 0, new_line_data)
-                processed_order_lines.append(new_line)
-                # in place modification of the sale order line in the list
-                line_list[idx] = new_line
+                if command_line[0] in (0, 1):  # create or update values
+                    # keeps command number and ID (or 0)
+                    old_line_data = command_line[2]
+                    new_line_data = self._play_line_onchange(
+                        old_line_data, processed_order_lines, order)
+                    new_line = (command_line[0], command_line[1], new_line_data)
+                    processed_order_lines.append(new_line)
+                    # in place modification of the sale order line in the list
+                    line_list[idx] = new_line
         return order