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

fw-bot commented 2 weeks ago

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: odoo/o-spreadsheet#4497

robodoo commented 2 weeks ago

fw-bot commented 2 weeks ago

@hokolomopo @pro-odoo this PR targets master and is the last of the forward-port chain containing:

odoo/o-spreadsheet#4512
odoo/o-spreadsheet#4510
odoo/o-spreadsheet#4509

To merge the full chain, use

@robodoo r+

More info at https://github.com/odoo/odoo/wiki/Mergebot#forward-port

pro-odoo commented 2 weeks ago

robodoo r+

odoo / o-spreadsheet

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

Description

review checklist