[FIX] mail: rename ambiguous get method in model manager

To ease debugging in general, avoid having multiple methods with the same name,
especially in the same file.

This is especially useful when the method name appears in a stack trace, for
example in the performance tool.

Part of task-2702450

closes odoo#81003

Signed-off-by: Alexandre Kühn (aku) <[email protected]>

Author: seb-odoo

addons/mail/static/src/model/model_manager.js

@@ -714,7 +714,7 @@ export class ModelManager {
          */
         const nonProxyRecord = new Model({ localId, valid: true });
         const record = !this.env.isDebug() ? nonProxyRecord : new Proxy(nonProxyRecord, {
-            get: function (record, prop) {
+            get: function getFromProxy(record, prop) {
                 if (
                     !Model.__fieldMap[prop] &&
                     !['_super', 'then'].includes(prop) &&
@@ -1321,7 +1321,7 @@ export class ModelManager {
             // Add field accessors.
             for (const field of Model.__fieldList) {
                 Object.defineProperty(Model.prototype, field.fieldName, {
-                    get() { // this is bound to record
+                    get: function getFieldValue() { // this is bound to record
                         for (const listener of this.modelManager._listeners) {
                             listener.lastObservedLocalIds.add(this.localId);
                             if (!this.modelManager._listenersObservingLocalId.has(this.localId)) {