- 13 Jul, 2018 3 commits
-
-
Jérome Perrin authored
We use AttributeUnicityConstraint for this kind of checks nowadays. Also the way "context document" was accessed does not work in RenderJS UI
-
Jérome Perrin authored
This is the job of a constraint evaluated during workflow transition, no need for an external validator here.
-
Jérome Perrin authored
We have a rule "Do Not Test Required In Field View Form"
-
- 09 Jul, 2018 1 commit
-
-
Vincent Pelletier authored
manage_afterClone clears workflow history, which is precisely what "duplicate" is trying to avoid. This fixes a regression introduced in commit 7e810d5a Author: Vincent Pelletier <vincent@nexedi.com> Date: Tue Feb 20 17:06:46 2018 +0900 CopySupport: Factorise clipboard-related methods and code paths.
-
- 06 Jul, 2018 1 commit
-
-
Romain Courteaud authored
-
- 05 Jul, 2018 13 commits
-
-
Nicolas Wavrant authored
-
Ayush Tiwari authored
-
Ayush Tiwari authored
The Diff view in new UI will be using IFrames to display the diff. Also, add changes in tests: erp5_ui_test: Update history test according to use of diff gadget erp5_web_renderjs_ui_test: Update tests according to new Diff view for Historical Revisions testXHTML: Add Historical Comparison View to the list of listbox with no selection name testXHTML: Simplify test_html_file and include way to ignore single html file
-
Romain Courteaud authored
Language modification is done on another page, accessible from many places for now.
-
Xiaowu Zhang authored
-
Romain Courteaud authored
-
Xiaowu Zhang authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
- 04 Jul, 2018 3 commits
-
-
Vincent Pelletier authored
The original need to touch this comment is that it mentions now-removed Folder_reindexObjectList script. But also: - it mentions multi-ZEO setups, which are now discouraged if favour of using NEO - it mentions pre-conceived optimisations, which can be wrong and hence misleading (up-to-date measures are needed before deciding what to optimise) - it mentions a number of activity nodes which is nowadays exceeded without seeing any obvious issue caused by activity table deadlocks (which are nevertheless still likely an issue, just not yet breaking scalability)
-
Vincent Pelletier authored
These are now superseded by recursiveReindexObject.
-
Vincent Pelletier authored
This method does not belong to Folder (nor Base, even as an alias), as it recurses wihout any limit in a tree whose node count is unknown. Such generic recursion should either be deferred (split in multiple transactions, each processing a limited chunk of the tree) or not be recursive (which of course depends on the need). There is a single caller left (constructInstance) where the subtree size is assumed to be reasonable, as it was constructed inside current transaction. So essentially rename this method so it is only used in such conditions.
-
- 03 Jul, 2018 9 commits
-
-
Vincent Pelletier authored
isAncestryIndexable is a parent-recursive version of isIndexable: it will return False if any parent is non-indeable. This simplifies disabling indexation of an entire tree, from its root to its deepest leaves. isSubtreeIndexable allows preventing any subobject from being indexable, without having to mark each individual subobject as non-indexable. This is different from isAncestryIndexable, as it allows having an indexable root, but no children indexable. This is especially useful for Trash Bins, where the bin itself should be indexable (because there is no reason not to) but its content must not be indexed, while being left as it was found in its original location.
-
Vincent Pelletier authored
Should make Folder_reindexAll and most custom indexation methods obsolete. Remaining valid reindexation methods are: - reindexObject: for a single document, which may contain subdocuments which indexation is not necessary - recursiveReindexobject: for any subtree of documents - ERP5Site_reindexAll: for site-wide reindexations, as there is a semantic- dependent indexation order. Also, uniformise and factorise spawning immediateReindexObject. Also: - testSupply: Drop check for the previous magic threshold. _recurseCallMethod takes care of it all now. - testXMLMatrix: Let activities execute before changing cell id. This works only because recursiveReindexObject on the matrix spawns a single recursiveImmediateReindexObject activity on that context. Now, up to 1k immediateReindexObject activities (for the first 1k sub-objects) are spawned immediately, preventing their renaming immediately after commit. So let test wait for indexation before trying to rename. - testERP5Security: More activities are now spawned immediately, adapt.
-
Vincent Pelletier authored
after_method_id is too aggressive a dependency. after_path_and_method_id is marginally better, but is bad practice nevertheless, as such activity dependencies remain very hard to introspect. Tags allow more variations, and are hence easier to follow. Also, add XXX comments when (recursive) reindexation is only added to represent the dependency, without certainty that a reindexation is actually needed per-se: maybe another place in the code chould just appropriately tag its reindexation activities instead, saving at least the cost of spawning one more activity set, and at most the cost of actually reindexing these documents.
-
Vincent Pelletier authored
Also, drop isIndexable instance property, unmasking class property, which already has the value we need. Also, factorise catalog clearing by reusing ERP5Site_reindexAll's.
-
Vincent Pelletier authored
Allows having a more consistent API between this tool and all other Nexedi-maintained tools.
-
Vincent Pelletier authored
Reduces code duplication.
-
Vincent Pelletier authored
portal_activities contains documents which need to be (and are) indexed. For consistency, the tool itself should be indexable.
-
Vincent Pelletier authored
So it is not just an unused constant (OFS_HANDLER) and no implementation.
-
Vincent Pelletier authored
Also, clarify what exactly was the broken BTree marker value, fixing isBTree accessor. Update associated tests.
-
- 02 Jul, 2018 3 commits
-
-
Arnaud Fontaine authored
erp5.util.testbrowser: Refactor of openNoVisit() and {Image,Submit}Control for recent zope.testbrowser (followup of 5a0b3763).
-
Vincent Pelletier authored
And specifically, Domains (more portal types may be added to the relation string field widget as needed). Predicates get expanded to the list of documents they match.
-
Vincent Pelletier authored
Can be used to identify whether a document should be used to match other documents, rather than as a final document itself.
-
- 29 Jun, 2018 2 commits
-
-
Romain Courteaud authored
Reorganize code by parsing the appcache with jio. Interface checking is done by a standalone gadget whch directly display the status. Drop not needed code (do not reimplement getInterfaceList for example). Force interface strict HTML format
-
Romain Courteaud authored
-
- 26 Jun, 2018 2 commits
-
-
Tristan Cavelier authored
/reviewed-on !706
-
Tristan Cavelier authored
-
- 25 Jun, 2018 1 commit
-
-
Xiaowu Zhang authored
/reviewed-on nexedi/erp5!705
-
- 22 Jun, 2018 2 commits
-
-
Romain Courteaud authored
-
Romain Courteaud authored
-