erp5_pdm: use preferred category child item list method in my_variation_category_list field library.

Python script needs to be recompiled when they comes from a change of
class. This was done when business template was installing scripts, but
not when automatic migration from old catalog to new catalog was done.
So make sure to call _compile directly in changeObjectClass to make sure
Python Scripts would always work after a change of class.

Replace the airplane icon by a search button if not value is set on the field.
This make it more usable on mobile, where it's easier to browse than typing text.

Regression introduced by nexedi/erp5@cf1a0610

Calling these can break catalog by not caring about concurrent indexation
activities. Even non-restricted code should not call these, but:
- they cannot be reliably renamed, as there exist activity dependencies
  looking for these exact names (which is a bad practice on its own)
- there is no simple access control mechanism for non-restricted code
So rely on developer discipline instead.

Also, simplify scripts while doing so.

These methods must not be called synchronously:
- they can break catalog by not being careful enough about other
  reindexations which may happen in parallel. See the serialization_tag
  mechanism for more.
- indexation gets executed in the security context of the user causing the
  call, which may lead to an indexation result different from what happens
  when indexation happens with an all-accesses user.

To remove the need to directly call immediate reindexation methods,
allowing to prevent further misuses of these.

Do not check portal's isIndexable, it is checked again in ZSQLCatalog.
Directly bind superclass callable instead of doing an extra getattr +
call.
Trivial python simplifications.
Assorted coding style improvements.
Drop unhelpful comments and docstrings.

getSecurityQuery returns None when caller is superuser. Avoid passing it
to buildSQLQuery, as it would mistake security_query for a column name.

Float fields are needed to round amounts.
Also, discard empty form groups.
Also, make "hidden" the first group, so any created field gets hidden by default
(which is what we want for all listbox "editable" fields).
Also, remove overrides which seem unneeded for report appearance:
- "lines" is not used for OD* rendering
- "columns" is already the same as in original field
"editable_columns" is used in odt (and maybe in ods, unclear), but
inconsistently with its intended use... so leave it, sadly.

Fixes bug /#/bug_module/20171211-153B610

/reviewed-on nexedi/erp5!526

Removed code from not implemented and not needed compatibility layer.

/reviewed-on nexedi/erp5!525

/reviewed-on nexedi/erp5!527