report - missing syntax check for paranthesis

Bug #798570 reported by Mihai Satmarean

This bug report was converted into a question: question #162294: report - missing syntax check for paranthesis.

8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Odoo Server (MOVED TO GITHUB)
Invalid
Low
OpenERP's Framework R&D

Bug Description

I have this line in the picking report

          <para style="terp_default_8"><b>VAT:</b> [[ (picking.address_id.partner_id and picking.address_id.partner_id.vat) or '') ]]</para>

Problem that I found is that I had an extra closed parenthesis after the word "or ''" , which would be nice to be detected when checking syntax of the report.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello Mihai Boiciuc,

I have checked your issue but I am not getting exactly your problem so would you please elaborate more
about this issue and provide proper step on this.

Thanks and waiting for your reply.

Changed in openobject-addons:
status: New → Incomplete
Revision history for this message
Mihai Satmarean (mihai-satmarean) wrote :

Hi,

sorry for the poor description, I expected that the line of the report which haunted me for days to be obvious. But I guess is not.

SO, I accidentally left an closed parenthesis in the line, which was not detected by the report parser, so I assumed that the internal syntax checker of the report might have missed that. And could be corrected not to miss them again.

Revision history for this message
Vishal Parmar(Open ERP) (vpa-openerp) wrote :

Hello,

Report parser will ignore all the extra braces issue and does not generate any taceback or warning message for end user awareness.

Thanks.

affects: openobject-addons → openobject-server
Changed in openobject-server:
assignee: nobody → OpenERP's Framework R&D (openerp-dev-framework)
importance: Undecided → Low
status: Incomplete → Confirmed
Revision history for this message
Yogesh (SerpentCS) (yogesh-serpentcs) wrote :

Hello,

If openerp server start with --log-level=debug then it will generate traceback in server side.

Thanks,

Changed in openobject-server:
status: Confirmed → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related questions

Remote bug watches

Bug watches keep track of this bug in other bug trackers.