~therp-nl/openobject-server/6.1-bug-1071710-chs

Viewing all changes in revision 4366.

  • Committer: Olivier Dony
  • Date: 2013-12-02 09:58:45 UTC
  • Revision ID: odo@openerp.com-20131202095845-1yv86y5olpntz1qg
[FIX] [FIX] orm.browse_record: access error due to prefetch of indirectly referenced records (OPW #596679)
  
  The browse_record prefetching algorithm attempts to
  load data for all known records from the requested
  model (i.e. all IDs present in the browse cache),
  regardless of how indirectly/remotely they were
  referenced. An indirect parent record may therefore
  be prefetched along with its directly browsed children,
  possibly crossing company boundaries involuntarily.
  
  This patch implements a fallback mechanism when
  the prefetching failed due to what looks like an
  ACL restriction. This being a fuzzy concept at the
  moment, it does its best to only catch a restricted
  set of exceptions, and retry loading the data for
  the directly requested ID only.
  
  This may cause a small performance penalty in case
  of real errors (with some spurious logging too),
  but should only be triggered in very few cases.
  
  The downside when this happens is that the prefetching for that
  model gets effectively disabled, requiring multiple
  SQL queries for further access to the data of
  the other directly browsed records.
  
  This EAFP approach seems safer and faster than
  a LBYL technique where we would have to filter
  all indirect m2o references according to ACLs
  before allowing them to enter the cache.

(This is a backport of revision odo@openerp.com-20131120100627-031fljyf4ckprc9b
from the 7.0 branch).

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: