asSQLExpression() failed when executed several times on the same instance,
because too much code was skipped when 'column_map' is cached.

Avoid getting the list twice, follow coding style, explicitly access portal
methods on portal.

Remove condition to test if the event path destination predicate applies for this destination. On this this case, we can trust on the previous selection using the domain

call getObject() once explicitly to reduce number of calls.

this code was working just by chance because active_knowledge_pad is a brain, not an ERP5 object.

Revert "fix undefined variables"

This reverts commit 77a53809.

Revert "Listbox: calculate a mapping uid -> object once"

This reverts commit 84029c01.

Revert "fix list method returning duplicate uids"

This reverts commit 386b0e6b.

Revert "test for dialog listbox with editable fields"

This reverts commit e046d8f5.

Instead of looping through the list each time.
Also check that the list method does not return duplicate uids.

This cache is used by AccountingTransactionLine_getNodeItemList

It is useless to clear the cache in activities, because this would only work when using only one zope instance. Instead, clear the cache in the test for now.

Also do not hardcode 'business_process_module'.

>>> self.closure_cache[business_link] = new_business_process
was done after 'business_link' was overridden by the for loop.

This also the following line in getBusinessLinkClosure():
  if simulation_path.startswith(path):

For example, '/erp5/portal_simulation/1/10'
should not match '/erp5/portal_simulation/1/1'