Mixed commit:

  43b1ae1b
  2f05fe1b

Returning true allows to add movements to existing one, and this group shall
always separate them.

uid is used internally during recursive calls and using uid can lead to
traverse all lines of catalog.

Note: Generic code shall have no constraints at all.

Conflicts:

	bt5/erp5_accounting/bt/revision

If 2 lines have the same effective date (catalog has a 1 second precision),
always return the validate and open first.

Commit: 02d06501

Changes applied from history:

http://git.erp5.org/gitweb/erp5.git/history/refs/heads/interaction-drop:/product/ERP5/Document/SubscriptionItem.py?js=1

Original commit from : Lukasz Nowak <luke@nexedi.com>
                       3e45ec35

From Lukasz Nowak:

1- Compensation is undesired. (51c8a250)

  Raise loudly with message.

2- Disallow any compensation. (3c74ed07)

3- Emit more informative log. (534b2e77)

Also minimise activity hurricane by calling expand directly.

Open Orders are searched using indexation timestamp, which allows to see them
in "windows", and does not repeat expand if not needed.

Generate activities and allow to pass the tag. Use search and activate
everywhere which will allow to walk through objects without killing the cluster
even in case of really big documents.

Avoid calling isDivergent which can take few minutes to finish.

causality_state comes from well designed causality workflow, which informs
enough about delivery state. Fetching causality_state property is extremely
fast.

Notes:

Do nothing on 'calculate' instead of disabling *_causality_interaction_workflow as
these interaction workflows do not generate activities directly, and do work
that can't be postponed.

By contrast, 'calculate' transition of delivery_causality_workflow is modified
so that no 'updateCausalityState' activity is created by default.

This commit also update list of method_id which should not call calculate_causality

Conflicts:
	bt5/erp5_crm/bt/revision

Because most of the times we configure roles so that users assigned to a group can only see documents from this group, it's a better default that in reports they can only select the groups they are affected to (and the sub categories of these groups).

For more complex security configurations, the script `Base_getDialogSectionCategoryItemList` can be customised.

See merge request nexedi/erp5!1136

"Split and Defer" action

![screenshot of split_and_defer transition](/uploads/2a9d4efc139b881bfec82a425996c659/image.png)

functionality can be achieved with "Solve Divergences" action with a "Split and Defer" 

![screenshot of solve_divergence transition choosing split and defer action](/uploads/704c4713d842455c4e369a99d8f4867a/image.png)

we don't need two ways of doing the same thing.

This removes the workflow transitions and the dialogs and update the tests to use the "Solve Divergences" action instead.

See merge request nexedi/erp5!1148

Main goal was to make erp5_trade 100% translated, also the divergence explanations.

The result of the solve divergence dialog is:

![solve divergence dialog in french](/uploads/7ba5d3fcccb58b223471566d528c5c0a/image.png)

This also fixes a few small problems with existing translations

See merge request nexedi/erp5!1149

Fix a problem with `WebPage.getImplicitSuccessorValueList`, it did not detect references such as:

```
<img src="reference">
```

when the web page was in the context of a web site, because of some wrong relative URL calculations.

See merge request nexedi/erp5!1153

Following up nexedi/erp5!941 , use a script ( nexedi/erp5$444 ) to rewrite most of XML.

The script was not so clever and skipped several cases that would be hard to implement.


See merge request nexedi/erp5!1156

'NoneType' object is not iterable

Since 05210e87 (erp5_web_renderjs_ui: add LabelField support,
2020-05-26) we have label field support, so this labelfield is no longer
rendered with the fallback "read only field" gadget and markup is
different. Update selector to make the test happy.

this is useful for an intermadiate dialog that does not require submit
button but still top 'cancel' link works.

Fixes test failure:

    ======================================================================
    FAIL: testNamingConvention (testERP5NamingConvention.TestNamingConvention)
    ----------------------------------------------------------------------
    Traceback (most recent call last):
      File "./parts/erp5/Products/ERP5Type/tests/testERP5NamingConvention.py", line 120, in testNamingConvention
        self.assertEqual(0, len(final_result_list), "\n".join(final_result_list))
    AssertionError: erp5_ui_test/Foo_viewParallelListField : Title of the Form itself : 'Foo_viewParallelListField' is not titlecased

this report got broken in 2178cbed (pdm: fix pylint messages and
enable coding style test, 2020-03-09) there's no "Base" portal type.

This report has no test coverage, but the bigger problem is that the
graph from this report is not readable.

here as well we need to have body scripts after body for same reasons as
6b8e263b (monaco_editor: put body before head, 2020-06-01)

This fixes testDynamicClassGeneration failure since the migration of
IMovement (e8e5f273) which uncovered this issue.

ZODB Components: pylint: Generate astroid Module AST once for both the Component Module and its versioned Module.

This reverts commit 5b93b30f.

if such error occurs, this listfield show "You selected an item that was not in list" error message when save

See merge request nexedi/erp5!1155

- Add a Post interface so that we can use `Post` as a prefix for skins.
- fix pylint messages
- enable coding style test

See merge request nexedi/erp5!1152

See merge request nexedi/erp5!1150

Display a ParallelList field as a list of sub fields calculated with the hash script.

Keep previous HAL properties to keep compatibility for client without the JS implementation.

Base_extractReferredObjectDictFromHtml except that base url to have a
leading /, because we ignore the first character of the normalized URL
returned by Base_normalizeUrlPathname.

By using / as base URL when we are in the context of the website, the
paths passed to Base_normalizeUrlPathname always start by / so we can
safely drop / from the normalized path returned by
Base_normalizeUrlPathname

because relative URL calculations in referenced pages extractions is
different when the page is in the context of a web site, we also have to
test this case

- compare references not uids
- set self.maxDiff=None to see full diff