- 17 Jan, 2020 2 commits
-
-
Jérome Perrin authored
The fixes from !630 were not enough, exceptions set on days were calendar did not repeat were also confusing other exceptions after this. Simplify implementation a lot, instead of keeping track of the next exception date, start by building a set of all exceptions dates and use membership of this set as a criterion to skip exceptions.
-
Arnaud Fontaine authored
-
- 15 Jan, 2020 6 commits
-
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Arnaud Fontaine authored
ZODB Components: Preparation of erp5_base migration from FS: Fix pylint no-name-in-module on newTempXXX (04b49859).
-
Jérome Perrin authored
Change upgrader internal API to use `filter_dict` instead of `filter` which is a builtin. Remove a lot of unused code in extensions /reviewed-on nexedi/erp5!1014
-
- 14 Jan, 2020 12 commits
-
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
ZODB Components: Preparation of erp5_base migration from FS: Fix pylint no-name-in-module on newTempXXX (04b49859).
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
ZODB Components: BusinessTemplate: On migration dialog, ignore source files not available anymore (ex: change branch).
-
Jérome Perrin authored
Test should not depend on external websites, we already use erp5.com for some other tests, so let's use it also for RSS. /reviewed-on nexedi/erp5!1028
-
Vincent Pelletier authored
Deleting matrix cells with manage_delObjects is not a good solution, as this breaks matrix index consistency. Also, call isBasePricePerSlice rather than introspecting wrapped method arguments - the only wrapped method is _setBasePricePerSlice anyway ! Also, explicitly trigger SupplyCell_updateSliceBasePrice as it depends on isBasePricePerSlice value at line level, and it just changed, and cell-level interaction workflow does not trigger on line changes. Also, get rid of a trivial local variable. Also, fix the exception expected from list.index . Also, raise if multiple quantity criterion are somehow found.
-
- 13 Jan, 2020 2 commits
-
-
Arnaud Fontaine authored
-
Jérome Perrin authored
When server replies with a non XML content type feeparser attempt to parse anyway, but set the bozo flag. https://universal-feedparser.readthedocs.io/en/latest/#handling-incorrectly-declared-media-types If parsing fail because the feed is not a valid RSS XML stream, feedparser fail in other ways, so we can safely ignore this error. This should repair recent failures with testFeedReaderGadget now that le monde changed configuration on their website: $ curl -sD - -o /dev/null https://www.lemonde.fr/rss/une.xml | grep Content-Type Content-Type: text/html;charset=UTF-8 /reviewed-on nexedi/erp5!1027
-
- 10 Jan, 2020 18 commits
-
-
Romain Courteaud authored
See nexedi/erp5@b678a025
-
Romain Courteaud authored
-
Romain Courteaud authored
Introduce the object_jio_jump action category to mark all compatible actions. erp5_xhtml_style supports object_jio_jump. A correct redirection is needed to jump in both UI. ERP5JS will display modules on the current document context, as if the jump was instead a object_view action.
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
-
Romain Courteaud authored
This improve the compatibility with ERP5JS
-
Romain Courteaud authored
-
Romain Courteaud authored
It will reduce the HTML size and prevent any browser parsing incompatibility.
-
Arnaud Fontaine authored
ZODB Components: erp5_banking_core: Fix testERP5Banking{AccountingCancellation,BankAccountInventory,CheckDeposit} failures. BankingOperation monkey patches several ERP5 basic classes to add getBaobab* methods. With ERP5Banking Product on FS, BankingOperation was imported at Zope startup when loading Products and monkey patches were applied.
-
Arnaud Fontaine authored
Moreover, since this was added and because of a bug in bt5 implementation of provide dependencies, it was not possible to install this bt5 from TemplateTool on a fresh instance: [...] File "product/ERP5Type/tests/runUnitTest.py", line 417, in _installBusinessTemplateList with_test_dependency_list=True)] File "product/ERP5/Tool/TemplateTool.py", line 1200, in resolveBusinessTemplateListDependency return self.sortBusinessTemplateList(list(bt5_set)) File "product/ERP5/Tool/TemplateTool.py", line 947, in sortBusinessTemplateList "Circular dependencies on %s" % reverse_dependency_dict.keys() NotImplementedError: Circular dependencies on ['erp5_mysql_innodb_catalog', 'erp5_banking_core', 'erp5_property_sheets', 'erp5_core', 'erp5_pdm', 'erp5_trade', 'erp5_accounting', 'erp5_core_proxy_field_legacy', 'erp5_simulation', 'erp5_base'] -> erp5_banking_core depends on erp5_trade. --> erp5_trade depends on erp5_base. ---> erp5_base depends on erp5_full_text_catalog meta bt5 resolved to erp5_banking_core (provision_list). This fixes testBanking* Unit Tests failures since ERP5Banking migration to ZODB Components (0d2bd4c0) not triggered with FS Unit Tests because it does not try to resolve dependencies and just install them in getBusinessTemplateList() order.
-
Arnaud Fontaine authored
ZODB Components: Cosmetic: Remove Unit Test FS-only getBusinessTemplateList() now that it has been migrated.
-
Arnaud Fontaine authored
ERP5Site: Do not override newContent() but use FolderMixIn implementation to have a consistent behavior. ERP5Site was raising error when ID or portal_type was not passed but there is no reason to do so (as it is done in FolderMixIn.newContent()). This allows calling ERP5Site.newContent(temp_object=True) without ID, likewise Folder.
-
Arnaud Fontaine authored
-
Arnaud Fontaine authored
Folder: Allow to create temp_objects anywhere, without checking if it's in allowed types for that Portal Type. Until now, creating a temp_object with newContent() required for the temp_object to be in allowed types. However, this was inconsistent with (now deprecated) newTemp*() functions (calling directly constructInstance()) not having such limitations. Besides lifting such limitation being needed to properly replace newTemp*() calls, portal_trash.newContent() pattern is already commonly used used here meaning there is a real use case. This also fixes testBanking* failures: newTempCashDeliveryLine() created objects on 'Cash Inventory' and latter call Delivery.isAccountable() which does not exist on portal_trash. See also 04b49859. /reviewed-on nexedi/erp5!1025
-
Nicolas Wavrant authored
-
Nicolas Wavrant authored
-