~therp-nl/openobject-server/7.0-lp1253200-dont_browse_in_res_users_context_get

Viewing all changes in revision 5136.

  • Committer: Olivier Dony
  • Date: 2013-11-20 10:06:27 UTC
  • Revision ID: odo@openerp.com-20131120100627-031fljyf4ckprc9b
[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.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: