- 02 Oct, 2018 2 commits
-
-
Vincent Pelletier authored
Subsequent activities will produce an empty dictionary, causing inconsistencies in activity dependencies.
-
Vincent Pelletier authored
It is set in constructInstance from received activate_kw, but is then if ActiveObject.activate is not used - as is the case when reindexObject is made an alias of recursiveReindexObject (ex: ERP5.Document.Delivery), as the latter calls recurseCallMethod which uses ActivityTool.activateObject which (purposefully, visibly) ignores getDefaultActivateParameterDict (unlike ActiveObject.activate). Also, do not step on activate_kw, for readability.
-
- 01 Oct, 2018 1 commit
-
-
Thomas Gambier authored
/reviewed-on nexedi/erp5!757
-
- 28 Sep, 2018 6 commits
-
-
Sebastien Robin authored
-
Sebastien Robin authored
The timeout of 10 seconds was too short for some machines
-
Vincent Pelletier authored
Drop failed activities instead of retrying execution and expecting it to both fail and do something (execute alarm, then index result).
-
Sebastien Robin authored
-
Romain Courteaud authored
This also reverts commit f3519857.
-
Jérome Perrin authored
/reviewed-on nexedi/erp5!756
-
- 26 Sep, 2018 3 commits
-
-
Thomas Gambier authored
/reviewed-on nexedi/erp5!753
-
Jérome Perrin authored
-
Jérome Perrin authored
-
- 25 Sep, 2018 5 commits
-
-
Thomas Gambier authored
After clicking on "Documents", we need to wait for the listbox to be loaded before doing anything else. In particular, if we do a search after clicking on "Documents" the listbox is loaded twice (once after clicking on Documents and once after clicking on search). If we don't wait for the listbox to be loaded before doing the search, the search macro might end too early.
-
Thomas Gambier authored
Add manifests to make the app a progressive webapp. There are 2 manifests : one during the installation process (while the app is loading) and one for the app itself. Note that the "start_url" parameter is made for the officejs app URL ([your_app].app.officejs.com/[release_number]). It doesn't work on an ERP5 instance ([instance_url]/erp5/web_site_module/[your_app]). If you want to make it work on the ERP5 instance for testing purposes, remove one "../". Update the Javascript Editor logo according to Vincent's wishes.
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Thomas Gambier authored
The app will be labelled "Javascript Editor" for now but the domain name will be codemirror.app.officejs.com. Update dropbox_app_key (generated by Vincent). Reorder totally the apps on officejs appstore (as suggested by Vincent).
-
- 24 Sep, 2018 1 commit
-
-
Vincent Bechu authored
/reviewed-on nexedi/erp5!754
-
- 21 Sep, 2018 9 commits
-
-
Sebastien Robin authored
-
Tristan Cavelier authored
-
Thomas Gambier authored
Don't change anything, just make it pass the linter.
-
Thomas Gambier authored
/reviewed-on nexedi/erp5!750
-
Nicolas Wavrant authored
In web_site_module/predecessor_test_web_site/section_with_content_page. This publication_section was already sel on the web section (but seemed not active ??), but not on the web page. Then this commit just adds the missing category and update the web page
-
Romain Courteaud authored
-
Romain Courteaud authored
Those classes were used by JQuery Mobile.
-
Georgios Dagkakis authored
-
Thomas Gambier authored
The tests are exactly the same as for text editor (except there is no "migration test" which was not really to test TextEditor)
-
- 20 Sep, 2018 8 commits
-
-
Thomas Gambier authored
Codemirror app is now using the same mechanism as text_editor. For now, it has exactly the same features as text editor but it will create "Web Script" instead of "Web Page" documents. The codemirror mode is forced to "javascript" and the content type of the document if forced to "application/javascript".
-
Ayush Tiwari authored
DiffTool: removePropertyList function should be working same for both ERP5 object and dictionaries of ERP5 object properties Before we use to neglect removing some properties from ERP5 object properties' dict, which led to error in case of use of DiffTool for Historical Revisions dict. Thus, its important for removeProperties function to act similar in both cases. Also, add `base_data` in attribute_set to be removed because `data`, `base_data` and `text_content` are same
-
Ayush Tiwari authored
This is essential to have better aesthetic of Deepdiff diff for tuples. In general, we treat iterables as long string so as to bypass the complicated diff for iterables provided by DeepDiff, but for tuples this was creating a problem as it was not possible to understand the diff properly. Hence, by soting them, we get more understandable diff for tuples. Also, there is no need to override `__diff_tuple` function as it is itself dependent on `__diff_iterable` which we already override.
-
Ayush Tiwari authored
Following convention, it makes more sense to use view mode for DiffTool view. Also, we don't need selection as of now. Also, update the tests
-
Ayush Tiwari authored
View wise, the date and path for new version/obvject should be on right side which in our case is `your_second_date`. Thus, the case where we run diff action on single object, the right side should show the current version and the left side shoule show the last version.
-
Richard authored
/reviewed-on nexedi/erp5!741
-
Jérome Perrin authored
/reviewed-on nexedi/erp5!748
-
Yusei Tahara authored
-
- 19 Sep, 2018 1 commit
-
-
Jérome Perrin authored
/reviewed-on nexedi/erp5!739
-
- 18 Sep, 2018 4 commits
-
-
Vincent Pelletier authored
-
Jérome Perrin authored
/reviewed-on !740
-
Jérome Perrin authored
zope.conf default encoding is also used for responses and converters, so for consistency we should also force default encoding there. This is especially useful when running tests, because zope.conf is not used and default values are used.
-
Jérome Perrin authored
-