Odoo Server (MOVED TO GITHUB)

Code review comment for lp:~therp-nl/openobject-server/ronald@therp.nl_fix_ir_ui_view_arch_lp925840-6.1

  1. ronald@therp.nl_fix_ir_ui_view_arch_lp925840-6.1
  2. Merge into trunk
Revision history for this message
Raphael Collet (OpenERP) (rco-openerp) wrote :

Stripping all whitespaces in views is inadequate. It is common to edit a view with the views form. There, you want to keep the indentation of the XML in the field 'arch'.

Note that the impact of reducing the size of views on performance is negligible. (The number of queries sent to the DBMS has much more impact than the size of the results.)

review: Disapprove

« Back to merge proposal