Comment 2 for bug 1156830

Revision history for this message
Amit Bhavsar (Open ERP) (amb-openerp) wrote :

Hello Daniel,

It's really nice Amelioration, But this is not a bug but rather than it's your suggestion to Improve the code of qweb2.js for manage complex expressions.

We can also Improve this code with out change qweb2.js in following way...
<li t-if="record.city.raw_value or record.country.raw_value">
    <field name="city"/>
        <t t-if="record.city.raw_value and record.country.raw_value">,</t>
    </t><field name="country"/>
</li>

But.......... ... ..
It's good Implementation as per the coding conventions point of view. So we can consider this is as your suggestion and considering it as a "Wishlist". and Let the team to take decision on it.

@Antoine (al) : please give look.

Thanks for the reporting!