default '', or None chained_location_type in stock/stock.py is not valid

[Raphaël Valyi - http://www.akretion.com]

on trunk,

If you try to create a new stock.location ( Stock Management > Configuration > Locations > New ), then, by default, the required chained_location_type (Chained Location Type) field is set to blank ('', or None), which is exactly what you would expect from
the following code:

        'chained_location_type': fields.selection([('','None'),('customer', 'Customer'),('fixed','Fixed Location')],
            'Chained Location Type', required=True),
[...]
    _defaults = {
        'active': lambda *a: 1,
        'usage': lambda *a: 'internal',
        'allocation_method': lambda *a: 'fifo',
        'chained_location_type': lambda *a: '',
...

The trouble is that you can't save it like that! Even if you manually select 'None' in the select box, you can't save it. (the only workarround I have is to select 'Fixed Location'.

I think the default 'None' should just be valid instead.

Hope this helps,

Raphaël Valyi.

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

I think it has been fixed by this commit http://bazaar.launchpad.net/~openerp/openobject-addons/trunk/revision/1915
but I'll test it soon and confirm.

[Christophe Simonis (OpenERP) (kangol)]

I confirm the patch

[Raphaël Valyi - http://www.akretion.com (rvalyi)]

yeah, it's fixed indeed, thank you.