Skip to content

Conversation

@Polymorphe57
Copy link
Contributor

@Polymorphe57 Polymorphe57 commented Oct 7, 2025

The goal is to apply to OWL the same type of changes that were made in odoo/odoo#197296.
Note that we have a stricter version here. In case some attrs are found on a t-call, the only parameters given to the template called are the one listed or found in the t-call-context.

@Polymorphe57 Polymorphe57 force-pushed the master-t-call-dam branch 4 times, most recently from 5b2cfaa to 29c594a Compare October 7, 2025 15:51
@Polymorphe57 Polymorphe57 changed the title Master t call dam [IMP] qweb: Make t-call parametric, enable lazy XML evaluation Oct 7, 2025
@Polymorphe57 Polymorphe57 force-pushed the master-t-call-dam branch 12 times, most recently from aa0587c to 8d28359 Compare October 10, 2025 13:56
@Polymorphe57 Polymorphe57 marked this pull request as ready for review October 10, 2025 14:02
@Polymorphe57 Polymorphe57 force-pushed the master-t-call-dam branch 4 times, most recently from 59d5146 to 956d1e1 Compare October 10, 2025 15:42
const b2 = ctx[zero];
const b3 = ctx[zero];
const b2 = ctx[zero]?.(node, key);
const b3 = ctx[zero]?.(node, key + \`__1\`);
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useful to have different keys?

@Polymorphe57 Polymorphe57 force-pushed the master-t-call-dam branch 2 times, most recently from 4445155 to 5fbcb50 Compare October 13, 2025 12:47
@Polymorphe57 Polymorphe57 force-pushed the master-t-call-dam branch 3 times, most recently from 31e16a9 to 5efcc86 Compare October 22, 2025 14:29
The goal is to apply to OWL the same type of changes that were made in odoo/odoo#197296.
Note that we have a stricter version here. In case some attrs are found
on a t-call, the only parameters given to the template called are the
one listed or found in the t-call-context.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant