- 04 Jun, 2024 1 commit
-
-
Titouan Soulard authored
When pasting an image in CKEditor, the image was displayed twice. This behaviour is due to the new paste-as-base64 feature implemented in CKE, which conflict with existing plugins implementing the same feature. To solve it, add `clipboard_handleImages` option to the CKE configuration object, which disables its own image-pasting feature. See https://github.com/ckeditor/ckeditor4/issues/4874 for the corresponding issue on CKEditor’s repository. /cc @jerome @romain @tomo /reviewed-by @jerome /reviewed-on nexedi/erp5!1951
-
- 03 Jun, 2024 5 commits
-
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
fix convertToERP5Workflow() for the case where permission_roles is not yet configured.
-
Kazuhiko Shiozaki authored
if acquire_permission is not yet set in a Workflow State, we should consider it as 'acquire all permissions', like DCWorkflow implementation.
-
Kazuhiko Shiozaki authored
If state_permission_role_list_dict is not yet set in a Workflow State, the UI displays permission/role table with all unchecked. Once we save it in Permissions tab without checking anything, state_permission_role_list_dict will become like {'Access contents information': (), 'Add portal content': (), 'Modify portal content': (), 'View': ()} This commit will make the behaviour same for these 'before save' and 'after save' cases as the UI displays exactly same.
-
- 31 May, 2024 4 commits
-
-
Jérome Perrin authored
See merge request nexedi/erp5!1941
-
Jérome Perrin authored
In Check Source Code report on business template and CodingStyle tests, the lines were off by 1 and column off by 2 for python scripts.
-
Jérome Perrin authored
The default manager user (ERP5TypeTestCase) now have a random password generated at the beginning of test. The password is stored as a class attribute of the test case instance. Many tests have been updated to not generate manager user and use the existing one instead. When tests need to create users (for example users in the root acl_users), we try to give them a random password and to delete the users afterward. For functional tests, the approach is that ERP5TypeFunctionalTestCase sets cookies with manager username and password before running zelenium tests, so that in case tests want to log in again as manager, they can read the username and password from cookies. Another significant changes is that we no longer have the same user in ERP5/acl_users and /acl_users, some tests were logging in as the root user in ways that never seemed intentional. This also revealed (through test_manager_actions_on_portal from testERP5Core) that some tests were running with a user without all the expected permissions and the "Manager Components" actions was not visible for the default manager user. Fixing this also revealed that two actions ("Manage Components" and "Manage Callables") had the same priority, so the later was modified to use a different priority.
-
Jérome Perrin authored
pycrypto is unmaintained and the python3 version no longer work with buildout 3 (it uses a tricky way to run 2to3 in setup.py that no longer happens with pip based buildout3) Also make the rest of this file python3 compatible.
-
- 29 May, 2024 2 commits
-
-
Jérome Perrin authored
only the case without an URL column was properly handled
-
Roque authored
See merge request nexedi/erp5!1945
-
- 28 May, 2024 2 commits
-
-
Roque authored
-
Gabriel Monnerat authored
See merge request !1909
-
- 27 May, 2024 3 commits
-
-
Gabriel Monnerat authored
If erp5_credential is installed in ERP5, we will use the dialog from erp5_credential instead of accessing portal_password directly.
-
Roque authored
- setting configurator header link - allow auto_sync when adding opml - update form title - fix landing page - fix dispatch - fix form naming - fix form and fields naming (coding style tests)
-
- 22 May, 2024 4 commits
-
-
Rafael Monnerat authored
-
Rafael Monnerat authored
Otherwise, alarms to fetch what was paid (status) don't work.
-
Rafael Monnerat authored
Mistake was probably there because copy and paste
-
Jérome Perrin authored
Raise an exception instead of casting to string unhandled search values, because it's more likely that caller accidentally passed a wrong value.
-
- 20 May, 2024 8 commits
-
-
Roque authored
See merge request nexedi/erp5!1864
-
Roque authored
-
Roque authored
- site web section changes - update app router - update app panel - adapt monitoring jio gadget and jio storage - add form views configuration - add new portal types, views and actions - migration_version check - update precache manifest - sync checks and redirection - refactor dispatch page according to new monitor urls - add opml redirects to front page - slapos portal types - drop obsolete files - drop auto-sync on add OPML page - parameters gadget field for instance tree view - custom views for software instance resources and processes - options header gadget (custom header options) - custom submit gadgets - graph gadget field
-
Roque authored
- allow custom app configurator - render listbox form field extra configuration - fix date format rendering - allow configuration form fields to have extra js code - allow to customize listbox column styles - allow to customize listbox jio_alldocs - fix custom view handling - fix gadget field rendering - allow dev user to set custom header options - truncate long string fields - add custom submit gadget option - improve renderjs_extra - update graph gadget
-
Roque authored
-
Jérome Perrin authored
Some general changes to test framework and shared test code See merge request nexedi/erp5!1936
-
Jérome Perrin authored
this happens with new ghostscript, but this does not seem a critical problem.
-
Jérome Perrin authored
The test was only testing with empty cache
-
- 19 May, 2024 2 commits
-
-
Jérome Perrin authored
This should not happens, but if this happens we should not just return in the middle of the loop, because this returned value is not used, so we would just silently edit partially.
-
Jérome Perrin authored
The first commit of this merge request was introduced for zope4py3 branch but it revealed issues on py2 as well, i.e. we don't have roles in auto-generated tester methods. ``` ====================================================================== FAIL: test_method_protection (testSecurity.TestSecurity) This test will list all implicitly Public methods in any objects in ZODB. ---------------------------------------------------------------------- Traceback (most recent call last): File "/(SR)/parts/erp5/Products/ERP5/tests/testSecurity.py", line 113, in test_method_protection self.fail(message) AssertionError: The following 7 methods have a docstring but have no security assertions. /(SR)/parts/erp5/product/ERP5Form/PreferenceTool.py:65 isPreferredVcsPushMode /srv/slapgrid/slappart19/t/eiy/soft/8a7759fd7b65b20d9f87713605745d05/parts/erp5/product/ERP5Type/Accessor/AcquiredProperty.py:217 hasTelephoneValidationState /(SR)/parts/erp5/product/ERP5Type/Accessor/Base.py:219 hasViewFormIdList /(SR)/parts/erp5/product/ERP5Type/Accessor/Constant.py:94 isWebDocumentType /(SR)/parts/erp5/product/ERP5Type/Accessor/Content.py:224 hasImage /(SR)/parts/erp5/product/ERP5Type/Accessor/ContentProperty.py:283 hasImageWidthList /(SR)/parts/erp5/product/ERP5Type/Accessor/Translation.py:233 hasFrTranslatedTitle ---------------------------------------------------------------------- ``` See merge request nexedi/erp5!1911
-
- 17 May, 2024 9 commits
-
-
Jérome Perrin authored
-
Jérome Perrin authored
for same reason as the xhtml_style version, there is no problem with having more than one listbox, see d2dedca3 (core: remove useless log in Base_callDialogMethod, 2020-07-03)
-
Jérome Perrin authored
-
Jérome Perrin authored
If preferences are not set, this causes a non existant style to be used
-
Jérome Perrin authored
If a transaction is selected but instead of having accounts as source/destination on lines it acquires the organisation set as source/destination on the transaction, this causes an AttributeError getGapList when trying to use this organisation as an account. To fix this, adjust the getMovementHistoryList parameters to select only lines using accounts, by passing node_uid, which is O(n) on the number of accounts, but the number of accounts is not supposed to be too large and other reports also do this assumption.
-
Jérome Perrin authored
This prevents one failing test to make all subsequent tests to fail
-
Jérome Perrin authored
this is removed from python3, it was not so useful here either
-
Kazuhiko Shiozaki authored
-
Kazuhiko Shiozaki authored
-