Skip to content

Commit 81fac42

Browse files
authored
Merge pull request #19436 from bertdeblock/fix/observer-keys-with-colons
[BUGFIX beta] Support observer keys with colons
2 parents a77343f + 09b767d commit 81fac42

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

packages/@ember/-internals/metal/lib/observer.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ export function activateObserver(target: object, eventName: string, sync = false
9696
if (activeObservers.has(eventName)) {
9797
activeObservers.get(eventName)!.count++;
9898
} else {
99-
let [path] = eventName.split(':');
99+
let path = eventName.substring(0, eventName.lastIndexOf(':'));
100100
let tag = getChainTagsForKey(target, path, tagMetaFor(target), peekMeta(target));
101101

102102
activeObservers.set(eventName, {

packages/@ember/-internals/metal/tests/observer_test.js

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,6 +87,21 @@ moduleFor(
8787
assert.equal(count, 1, 'should have invoked observer');
8888
}
8989

90+
async ['@test observer supports keys with colons'](assert) {
91+
obj = {};
92+
let count = 0;
93+
94+
addObserver(obj, 'foo:bar:baz', function () {
95+
assert.equal(get(obj, 'foo:bar:baz'), 'bar', 'should invoke AFTER value changed');
96+
count++;
97+
});
98+
99+
set(obj, 'foo:bar:baz', 'bar');
100+
await runLoopSettled();
101+
102+
assert.equal(count, 1, 'should have invoked observer');
103+
}
104+
90105
async ['@test observer should fire when dependent property is modified'](assert) {
91106
obj = { bar: 'bar' };
92107
defineProperty(

0 commit comments

Comments
 (0)