These conditions were added many years ago and they should not be needed
since checkboxes work without this extra complexity

ZODB Components: Ignore low-level TypeProvider (TypesTool and SolverTool) classes in FS migration Dialog.

Since 7b4a3d4c instead of subscribing after add code unsubscribed which is illogical. Revert back. Credit Kaz.

Tests passing : https://nexedi.erp5.net/test_result_module/20200327-660269E/view?ignore_layout:int=1

See merge request !1092

testUpgradeInstanceWithOldDataFs: New Unit Test to check upgrade of old instance to current ERP5 version.

After copying old instance ZODB and SQL from erp5-bin.git, this checks whether
instance can be successfully through upgrader. Introduced because of 140be41d.

Since 7b4a3d4c instead of subscribing after add code unsubscribed which is illogical. Revert back. Credit Kaz.

See merge request !1085

runUnitTest: Set `skip-ownership-checking true` as we already do for a normal instance through zope.conf.

Related to 58c9f7c3c688854958592590dd396929b5a0c8d4 in slapos.git.

In nexedi/erp5@935b39f9 we accidentally enabled coding style test for `erp5_project`. Let's fix the current problems reported by coding style test.

See merge request !1087

See merge request !1079

It was required by Handlebars

Allow to use it from erp5_xhtml_style.

This business template is not used and contain a preference referencing
non existing paths, which causes errors in coding style test with
test_moduleListMethod that cannot display accounting module, because
preference is set to group/my_group which does not exist.

Instead of trying to fix, let's remove the business template if it's not
used.

Project_viewAccountingTransactionList uses some scripts from
erp5_accounting

metadata was not committed, so next time VCS tab is used on this
business template the path is marked as removed.

authentication policy, invoicing, ingestion and credential

See merge request !1081

When these printouts were made, cloudooo did not had so many fonts as
today, and it was picking up "Liberation Sans" for the "Arial" used in
these templates;

Since
nexedi/slapos@135cbe43
( yes, 5 years ago ) we have more fonts, and since then the rendering of
these printout changed, a "Roman" font was selected instead.

This changes the templates to use explicitly "Liberation Sans", which
makes the printouts use the old looks even on newer cloudooo .

See merge request !1080

In case of erp5_core business template update this code is called (with update=True argument)
and thus it will effectively override previous portal_alarms' state which
is wrong as no "general purpose" code should overwrite potential
developer configuration.
Tests passing at https://nexedi.erp5.net/test_result_module/20200324-3BC4D696 thus merging right away.

See merge request !1086

See merge request !1050

business template update this code is called (with update=True argument)
and thus it will effectivly override previous portal_alarms' state which
is wrong as no "general purpose" code should ovewrite potential
developer configuration.

Also extend coding style test to cover this kind of issues with business templates.

See merge request !1083

This caused upgrader errors such as:

    Error: Web Site web_site_module/ooffice_spreadsheet references a non existant appcache gadget_officejs_spreadsheet.appcache

This caused upgrader errors such as:

    Error: Web Site web_site_module/officejs_smart_assistant references a non existant appcache gadget_officejs_spreadsheet.appcache

We want to detect business templates with problems that would prevent to
run upgrader.