openerp-dev-web team mailing list archive

["Xavier \(Open ERP\)" <xmo@xxxxxxxxxxx>]

Review: Needs Fixing
The '/' prefix was added at the same time the conditional above was modified to add a 'split' in it.

I'm pretty sure if we go through conditional, things are going to break because there will not be a final '/' and  so we'll get `somefield_terp_model` instead of `somefield/_terp_model`.

I think we need to leave it before `_terp_model` but remove adding of trailing '/' in original creation of `prefix`. What do you think?
-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5796-sma/+merge/61388
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-5796-sma.