- 23 Apr, 2021 1 commit
-
-
Arnaud Fontaine authored
This also moves all Configurator Workflows in workflow_module to portal_workflow (workflow_module was an implementation of Workflows based on ERP5 objects and not using DCWorkflow code). * Workflows are now defined on on portal_workflow._chains_by_type anymore but, as everything else, on the Portal Type itself. * portal_workflow can contain and work at the same time with legacy and new Workflows (ERP5Type/patches/DCWorkflow.py monkey-patching DCWorkflow classes to provide the same API). * Existing Workflow Scripts should work as they are and the code can be updated later on to take advantage of the new API: + With legacy implementation Workflow {Scripts,Transitions,Worklists,States} were in a Folder ({scripts,transitions,worklists,states} attribute) but all of these are now in the Workflow itself and their IDs are prefixed (PropertySheet-style), for example `script_`. Legacy attributes are provided in new implementation to call the new API. + When calling a Workflow Script, `container` was bound to its parent, namely WF.scripts (Folder) and a Workflow Script could call another. Now `container` is bound to the WF itself and Workflow Scripts are in a Workflow directly. New implementation `scripts` attribute handle such use case. + Override portal_workflow.__getattr__ so that a Workflow Script can call another one without prefix. * Worklist are Predicate: Worklist filter objects based on given criterions and thus it makes more sense for a Worklist to be a Predicate (albeit a Predicate with only Identity Criterion and nothing else). + Criterion Properties: * state_variable. * local_roles (SECURITY_PARAMETER_ID). * Any Workflow Variables with for_catalog == 1. erp5_performance_test:testWorkflowPerformance were ran to compare DCWorkflow and ERP5Workflow implementations and it seems to be about 4% slower with the new implementation (legacy: 7.547, 7.593, 7.618, 7.59, 7.514 and new: 7.842, 7.723, 7.902, 7.837, 7.875). Work done by Wenjie Zheng, Isabelle Vallet, Sebastien Robin and myself.
-
- 21 Apr, 2021 1 commit
-
-
Arnaud Fontaine authored
-
- 19 Apr, 2021 2 commits
-
-
Jérome Perrin authored
Since nexedi/slapos@0aaa1ccd cloudooo fonts are now configured correctly. The snapshots from these tests have been expecting the old, wrong fonts. This update snapshots to use the new fonts, so that we can update production cloudooo used for tests.
-
Jérome Perrin authored
In e559ecd5 ([erp5_crm] Activate actions for ERP5JS, 2019-09-25) we introduced a regression, support for erp5_xhtml_style got broken in "Create Response" when enabling ERP5JS. It was fixed later in 3125590f (fixup! [erp5_crm] Activate actions for ERP5JS, 2020-10-23). This extends test coverage for: - "Create Response" in erp5_xhtml_style - "Create New Event" in both ERP5JS and erp5_xhtml_style See merge request !1397
-
- 16 Apr, 2021 1 commit
-
-
Jérome Perrin authored
When rendering error page, the default header_title is computed in global_definitions and this is done by concatenating string:ERP5 (which is unicode) and the title of the portal (which is an UTF-8 encoded string), which causes an UnicodeDecodeError, because the portal title was decoded as ASCII. Because header_title is usually an UTF-8 encoded string, also use an UTF-8 encoded string for 'ERP5' default value.
-
- 15 Apr, 2021 8 commits
-
-
Kazuhiko Shiozaki authored
-
Jérome Perrin authored
Following current convention of having x_ui_test for the framework and the xhtml style tests and x_renderjs_ui_test for the ERP5JS version of the tests.
-
Jérome Perrin authored
-
Jérome Perrin authored
to ensure no regressions in legacy UI
-
Jérome Perrin authored
-
Jérome Perrin authored
When the preferred text format is set, the default value of the text editor for the reply body shows the original message quoted.
-
Jérome Perrin authored
Also remove a non applicable copy-pasted comment
-
Jérome Perrin authored
-
- 14 Apr, 2021 9 commits
-
-
Xiaowu Zhang authored
See merge request !1395
-
Arnaud Fontaine authored
Revert "erp5_officejs: erp5_hal_json_style Skin is used and without it erp5_officejs cannot be installed." This reverts commit 1b706b6c. erp5_officejs depends erp5_web_renderjs_ui which in turn depends on erp5_hal_json_style.
-
Xiaowu Zhang authored
There is no such as thing as "Long Title" in erp5 There is "Title" and "Short Title"
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
-
Jérome Perrin authored
-
Jérome Perrin authored
because this business template uses fields from erp5_trade field libraries
-
Jérome Perrin authored
-
Jérome Perrin authored
These fields were created when proxify action was broken. This re-exports the fields after using fixConsistency on them. see also nexedi/erp5!1352
-
- 13 Apr, 2021 6 commits
-
-
Kazuhiko Shiozaki authored
also add rel="noopener noreferrer" for security.
-
Kazuhiko Shiozaki authored
-
Vincent Pelletier authored
It seems this workfliow was created by copy-pasting event_workflow, without cleaning these variables up: they are used by a transition dialog on event_workflow but nothing on this workflow. Still, they are stored on every single document on every single transition, wasting resources. So remove them.
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
erp5_jquery_sheet_editor: unset Accelerated HTTP Cache Manager configuration in intermediate template.
-
- 12 Apr, 2021 3 commits
-
-
Kazuhiko Shiozaki authored
-
Vincent Pelletier authored
-
Jérome Perrin authored
-
- 09 Apr, 2021 4 commits
-
-
Nicolas Wavrant authored
See merge request nexedi/erp5!1385
-
Nicolas Wavrant authored
-
Nicolas Wavrant authored
So different test suites can have a same named sequence without theme colliding with each other
-
Nicolas Wavrant authored
-
- 07 Apr, 2021 1 commit
-
-
Arnaud Fontaine authored
-
- 06 Apr, 2021 1 commit
-
-
Jérome Perrin authored
-
- 02 Apr, 2021 2 commits
-
-
Jérome Perrin authored
When installing the tested software failed, no test result lines have been created, this should have status FAILED.
-
Jérome Perrin authored
person reference contained the username, so test was also passing, but what we really want to test is that the username is in the mail body.
-
- 01 Apr, 2021 1 commit
-
-
Jérome Perrin authored
ee64d50c (patches/CookieCrumbler: update patch for CMF 2.3, 2019-09-09) introduced a NameError, getRequest was not imported.
-