Code review comment for lp:~openerp-commiter/openobject-client/client-image-widget

Revision history for this message
Christophe Simonis (OpenERP) (kangol) wrote :

I don't get it. I don't see why you need to define a new widget ? Is keeping widget="image" not enough, and just handle the case where this is display in tree view? Beeing displayed into a x2many [1] is not a special case.

[1] this is a one2many, not a many2one. And only for this error, the merge is in "Needs Fixing"

review: Needs Fixing

« Back to merge proposal