pengyuthon/odoo_source
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[FIX] web: basic model: field context on m2m operation

Browse Source 
When a m2m field add a context in its arch, it's passed through to the
rpcs made to the server. In a m2m situation, while the name_search
worked properly, the read operation on a dropdown item select did not
provide this context.
This PR fixes that.

closes odoo/odoo#102741

X-original-commit: 5c7066fc064ecc175104603d524b52991678a2cd
Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
This commit is contained in:
Simon Genin (ges)
2022-10-08 07:51:40 +02:00
parent 6f707dc833
commit d1f7192f09
2 changed files with 74 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
addons/web/static/src/legacy/js/views/basic/basic_model.js
+1 -1
View File
@@ -2059,7 +2059,7 @@ var BasicModel = AbstractModel.extend({
model: list.model,
method: 'read',
args: [_.pluck(data, 'id'), fieldNames],
context: _.extend({}, record.context, field.context),
context: _.extend({}, record.context, field.context, list.getContext()),
}).then(function (records) {
_.each(records, function (record) {
list_records[record.id].data = record;
addons/web/static/tests/views/fields/many2many_tags_field_tests.js
+73
View File
@@ -1686,4 +1686,77 @@ QUnit.module("Fields", (hooks) => {
await editInput(target, ".o_field_many2many_tags .o-autocomplete--input", "new tag");
assert.containsNone(target, ".o-autocomplete.dropdown li.o_m2o_dropdown_option");
});
QUnit.test("Many2ManyTagsField with arch context in form view", async (assert) => {
patchWithCleanup(browser, {
setTimeout: (fn) => Promise.resolve().then(fn),
});
await makeView({
type: "form",
resModel: "partner",
serverData,
arch: `<form><field name="timmy" widget="many2many_tags" context="{ 'append_coucou': True }"/></form>`,
async mockRPC(route, args, performRPC) {
const result = await performRPC(route, args);
if (args.method === "read") {
if (args.kwargs.context.append_coucou) {
assert.step("read with context given");
result[0].display_name += " coucou";
}
}
if (args.method === "name_search") {
if (args.kwargs.context.append_coucou) {
assert.step("name search with context given");
for (const res of result) {
res[1] += " coucou";
}
}
}
return result;
},
});
await selectDropdownItem(target, "timmy", "gold coucou");
assert.verifySteps(["name search with context given", "read with context given"]);
assert.strictEqual(target.querySelector(".o_field_tags").innerText, "gold coucou");
});
QUnit.test("Many2ManyTagsField with arch context in list view", async (assert) => {
patchWithCleanup(browser, {
setTimeout: (fn) => Promise.resolve().then(fn),
});
await makeView({
type: "list",
resModel: "partner",
serverData,
arch: `<list editable="top"><field name="timmy" widget="many2many_tags" context="{ 'append_coucou': True }"/></list>`,
async mockRPC(route, args, performRPC) {
const result = await performRPC(route, args);
if (args.method === "read") {
if (args.kwargs.context.append_coucou) {
assert.step("read with context given");
result[0].display_name += " coucou";
}
}
if (args.method === "name_search") {
if (args.kwargs.context.append_coucou) {
assert.step("name search with context given");
for (const res of result) {
res[1] += " coucou";
}
}
}
return result;
},
});
await click(target.querySelector("[name=timmy]"));
await selectDropdownItem(target, "timmy", "gold coucou");
assert.verifySteps(["name search with context given", "read with context given"]);
assert.strictEqual(target.querySelector(".o_field_tags").innerText, "gold coucou");
});
});