Code review comment for lp:~akretion-team/openerp-product-attributes/openerp-product-attributes_limit_database_column_name

Revision history for this message
Guewen Baconnier @ Camptocamp (gbaconnier-c2c) wrote :

Thanks for the change!
Why not

re.sub(r'\W', '', string)

Instead of

filter((lambda x: re.search('[0-9a-z_]', x)), string)

?

« Back to merge proposal