py2/py3: import InstanceDict from DocumentTemplate._DocumentTemplate instead of deprecated DocumentTemplate.DT_Util.

use urlsplit instead, prepending '//' like discussed in https://bugs.python.org/issue27485#msg270215

fixes errors on python3, File needs bytes

This not closed comment tag was probably a problem for Chameleon, this
is from a commit from Aurélien in 2021, adjusted by Jérome in 2023 (to
actually remove the code that were commented out, not just removing the
comment and leaving the code)

New cloudooo uses text/csv as mimetype for CSV, which is correct, but this revealed issues in ERP5:
 - ERP5 did not detect text/csv for csv extensions
 - downloading a file with content type text/csv caused an error

See merge request nexedi/erp5!1859

See merge request nexedi/erp5!1860

  in addition to None, id can be an empty list on this case

See merge request nexedi/erp5!1858

DownloadableMixin uses mimetypes_registry to guess an extension from
the mimetype, but this was wrong when the entry in mimetype only defines
globs, an error like this was raised:

      Module erp5.component.mixin.erp5_version.DownloadableMixin, line 143, in index_html
        output_format = mimetype_object.globs.strip('*.')
    AttributeError: 'list' object has no attribute 'strip'

Also blindly apply the same fix in OOoTemplate, as it used the same
problematic pattern.

we use to have two entries for csv:

  - text/comma-separated-values
     - mimes: ["text/comma-separated-values"]
     - extensions: ["csv"]
     - globs: []
  - CSV document
     - mimes: ["text/csv", "text/x-comma-separated-values", "text/x-csv"]
     - extensions: []
     - globs: ["*.csv"]

but text/comma-separated-values does not really exist, rfc4180 recommends
text/csv.

The problem with this configuration is that when ERP5 picks a mime type
for csv extension, it uses text/comma-separated-values, as this one has
extensions set.

Change the configuration to delete "text/comma-separated-values" and keep
everything in "CSV document":

  - CSV document
     - mimes: ["text/csv", "text/x-comma-separated-values", "text/x-csv", "text/comma-separated-values"]
     - extensions: ["csv"]
     - globs: ["*.csv"]

The previous pattern of using click and then pause is fragile and seems
to be the cause for test failures.

Instead of disabling in javascript, which cause the button to be visible
for a few milliseconds, disable directly in the HTML file.

Do not hide "document properties", because it can be useful and it does
not hurt and "edit images" because we don't hide the buttons to edit
annotations either, so this is consistent.

The setter does this already so no change is expected, but these are API
misuses and they just look wrong.

Also, modernise syntax a bit.

   The JSONEditor relies on call async load for load promises. The current code waits for the promises to finish and throw errors explicitly.

    This allows the renderjs properly handle the error on the same chain of promises.

See merge request nexedi/erp5!1855

it was removed during df2642cf (corporate_identity_test: update
snapshots for new cloudooo, 2023-11-29) because it looked unused, but it
was actually used.

   the update_catalog parameter is explicity destinated to the entries
   on the template_list, never to its dependencies resolved.

   The side effect was that reindexation is trigger to the whole list of
   business templates in the site.

These actions should be always displayed, not only when user can modify,
otherwise the action is no longer displayed once the transaction
becomes delivered.

also adjust test to use test name for expected image id

As utility to activate whenever we have a process going wrong and
we do not want disturb the user or stop (rollback) the process
but we do want developpers to be notified.

See merge request nexedi/erp5!1854