← Back to team overview

openerp-dev-web team mailing list archive

Re: lp:~openerp-dev/openobject-client-web/6.0-opw-5029-4852-sma into lp:openobject-client-web/6.0

 

> Still getting significantly more errors than on GTK, even with branch merged:
> 
> Error while parsing "dict({'check_total': check_total, 'invoice_line':
> invoice_line, 'address_invoice_id': address_invoice_id, 'partner_id':
> partner_id, 'price_type': 'price_type' in dir() and price_type or False})"
> Traceback (most recent call last):
>   File "addons/openerp/utils/tools.py", line 50, in expr_eval
>     value = eval(string, context)
>   File "<string>", line 1, in <module>
> NameError: name 'check_total' is not defined
> 
> Error while parsing u"{'type':parent.type}"
> Traceback (most recent call last):
>   File "addons/openerp/utils/tools.py", line 50, in expr_eval
>     value = eval(string, context)
>   File "<string>", line 1, in <module>
> NameError: name 'parent' is not defined
> 
> Error while parsing u"{'type':parent.type}"
> Traceback (most recent call last):
>   File "addons/openerp/utils/tools.py", line 50, in expr_eval
>     value = eval(string, context)
>   File "<string>", line 1, in <module>
> NameError: name 'parent' is not defined
> 
> as can see, dict `{'type':parent.type}` is parsed twice, and there is still
> error with check_total

Now Fixed context evaluation problem, No need to evaluate context(field_context) with parent_context separately, because it raises `key` not defined` from expr_eval. and we have evaluated context with ctx which contains parent_context too.


-- 
https://code.launchpad.net/~openerp-dev/openobject-client-web/6.0-opw-5029-4852-sma/+merge/56563
Your team OpenERP R&D Team is subscribed to branch lp:~openerp-dev/openobject-client-web/6.0-opw-5029-4852-sma.



References