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.

_______________________________________________
Mailing list: https://launchpad.net/~openerp-dev-web
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~openerp-dev-web
More help   : https://help.launchpad.net/ListHelp

Reply via email to