Hello,
I know this bug is a bit old, but I have the same problem here. Albert's
patch works perfectly.
Jay: the problem with check_type() is that it's only called if the
column exists either in self._columns or in self._inherit_fields. When
exporting fields from related fields (i.e. when the column doesn't
belong to the current object) the column types aren't available. Maybe
this is the real bug (?) and it should really never happen.
To reproduce, try going to Sales Management/Sales Orders, create one
entry with a shipping address without country and then export Shipping
Address/Country/Contry Code. At that point 'country_id' isn't found in
self._columns or in self._inherit_fields and check_type isn't called.
Regarding the proposed patch, I find defaulting to an empty string when
marshaling a null object (of an unknown column type) is much better than
crashing.
Thanks!
** Changed in: openobject-server
Status: Invalid => Confirmed
--
You received this bug notification because you are a member of OpenERP
CTP, which is subscribed to OpenERP Server.
https://bugs.launchpad.net/bugs/583265
Title:
CSV export crash when browse null is in some relations
Status in OpenERP Server:
Confirmed
Bug description:
CSV export generates client side error can not marshal
osv.orm.browse_null.
The error comes from the fact in that browse_null are exported instead of
false.
I have made a temporary patch that remove the browse_null, but I did not
have the time to trace the export functions (that lack a little comments), so
the patch can be far better.
Regards
Nicolas
To manage notifications about this bug go to:
https://bugs.launchpad.net/openobject-server/+bug/583265/+subscriptions
_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-gtk
Post to : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-gtk
More help : https://help.launchpad.net/ListHelp
