erp5_base: no need to change quantity if already zero

bt5/erp5_base/MixinTemplateItem/portal_components/mixin.erp5.BuilderMixin.py

@@ -497,9 +497,10 @@ class BuilderMixin(XMLObject, Amount, Predicate):
     )
     #previous_movement_list= []
     for brain in previous_movement_list:
-      previous_movement = brain.getObject()
-      previous_movement.setQuantity(0)
-      previous_movement.setPrice(0)
+      if brain.getQuantity() != 0:
+        previous_movement = brain.getObject()
+        previous_movement.setQuantity(0)
+        previous_movement.setPrice(0)
 
     # Calculate first possible shipment:
     first_shipment_date = max(

bt5/erp5_base/MixinTemplateItem/portal_components/mixin.erp5.BuilderMixin.xml

@@ -44,11 +44,11 @@
             <key> <string>text_content_warning_message</string> </key>
             <value>
               <tuple>
-                <string>W:521, 35: Unused variable \'portal_type\' (unused-variable)</string>
-                <string>W:664, 18: Unused variable \'total_inventory\' (unused-variable)</string>
+                <string>W:522, 35: Unused variable \'portal_type\' (unused-variable)</string>
+                <string>W:665, 18: Unused variable \'total_inventory\' (unused-variable)</string>
                 <string>W:421,  4: Unused variable \'original_min_stock_value\' (unused-variable)</string>
                 <string>W:422,  4: Unused variable \'original_factor\' (unused-variable)</string>
-                <string>W:521, 14: Unused variable \'inventory\' (unused-variable)</string>
+                <string>W:522, 14: Unused variable \'inventory\' (unused-variable)</string>
               </tuple>
             </value>
         </item>