Sunday, December 17, 2023
3 changes · master
Miscellaneous changes
On my machine, this reduces the time of the qunit suite from around 115s-120s to 100s-105s. This reduces the time of init messaging with populate medium from around 3s to 1s. Avoid having to split local id to find record. This is a slow process that is better avoided. Make _fields a Map rather than an object. Name anonymous methods to get better trace and profiling. Move field update to separate methods. Start making the update method return whether the value has changed, to go into
Original PR description
On my machine, this reduces the time of the qunit suite from around 115s-120s to 100s-105s. This reduces the time of init messaging with populate medium from around 3s to 1s. Avoid having to split local id to find record. This is a slow process that is better avoided. Make _fields a Map rather than an object. Name anonymous methods to get better trace and profiling. Move field update to separate methods. Start making the update method return whether the value has changed, to go into the direction of removing reactive from internal code (especially for on change). Use markRaw or toRaw on technical values that don't need to be observed. Start better naming variables, especially having a specific name when the value is possibly a proxy, and keeping the simple name for when it's definitely not. https://github.com/odoo/enterprise/pull/52703 Forward-Port-Of: odoo/odoo#146109
Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#142710
Original PR description
Description of the issue/feature this PR addresses: Current behavior before PR: Desired behavior after PR is merged: --- I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr Forward-Port-Of: odoo/odoo#142710
This field is only a getter with no setter. https://github.com/odoo/odoo/pull/146109 Forward-Port-Of: odoo/enterprise#52703
Original PR description
This field is only a getter with no setter. https://github.com/odoo/odoo/pull/146109 Forward-Port-Of: odoo/enterprise#52703