No error handling when the weight on Magento is not a float

[Karsten Kinateder]

As example:

In Magento the product eav attribute "weight" is a system attribute and cant be changed from Magento Backend. There is no field validation in 1.7-1.8.1 for this field and the type is text in Magento Backend.

So it is possible, that a Backend User types "50 kg" or makes a typo in that field. Connector expects a float and throws exeption cause text is synchronized from magento.

Q&D: UPDATE 'eav_attribute' SET 'frontent_class' = 'validate-decimal' WHERE 'attribute_code' = 'weight'
to correct errors: UPDATE 'catalog_product_entity_text' SET 'value' = CAST(value as DECIMAL(10,2)) JOIN 'eav_attribute' USING 'attribute_id' WHERE 'attribute_code' = 'weight'

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Hi,

Thanks for reporting.

I propose to check if the weight is float during the mapping and to not import it if not, with a log message saying that it couldn't import the weight.

[Karsten Kinateder (karsten-m)]

I would try a type convert, before rejecting the import.

[Guewen Baconnier @ Camptocamp (gbaconnier-c2c)]

Yes, that's what I had in head, but what I wrote was not sufficiently precise indeed.
Thanks

[Karsten Kinateder (karsten-m)]

Not really, when I read second time, we both proposes same. I honk had read to fast.

P.S.:
Have found a typo and iam not so familar with launchpad. Have just started.
Need this a bug report?

https://code.launchpad.net/~karsten-m/openerp-connector-magento/openerp-connector-magento/+merge/214318

[Katja Matthes (katja-matthes)]

Hello,

I just related a branch with our current solution to this problem.