~openerp-dev/openobject-server/6.1-m2o-ops

Viewing all changes in revision 4355.

  • Committer: Olivier Dony
  • Date: 2013-06-20 14:02:50 UTC
  • Revision ID: odo@openerp.com-20130620140250-z4z0m0t5auviy6w2
[FIX] OPW 592482: browse_record should not prefetch binary fields
  
  This was apparently a long-standing issue due to a
  strange handling of the _prefetch attribute on
  columns: accessing a column would only trigger
  the prefetching if its _prefetch attribute was
  True, but the prefetching itself would also
  prefetch columns that had _prefetch False.
  
  We clearly want it the other way around, or
  at least we want _prefetch to decide whether
  a column is included in any given prefetching
  pass. We can skip the prefetching pass when
  the only field being accessed has _prefetch
  False because it is likely the other fields
  have already been prefetched separately.
  This last subtlety should not make any
  noticeable performance difference.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: