Review: Needs Fixing
Thanks for the merge proposal but I have a few questions.

What happens if self.localcontext is False? newtext won't be defined...

What's wrong with the line node.text = newtext? Is node.text reused later and 
we don't want to touch it?

Can you tell me how to reproduce the bug?
-- 
https://code.launchpad.net/~openerp-dev/openobject-server/trunk-bug-640677-nco/+merge/61564
Your team OpenERP R&D Team is subscribed to branch 
lp:~openerp-dev/openobject-server/trunk-bug-640677-nco.

_______________________________________________
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