(not yet for scripts under portal_skins)

adjusted (by Jérome) to just remove call to print in a few places where
it made more sense.

Rename triggerParachute function into land in DroneManager for
compliance with multicopters APIs.

I investigated a case where activities calling an external API were not
retried after the initial failure, when it was caused by
TimeoutReachedError:

https://lab.nexedi.com/nexedi/erp5/-/blob/f41b33cd91348a800db3b95b30efd857f583bcde/bt5/erp5_web_service/MixinTemplateItem/portal_components/mixin.erp5.RESTAPIClientConnectorMixin.py#L142

The interesting thing is that these activities are written in a way that
allow them to be retried (automatically, or manually) without causing
un-intended side effects, like creating twice a document in the external
API. So not having SQLBase retrying them automatically is a pity.

The bug is that the use of TimeoutReachedError here was intended only
for the activities that are aborted by CMFActivity itself, and not for
the userland code that can also use TimeoutReachedError.

This commit removes this special processing, as there is no need to
especially not retry the activities that trigger the CMFActivity
timeout. The max_retry decision should be taken by the calling code, and
not by CMFActivity.

   Without this the redirection places value unquoted causing parser to fail
   later on when resolve {n.me}.

   came_from cannot be None so we simplify the block a bit.

 Move Simulation to priority 3 helps to not block/delay immediateReindex
 on a scenario where everything is expanding. We use priority 3 to match
 what is used in SimulableMixin.updateSimulation and Delivery.localBuild

for consistency and because it's probably good to let user choose
sending mode.

This matches the form title

When producing DSN report, there was a few places where we depended on
dict iteration order, so the test did not pass when running with
PYTHONHASHSEED (erp5_payroll_l10n_fr:testDSNSocialDeclarationReport
fails with PYTHONHASHSEED 323). In these changes we add sorts to some
places where we were iterating on dicts so that the order of the Lines
is deterministic even with hash randomization enabled.
Co-authored-by: Nicolas Wavrant <nicolas.wavrant@nexedi.com>

test was simplified not to use difflib, unittest does a good job of
explaining the differences when using splitlines on long texts, no need
to compute a diff ourselves.

This makes it easier to find the problematic listbox and also report
once per listbox.

The XML had incorrect categories to non existent paths, fixing the XML
to remove these paths made the warning disappear.

nested warns where it is used and the modern approach for this is
contextlib.nullcontext

EntireQuery pass a PlaceHolderTableDefinition for almost all catalog
searches, so "from_expression" itself can not be deprecated at this
point

We were importing from an old alias in App.class_init, but this module
has been removed from Zope5

This was too late to enable all warnings, many modules were already
imported and already triggered lots of deprecation warnings that were
ignored.

Also, we no longer need to patch warnings.showwarning because we use
logging.captureWarnings(True) in runwsgi. As a result, the log events
will be slightly different, because logging's way of logging warnings
is a bit different.

test_warnings_redirected_to_event_log no longer make sense because we
only use logging.captureWarnings() in runwsgi, while running tests we
want warnings to be displayed on the console so that developer see them.

In runUnitTest environment this is not set up.