Callers up the stack should not mutate returned value (and all callers
found in this repository code follow this rule).
Saves a list copy and an extra call on each call, and a few lines of code.

work partially done by Vincent Pelletier (z_get_table_schema)

getLocalRolesGroupIdDict is supposed to return sorted & cast
allowed_roles_and_users value already.
Also, use "x in y" pattern over y.has_key(x) for readability.

This method is useful by some scripts of erp5_administration

caching_instance_method not used

Update callers which could not be fixed so they disable the exception.
Reviewed-by: Jerôme Perrin <jerome@nexedi.com>

Reviewed-by: Jerôme Perrin <jerome@nexedi.com>

- group_by, group_by_expression -> group_by_list
- select_list, select_expression -> select_dict
- sort_on, sort_on_order, order_by_expression -> order_list

Revert "Add auto_extend_select_list argument in buildSQLQuery() and use alias in group_by_expression and order_by_expression."
Revert "Support group_by_list and order_by_list in WebSection_getDocumentValueList."
Revert "Support order_by_list in Resource_zGetMovementHistoryList."

No need to extend SQLCatalog parameter to support group_by_list and order_by_list in sub-queries.

Add auto_extend_select_list argument in buildSQLQuery() and use alias in group_by_expression and order_by_expression.

If True, select_list is automatically extended to have columns used in
group_by_list and order_by_list. It is useful when use
select_expression in inner query and use group_by_expression or
order_by_expression in outer query.

Also, extract unwanted kw entries by just declaring them as separate
parameters.

getPhysicalPath is too slow to be called this often.
Flush cache on object clone, which is only useful as an after-renaming
event here.

Use pprofile instead for your profiling needs.

Also, get rid of single-use locals and unneeded list copy.

There are a lot of misses in this function because related keys must have
precedence over homonym columns, which makes it cost a *lot* of cpu time
to just return None.

Not perfect yet (what if definitions change during the same transaction ?),
but better than previous state.

Several ZSQLMethods do tests on parameter values, causing other parameter
values to be discarded (ie not indexed). A pseudo-code example is
  if isMovement[x]:
    append(getPrice[x])

- wrong default uid was used when migrating a site that never use portal_ids
  to generate security uids
- invalidate transactional cache of security uids