Code review comment for lp:~openerp-dev/openobject-addons/7.0-opw-596206-fka

Revision history for this message
Olivier Dony (Odoo) (odo-openerp) wrote :

Your patch is wrong because you're taking the characters *after* 64 instead of *before*!! Please review and test the actual effect of your code before submitting it.

In any case, it seems clear to me that the right fix is to correct fields.function to properly apply the size limit that was set on it when the type is "char". In this case `ref` is a fields.related which is also a fields.function that is stored, so this is the source of the bug. It will also avoid hardcoding a limit in the code while the one of the model should be used.

Martin Trigaux is preparing a branch to fix this.

Thanks!

review: Disapprove

« Back to merge proposal