See merge request nexedi/erp5!1723

The changes from ae15e7e1 (accounting: show the context when
showing invalid bank accounts, 2022-12-21) were a regression because
with the previous behaviour, by just looking at the transactions,
users could see with the ??? that something was wrong with the bank
account. After these changes they had to open the select element.

This restore somehow the previous behaviour by displaying the label
with ??? marker, but since we have the label, we display the
label (bank account reference) instead of the value (the relative URL)

This was working only with Organisations

Fix small problems displaying bank accounts on accounting transactions and be more strict during validation:
 - bank account must be validated
 - bank account must belong to the section

The latter was a problem for scenarios like this:
 - create transaction for supplier 1 with supplier 1 bank account
 - clone transaction
 - change to supplier 2
the bank account remains as supplier 1 bank account, although it became an invoice from supplier 2.
The transaction could be validated as long as the bank account is not invalidated

See merge request nexedi/erp5!1713

See https://developer.mozilla.org/en-US/docs/Web/HTML/Element/script/type

The only check with accounting transactions regarding bank account
was that the bank account is not invalidated. This makes the
constraint more strict by checking that the bank account belongs to
the corresponding entity and also that the bank account is validated.

A few tests needed to be updated to validate the bank accounts. Also
a legacy sequence test has been removed because it is now covered by
normal tests.

The parameter is still named `organisation` for "API stability", but
this parameter can be a person.

This was already used by the fields on accounting transaction for
source/destination_payment and more or less working already, except
for the case where the person had a subordination to an organisation
member of the group, in that case instead of showing the bank accounts
from the person this was showing the bank accounts from the
organisation

On a transaction without section / mirror section, on the profile tab
all accouts where displayed

drop skip_invalidated_bank_accounts option, we are now using
base_category instead.

erp5_accounting, erp5_payment_mean: display currently selected Bank Account even if it is invalidated.

no need to validate them in the test

We had nexedi/erp5@a6b588e7 but this was only solving the case of a trailing ;

See merge request nexedi/erp5!1710

See nexedi/erp5@0c2dd107

Coding crime.

See nexedi/erp5@cbb3b35b

`python` command doesn't exist anymore in most recent distribution and
our PASS (theia), but python3 has been available for years now.

See merge request nexedi/erp5!1719

https://lab.nexedi.com/nexedi/erp5/blob/bb7d107124e96a6176f56226d3093d8845c3949d/product/Formulator/Form.py#L228

See merge request nexedi/erp5!1718

context is actually a web section

replaces nexedi/erp5!1691 

See merge request nexedi/erp5!1709

See merge request nexedi/erp5!1715

There is no good reason to prevent editing in cancelled state and
it can be needed, for example to remove a cancelled payment
transaction from a payment transaction group.

This is different from the total of durations of test result lines
because they are executed in parralel on different testnodes. The
actual duration is just the difference between start and stop, so
that we don't have to calculate.

This makes it easier to understand the duration in minutes and hours

Ran https://github.com/python/cpython/blob/3.7/Lib/lib2to3/fixes/fix_asserts.py after adding the missing methods (assertRegex and assertRaisesRegex) on ERP5TypeTestCase.

Also ran a custom fixer rewriting
 
```py
self.assertTrue(a in b)
```

into

```py
self.assertIn(a, b)
```

See merge request !1708

Prepare for https://github.com/zopefoundation/Zope/pull/1075 ( Zope 4.8.7 )

See merge request nexedi/erp5!1707