Required as preparation for migrating to ERP5 catalog which itself has an accessor
`isIndexable` which we plan to remain disabled all the time. So better to rely
directly on isIndexable from erp5 site object.

This would also remove the need of evaluating the globals every time.

/reviewed-on nexedi/erp5!314

on Organisation and Person property sheets

correct case is *validated*

- get tools from portal, not context, to prevent slow acquisition
- stop using abbreviated names (ctool, stool etc)

This is not public API and is not supposed to be used outside of catalog
tool.

Using getObject here is also wrong because it does not apply security
and does not check provided argument type.

To prevent retrieving too many documents, we usually pass limit=2 for
safety.

To detect inconsistencies, when only one result is expected we rely on
`brain, = portal_catalog(limit=2)` style unpacking to fail if more than
one result were found.

This was an error reported by pylint. This changes fixes the behavior, but should have been the original behavior;

to InternalInvoiceTransaction_getAuthenticatedUserSection.

It was a violation of our naming conventions.

This was a violation to our naming conventions

/reviewed-on nexedi/erp5!313

Revert "erp5_officejs_afs_directory: add missing allow cross domain header"
This reverts commit 2674afaa.

Revert "erp5_officejs_afs_directory: rewrite custom storages and automate openhub data retrieval via proxy"
This reverts commit f74f420e.