Comment 5 for bug 942699

Revision history for this message
Stefan Rijnhart (Opener) (stefan-opener) wrote :

Just got bitten by this. I do not understand Vishal's reasoning. The product's copy() method does not work as expected. As per orm API, the defaults dictionary argument allows the developer to set values on the resulting resource. In the case of the product however, the 'name' argument is ignored for no good reason. Niels' solution is perfectly reasonable and does not break the case in which the 'name' argument is not present in the defaults dictionary. So I am setting this to 'confirmed' with a low priority.