This is in SP2 and SP3.


If CopyBvin is true, LineItem.Clone assigns the result.Bvin then copies the Inputs and Modifiers. But the property Getter for both Inputs and Modifiers uses code to look up the private _Inputs and _Modifiers collections like this:



_Modifiers = Orders.LineItemModifier.FindByLineItemId(Me.Bvin)



Since the Bvin has already been assigned, both the original (Me.Modifiers) and the clone (result.Modifiers) have the same values even before the Inputs and Modifiers are copied.



Basically this means that every time through the Clone method, the Inputs and Modifiers are copied doubled (if they were persisted in between calls).



The easy fix is to move the Bvin assignment to the bottom of the Clone method.