Comment 9 for bug 392114

Revision history for this message
Sébastien BEAU - http://www.akretion.com (sebastien.beau) wrote :

Hi hda, jay.
I've tested your patch too. And I confirm it doesn't work. Moreover it's look very strange to have two price_accuracy default value in the same file (the proposed patch by Jay doesn't address this)

In OpenERP command line arguments are prioritar and this is logic and normal.
But as there is a default value for the option --price_accuracy in the command line arguments, this value will be always being set.
For me the solution is to apply Christophe's patch which delete the value default in the command line arguments.
This patch works fine because :
- the program uses by default the price_accuracy defined at the start of the def __init__
- after, if a price_accuracy exists in the config file this new value will replace the default value
- and to finish if a new value is set with the command line arguments this value will replace the previous value.

Regards

Sébastien BEAU