KnockoutJS scramble controlsDescendantBindings

This commit is contained in:
the-djmaze 2024-03-03 17:27:18 +01:00
parent 4ee00d30ae
commit 667e50c9fb
6 changed files with 6 additions and 6 deletions

View file

@ -380,7 +380,7 @@ function applyBindingsToNodeInternal(node, sourceBindings, bindingContext) {
var initResult = handlerInitFn(node, getValueAccessor(bindingKey), allBindings, contextToExtend['$data'], contextToExtend);
// If this binding handler claims to control descendant bindings, make a note of this
if (initResult && initResult['controlsDescendantBindings']) {
if (initResult && initResult.controlsDescendantBindings) {
if (bindingHandlerThatControlsDescendantBindings !== undefined)
throw new Error("Multiple bindings (" + bindingHandlerThatControlsDescendantBindings + " and " + bindingKey + ") are trying to control descendant bindings of the same element. You cannot use these bindings together on the same element.");
bindingHandlerThatControlsDescendantBindings = bindingKey;

View file

@ -1,7 +1,7 @@
ko.bindingHandlers['html'] = {
'init': () => (
// Prevent binding on the dynamically-injected HTML (as developers are unlikely to expect that, and it has security implications)
{ 'controlsDescendantBindings': true }
{ controlsDescendantBindings: true }
),
'update': (element, valueAccessor) => {
// setHtml will unwrap the value if needed

View file

@ -50,7 +50,7 @@ function makeWithIfBinding(bindingKey, isWith, isNot) {
}, { disposeWhenNodeIsRemoved: element });
return { 'controlsDescendantBindings': true };
return { controlsDescendantBindings: true };
}
};
ko.virtualElements.allowedBindings[bindingKey] = true;

View file

@ -11,7 +11,7 @@ ko.bindingHandlers['options'] = {
}
// Ensures that the binding processor doesn't try to bind the options
return { 'controlsDescendantBindings': true };
return { controlsDescendantBindings: true };
},
'update': (element, valueAccessor, allBindings) => {

View file

@ -2,7 +2,7 @@ ko.bindingHandlers['text'] = {
'init': () => (
// Prevent binding on the dynamically-injected text node (as developers are unlikely to expect that, and it has security implications).
// It should also make things faster, as we no longer have to consider whether the text node might be bindable.
{ 'controlsDescendantBindings': true }
{ controlsDescendantBindings: true }
),
'update': (element, valueAccessor) => {
if (8 === element.nodeType) {

View file

@ -71,7 +71,7 @@
});
}, { disposeWhenNodeIsRemoved: element });
return { 'controlsDescendantBindings': true };
return { controlsDescendantBindings: true };
}
};