odoo/o-spreadsheet#4509

Created by fw-bot

label

odoo:saas-17.1-17.0-set-default-translations-as-loaded-adrm-5rTJ-fw

head

85dea883b37c770b337fae6ed529aefef5de8b31

merged

1 month ago by BI, Pierre Rousseau (pro)

	odoo/o-spreadsheet
17.0	#4497
saas-17.1	#4509
saas-17.2	#4510
saas-17.3	#4512
master	#4513

[FW][FIX] model: mark default translations as loaded

Description

Currently the default translations function is marked as not loaded. That means that if no custom translation method is set, the translations will never be marked as loaded, and charts will crash because we try to deepCopy LazyTranslatedString.

However we can't mark the default translations as loaded by default, because otherwise top-level translations would always be translated before we can set the translation method.

The solution is to mark the default translations as loaded only when starting the model.

Task: : 3999116

review checklist

• [ ] feature is organized in plugin, or UI components
• [ ] support of duplicate sheet (deep copy)
• [ ] in model/core: ranges are Range object, and can be adapted (adaptRanges)
• [ ] in model/UI: ranges are strings (to show the user)
• [ ] undo-able commands (uses this.history.update)
• [ ] multiuser-able commands (has inverse commands and transformations where needed)
• [ ] new/updated/removed commands are documented
• [ ] exportable in excel
• [ ] translations (_t("qmsdf %s", abc))
• [ ] unit tested
• [ ] clean commented code
• [ ] track breaking changes
• [ ] doc is rebuild (npm run doc)
• [ ] status is correct in Odoo

Forward-Port-Of: #4497