- 27 Nov, 2019 20 commits
-
-
Jérome Perrin authored
IntegrationSite_view/my_source_payment is a proxyfield to a field in erp5_accounting.
-
Jérome Perrin authored
It uses proxy fields from erp5_crm
-
Jérome Perrin authored
'base_view_document_selection' 'Base_viewDocumentList/listbox': set(['erp5_core']) 'Project_viewDocumentList/listbox': set(['erp5_web_project']) 'project_module_selection' 'ProjectModule_viewProjectList/listbox': set(['erp5_project']) 'ProjectModule_viewProjectManagementList/listbox': set(['erp5_web_project'])
-
Jérome Perrin authored
These selection names were already used in other business templates: 'web_page_module_view_web_page_list_selection' 'WebPageModule_viewWebPageListAsJioForCodemirror/listbox': set(['erp5_officejs_codemirror']) 'WebPageModule_viewWebPageListAsJioForTextEditor/listbox': set(['erp5_text_editor']) 'WebPageModule_viewWebPageList/listbox': set(['erp5_web']) 'DocumentModule_viewDocumentListAsJioForPdfViewer/listbox': set(['erp5_officejs_pdf_viewer']) 'image_selection' 'WebPageModule_viewWebPageListAsJioForSvgEditor/listbox': set(['erp5_officejs_svg_editor']) 'ImageModule_viewImageList/listbox': set(['erp5_dms'])
-
Jérome Perrin authored
PropertySheetChild is not an acceptable prefix, use Base as the context is used. Technically, this uses the "id as reference" mixin, but this mixin is a bit special as it has takes the suffix as parameter and does not reside in mixin module, so we just use Base as a prefix.
-
Jérome Perrin authored
FieldValidator is not an acceptable prefix, Base_validateDialogConfirmation seems better
-
Jérome Perrin authored
Rounding scripts: Planning is not an acceptable prefix, this script is called on the current context, so let's use Base_ as a prefix. Planning_roundBoundToMinute → Base_roundPlanningBoxBoundToMinute Planning_roundBoundToDay → Base_roundPlanningBoxBoundToDay Planning_roundBoundToInt → Base_roundPlanningBoxBoundToInt
-
Jérome Perrin authored
Module is an acceptable prefix for modules Introduce a list of skins IDs that does not match our conventions but are acceptable because this is what external API are calling.
-
Jérome Perrin authored
This `o` was an unused variable. The initial intention was probably this.
-
Jérome Perrin authored
These scripts are not used and does not respect naming conventions or have pylint errors.
-
Jérome Perrin authored
-
Jérome Perrin authored
This will allow to find problems when only business templates and their dependencies are installed. When we run testXHTML with all business template installed we don't see problems that may happen only when some business templates are not installed.
-
Jérome Perrin authored
These scripts are used to check the site configuration and also in CodingStyleTest. This is just the minimal so that CodingStyleTest can run, but this could be extended to several scripts currently in erp5_toolbox skinfolder of erp5_forge which overlaps with erp5_administration. Also, regarding CodingStyleTest, it seems that erp5_administration contain a lot of things, so we may want to extract a smaller business template containing only the scripts used for static analysis of business template.
-
Jérome Perrin authored
We want this check to run on all components.
-
Jérome Perrin authored
Some portal types such as Business Template were not tested by testXHTML because this tests starts by modules and recursively tests based on allowed content types. But types that are not created in a module but in a tool were never tested. Keep the current way of testing view of portal types in their natural containers by starting from modules and make another pass to test anyway portal types that might be contained in tool. In that case, we use a mapping of known tools IDs because I don't think we can introspect at this point.
-
Jérome Perrin authored
Maybe what's missing is an interaction to update cache internals with updateCache when a cache is added, but that's enough to make TestXHTML pass now. fixup! core: workaround non initialized cache when viewing cache statistics
-
Jérome Perrin authored
DublinCore was missing, so getCompactTitle was causing AttributeError _baseGetTranslatedTitle
-
Jérome Perrin authored
When listboxs has an editable field, but the field is non editable, it produces a markup like: <a href="link to the line"><EditableField/></a> This is fine for most fields, but some editable fields are rendered as an <input> even if they are not editable - this is the case for CheckBoxField. To prevent rendering <a><input></a> which is not valid HTML, configure the enabled field as non-editable on alarm list view, with this change listbox treats the field as an editable field and just render the field as <EditableField/> without the <a>. To keep the same visual appearance of having a disabled checkbox field, make this field disabled with extra.
-
Jérome Perrin authored
-
Jérome Perrin authored
The action was in erp5_core, but the form in erp5_forge. Move the action in erp5_forge. The listbox list method was raising error because it was using catalog with unsupported installation_state=. Change to use contentValues instead. This listbox had "Listbox" as a title, use "History" instead, which makes a little more sense in this context.
-
- 26 Nov, 2019 3 commits
-
-
Gabriel Monnerat authored
-
Jérome Perrin authored
MissingCategoryDocumentConstraint was incorrectly using unrestrictedTraverse, unlike category API which uses its own resolveCategory method with better semantics. Because of this, some problems were not reported: - categories using paths that resolve by acquisition, but does not exists as such (like in the test) - categories using path with dynamic objects, like a website language. /reviewed-on nexedi/erp5!986
-
Jérome Perrin authored
checkRevision now returns a callable, return a do-nothing lambda when the expected revision is already checked out /reviewed-on !990
-
- 25 Nov, 2019 5 commits
-
-
Gabriel Monnerat authored
-
Gabriel Monnerat authored
-
Gabriel Monnerat authored
With this merge request we add the feature of allow a gadget to call submitDialog *and* the prototype of scan documents from mobile with new ERP5 UI /reviewed-on nexedi/erp5!932
-
Gabriel Monnerat authored
For now, this is chrome only erp5_document_scanner_ui_test: Add tests to Document Scanner gadget
-
Gabriel Monnerat authored
-
- 22 Nov, 2019 12 commits
-
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Roque authored
-
Roque authored
-
Roque authored
-
Roque authored
- tests updated
-
Roque authored
- test extended
-
Roque authored
-
Roque authored
-