- 05 Feb, 2019 4 commits
-
-
Julien Muchembled authored
The original goal was to improve performance by removing the `processing_node_processing` index and the queries that modified these 2 useless columns.
-
Julien Muchembled authored
The root call to getExecutableMessageList (i.e. the one from distribute) is fast enough and won't hold old revisions of the database for too long. It is also completely read-only so it won't lock anything. This caused useless communication with the server.
-
Julien Muchembled authored
-
Julien Muchembled authored
As shown in the following example, on a big catalog table, MariaDB is able to use several indices at the same time ('...' are obfuscated unique values): > analyze select SQL_NO_CACHE uid, relative_url from catalog where reference='...' OR relative_url='...'; +------+-------------+---------+-------------+------------------------+------------------------+---------+------+------+--------+----------+------------+--------------------------------------------------+ | id | select_type | table | type | possible_keys | key | key_len | ref | rows | r_rows | filtered | r_filtered | Extra | +------+-------------+---------+-------------+------------------------+------------------------+---------+------+------+--------+----------+------------+--------------------------------------------------+ | 1 | SIMPLE | catalog | index_merge | Reference,relative_url | Reference,relative_url | 768,767 | NULL | 2 | 2.00 | 100.00 | 100.00 | Using union(Reference,relative_url); Using where | +------+-------------+---------+-------------+------------------------+------------------------+---------+------+------+--------+----------+------------+--------------------------------------------------+ 1 row in set (0.00 sec) So mixing different dependency types with OR should be fine (no need to split into more subqueries and join with UNION).
-
- 18 Jan, 2019 3 commits
-
-
Julien Muchembled authored
-
Julien Muchembled authored
-
Julien Muchembled authored
-
- 13 Jan, 2019 1 commit
-
-
Julien Muchembled authored
This fixes: Traceback (innermost last): Module Products.CMFActivity.ActivityTool, line 1373, in invokeGroup traverse(method_id)(expanded_object_list) Module Products.ERP5Catalog.CatalogTool, line 946, in catalogObjectList super(CatalogTool, self).catalogObjectList(tmp_object_list, **m.kw) Module Products.ZSQLCatalog.ZSQLCatalog, line 813, in catalogObjectList **kw TypeError: catalogObjectList() got an unexpected keyword argument 'group_id'
-
- 08 Jan, 2019 3 commits
-
-
Julien Muchembled authored
-
Julien Muchembled authored
/reviewed-on !819
-
Jérome Perrin authored
* Set MAKEFLAGS with -l to limit number of jobs by load and number of processes * set NPY_NUM_BUILD_JOBS & BUNDLE_JOBS to `node_quantity` instance parameter /reviewed-on !809
-
- 07 Jan, 2019 2 commits
-
-
Sebastien Robin authored
-
Julien Muchembled authored
Originally, uids somehow sorted messages by date of insertion, in particular for those that were created within the same second. But since random uids, such messages became validated or processed in random order. Note however that by default, messages created in the same transaction all have exactly the same date, so commit a42da4de ("CMFActivity: Do not use offset for scanning messages to validate.") forces us to keep the ordering on uids (in addition to priority/date). Existing instances will upgrade automatically, using the already existing code to upgrade tables in a generic way. You should see the following logs: INFO CMFActivity 'message_queue' table upgraded ALTER TABLE message_queue MODIFY COLUMN date datetime(6) NOT NULL AFTER uid, MODIFY COLUMN processing_date datetime(6) DEFAULT NULL AFTER processing INFO CMFActivity 'message_job' table upgraded ALTER TABLE message_job MODIFY COLUMN date datetime(6) NOT NULL AFTER uid, MODIFY COLUMN processing_date datetime(6) DEFAULT NULL AFTER processing INFO CMFActivity 'message' table upgraded ALTER TABLE message MODIFY COLUMN date datetime(6) NOT NULL AFTER uid, MODIFY COLUMN processing_date datetime(6) DEFAULT NULL AFTER processing /reviewed-on !820
-
- 02 Jan, 2019 1 commit
-
-
Georgios Dagkakis authored
so that options are kept
-
- 31 Dec, 2018 1 commit
-
-
Romain Courteaud authored
Same behaviour than xhtml style
-
- 21 Dec, 2018 2 commits
-
-
Julien Muchembled authored
-
Romain Courteaud authored
Next step is to activate more contextual listbox actions. Move listbox select button to the right, to simplify cancellation.
-
- 20 Dec, 2018 2 commits
-
-
Romain Courteaud authored
-
Romain Courteaud authored
-
- 17 Dec, 2018 4 commits
-
-
Romain Courteaud authored
WebSite redirection must be tested against different virtual host monster configuration TODO: No idea how to fetch the ipv6 in live test
-
Romain Courteaud authored
-
Jérome Perrin authored
https://lab.nexedi.com/nexedi/slapos/tree/master/software/proftpd/test was just named "test", we want the test result to be named "proftpd" instead. /reviewed-on nexedi/erp5!815
-
Nicolas Wavrant authored
-
- 14 Dec, 2018 2 commits
-
-
Romain Courteaud authored
See nexedi/erp5@04939506
-
Vincent Pelletier authored
Since recent rework of isSubtreeIndexable, direct indexation of a trash bin is fixed, but recursive indexation recursion condition broke: it stop just above the trash bin instead of stopping just below it.
-
- 13 Dec, 2018 1 commit
-
-
Arnaud Fontaine authored
Revert "mark file uploading tests as expected failure." as these Functional Tests are not supposed to fail. This reverts commit ade16831. enablePrivilege, and thus UniversalFileRead used to upload files, was disabled in Firefox 17. Since SlapOS Firefox has been upgraded, "The operation is insecure" error is raised when uploading files. However, this doesn't mean that these tests should be expected to fail (and if they are, they should be removed instead).
-
- 12 Dec, 2018 1 commit
-
-
Jérome Perrin authored
This workflow involved an Assignee who can open, close, re-open and an Assignor who can close definitively. This is usually configured so that accountants are Assignee and CFO is Assignor. We realized that re-opening a Period that was previously closed is something we don't want the accountants to do without CFO's approval. To support this configuration, we only allow Assignor to re-open. Now Assignee can open and close temporarily and Assignor can re-open and close definitively. /reviewed-on nexedi/erp5!813
-
- 11 Dec, 2018 4 commits
-
-
Romain Courteaud authored
This is really a hack, which may be integrated into renderJS directly Reduce potential Zelenium test timeout by waiting for iframe to be loaded one by one
-
Vincent Pelletier authored
This check got lost when the first isSubtreeIndexable call was done on parent document and not on self.
-
Vincent Pelletier authored
This partially reverts: commit 76e3c115 Author: Vincent Pelletier <vincent@nexedi.com> Date: Mon Dec 10 16:40:48 2018 +0900 Base: Fix isAncestryIndexable implementation. as it accidentally carried over a totally unrelated (and unfinished) change.
-
Vincent Pelletier authored
Start checking isSubtreeIndexable on parent document, not on current document. This bug broke Trash Bins indexation (and should not affect anything else, as only Trash Bins prevent content indexation currently). Extent trash tool unit test to verify actual indexation, and not just indexability API.
-
- 10 Dec, 2018 5 commits
-
-
Romain Courteaud authored
Fixup nexedi/erp5@559c5e8b
-
Romain Courteaud authored
See nexedi/erp5@994d8369
-
Vincent Bechu authored
/reviewed-on nexedi/erp5!768
-
Arnaud Fontaine authored
This should be tested even if it currently fails with "The operation is insecure" when trying to upload local ODS files.
-
Arnaud Fontaine authored
erp5_configurator_standard_ui_test: Fix testFunctionalConfigurator failures with erp5testnode (b955dbe0). The following error occurred because setupAutomaticBusinessTemplateRepository() was called and only added bt5/ directory but not bootstrap bt5s directory: BusinessTemplateMissingDependency: While analysing erp5_dhtml_style the following dependency could not be satisfied: erp5_xhtml_style () Reason: Business Template could not be found in the repositories Instead assume that bt5 repositories are set up properly. Also, running through erp5testnode requires apache-backend Timeout to be increased as waiting for upgrader activities to be processed can take ~15 minutes (!377).
-
- 07 Dec, 2018 2 commits
-
-
Ayush Tiwari authored
[erp5_core] Follow guidelines for Actions on portal types Catalog and Catalog Tool by adding dialog before executing Also, Cleanup useless actions and functions /reviewed-on nexedi/erp5!811
-
Jérome Perrin authored
* Assorted small fixes so that form comply to conventions * Minimal fixes of invalid html * make `mixin` usable in type definition The context is I was working on a custom configurator involving some matrixboxs, so I needed a simple business configuration item just to store the matrix result. This was possible with mixin, I just made a portal type based on Item class, enabling mxins for `Matrix` and `BusinessConfiguratorItem` and I did not have to define a class ( not sure this is really better ). Then the matrixbox was not rendering properly, because browser was confused by the broken html and build a "different DOM" and selectors did not apply. This is what `configurator: fix (a bit) invalid HTML` addresses. While doing this, fix some minor problems found with forms. /reviewed-on nexedi/erp5!814
-
- 06 Dec, 2018 2 commits
-
-
Jérome Perrin authored
Insert fields in a <div>, not in a <table>, fields are not valid table elements (they are not <tr>). Configurator still produce invalid HTML, this seem to be the minimal fix to have matrixboxs rendered properly without having to rewriting all configurator page templates.
-
Jérome Perrin authored
So that all string fields have same width
-