Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit f003d93

Browse files
jbedardjbedard
committed
Revert "fix($parse): standardize one-time literal vs non-literal and interceptors"
This reverts commit 60394a9.
1 parent aac5623 commit f003d93

File tree

3 files changed

+152
-107
lines changed

3 files changed

+152
-107
lines changed

‎src/ng/directive/ngClass.js

+45-8
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,13 @@ function classDirective(name, selector) {
1414
return {
1515
restrict: 'AC',
1616
link: function(scope, element, attr) {
17+
var expression = attr[name].trim();
18+
var isOneTime = (expression.charAt(0) === ':') && (expression.charAt(1) === ':');
19+
20+
var watchInterceptor = isOneTime ? toFlatValue : toClassString;
21+
var watchExpression = $parse(expression, watchInterceptor);
22+
var watchAction = isOneTime ? ngClassOneTimeWatchAction : ngClassWatchAction;
23+
1724
var classCounts = element.data('$classCounts');
1825
var oldModulo = true;
1926
var oldClassString;
@@ -36,7 +43,7 @@ function classDirective(name, selector) {
3643
scope.$watch(indexWatchExpression, ngClassIndexWatchAction);
3744
}
3845

39-
scope.$watch($parse(attr[name], toClassString), ngClassWatchAction);
46+
scope.$watch(watchExpression, watchAction, isOneTime);
4047

4148
function addClasses(classString) {
4249
classString = digestClassCounts(split(classString), 1);
@@ -78,9 +85,9 @@ function classDirective(name, selector) {
7885
}
7986

8087
function ngClassIndexWatchAction(newModulo) {
81-
// This watch-action should run before the `ngClassWatchAction()`, thus it
88+
// This watch-action should run before the `ngClass[OneTime]WatchAction()`, thus it
8289
// adds/removes `oldClassString`. If the `ngClass` expression has changed as well, the
83-
// `ngClassWatchAction()` will update the classes.
90+
// `ngClass[OneTime]WatchAction()` will update the classes.
8491
if (newModulo === selector) {
8592
addClasses(oldClassString);
8693
} else {
@@ -90,13 +97,15 @@ function classDirective(name, selector) {
9097
oldModulo = newModulo;
9198
}
9299

93-
function ngClassWatchAction(newClassString) {
94-
// When using a one-time binding the newClassString will return
95-
// the pre-interceptor value until the one-time is complete
96-
if (!isString(newClassString)) {
97-
newClassString = toClassString(newClassString);
100+
function ngClassOneTimeWatchAction(newClassValue) {
101+
var newClassString = toClassString(newClassValue);
102+
103+
if (newClassString !== oldClassString) {
104+
ngClassWatchAction(newClassString);
98105
}
106+
}
99107

108+
function ngClassWatchAction(newClassString) {
100109
if (oldModulo === selector) {
101110
updateClasses(oldClassString, newClassString);
102111
}
@@ -143,6 +152,34 @@ function classDirective(name, selector) {
143152

144153
return classString;
145154
}
155+
156+
function toFlatValue(classValue) {
157+
var flatValue = classValue;
158+
159+
if (isArray(classValue)) {
160+
flatValue = classValue.map(toFlatValue);
161+
} else if (isObject(classValue)) {
162+
var hasUndefined = false;
163+
164+
flatValue = Object.keys(classValue).filter(function(key) {
165+
var value = classValue[key];
166+
167+
if (!hasUndefined && isUndefined(value)) {
168+
hasUndefined = true;
169+
}
170+
171+
return value;
172+
});
173+
174+
if (hasUndefined) {
175+
// Prevent the `oneTimeLiteralWatchInterceptor` from unregistering
176+
// the watcher, by including at least one `undefined` value.
177+
flatValue.push(undefined);
178+
}
179+
}
180+
181+
return flatValue;
182+
}
146183
}
147184

148185
/**

‎src/ng/parse.js

+38-26
Original file line numberDiff line numberDiff line change
@@ -1801,8 +1801,8 @@ function $ParseProvider() {
18011801
if (parsedExpression.constant) {
18021802
parsedExpression.$$watchDelegate = constantWatchDelegate;
18031803
} else if (oneTime) {
1804-
parsedExpression.oneTime = true;
1805-
parsedExpression.$$watchDelegate = oneTimeWatchDelegate;
1804+
parsedExpression.$$watchDelegate = parsedExpression.literal ?
1805+
oneTimeLiteralWatchDelegate : oneTimeWatchDelegate;
18061806
} else if (parsedExpression.inputs) {
18071807
parsedExpression.$$watchDelegate = inputsWatchDelegate;
18081808
}
@@ -1888,7 +1888,6 @@ function $ParseProvider() {
18881888
}
18891889

18901890
function oneTimeWatchDelegate(scope, listener, objectEquality, parsedExpression, prettyPrintExpression) {
1891-
var isDone = parsedExpression.literal ? isAllDefined : isDefined;
18921891
var unwatch, lastValue;
18931892
if (parsedExpression.inputs) {
18941893
unwatch = inputsWatchDelegate(scope, oneTimeListener, objectEquality, parsedExpression, prettyPrintExpression);
@@ -1905,22 +1904,41 @@ function $ParseProvider() {
19051904
if (isFunction(listener)) {
19061905
listener(value, old, scope);
19071906
}
1908-
if (isDone(value)) {
1907+
if (isDefined(value)) {
19091908
scope.$$postDigest(function() {
1910-
if (isDone(lastValue)) {
1909+
if (isDefined(lastValue)) {
19111910
unwatch();
19121911
}
19131912
});
19141913
}
19151914
}
19161915
}
19171916

1918-
function isAllDefined(value) {
1919-
var allDefined = true;
1920-
forEach(value, function(val) {
1921-
if (!isDefined(val)) allDefined = false;
1922-
});
1923-
return allDefined;
1917+
function oneTimeLiteralWatchDelegate(scope, listener, objectEquality, parsedExpression) {
1918+
var unwatch, lastValue;
1919+
unwatch = scope.$watch(function oneTimeWatch(scope) {
1920+
return parsedExpression(scope);
1921+
}, function oneTimeListener(value, old, scope) {
1922+
lastValue = value;
1923+
if (isFunction(listener)) {
1924+
listener(value, old, scope);
1925+
}
1926+
if (isAllDefined(value)) {
1927+
scope.$$postDigest(function() {
1928+
if (isAllDefined(lastValue)) unwatch();
1929+
});
1930+
}
1931+
}, objectEquality);
1932+
1933+
return unwatch;
1934+
1935+
function isAllDefined(value) {
1936+
var allDefined = true;
1937+
forEach(value, function(val) {
1938+
if (!isDefined(val)) allDefined = false;
1939+
});
1940+
return allDefined;
1941+
}
19241942
}
19251943

19261944
function constantWatchDelegate(scope, listener, objectEquality, parsedExpression) {
@@ -1936,28 +1954,22 @@ function $ParseProvider() {
19361954
var watchDelegate = parsedExpression.$$watchDelegate;
19371955
var useInputs = false;
19381956

1939-
var isDone = parsedExpression.literal ? isAllDefined : isDefined;
1957+
var regularWatch =
1958+
watchDelegate !== oneTimeLiteralWatchDelegate &&
1959+
watchDelegate !== oneTimeWatchDelegate;
19401960

1941-
function regularInterceptedExpression(scope, locals, assign, inputs) {
1961+
var fn = regularWatch ? function regularInterceptedExpression(scope, locals, assign, inputs) {
19421962
var value = useInputs && inputs ? inputs[0] : parsedExpression(scope, locals, assign, inputs);
19431963
return interceptorFn(value, scope, locals);
1944-
}
1945-
1946-
function oneTimeInterceptedExpression(scope, locals, assign, inputs) {
1947-
var value = useInputs && inputs ? inputs[0] : parsedExpression(scope, locals, assign, inputs);
1964+
} : function oneTimeInterceptedExpression(scope, locals, assign, inputs) {
1965+
var value = parsedExpression(scope, locals, assign, inputs);
19481966
var result = interceptorFn(value, scope, locals);
19491967
// we only return the interceptor's result if the
19501968
// initial value is defined (for bind-once)
1951-
return isDone(value) ? result : value;
1952-
}
1953-
1954-
var fn = parsedExpression.oneTime ? oneTimeInterceptedExpression : regularInterceptedExpression;
1955-
1956-
// Propogate the literal/oneTime attributes
1957-
fn.literal = parsedExpression.literal;
1958-
fn.oneTime = parsedExpression.oneTime;
1969+
return isDefined(value) ? result : value;
1970+
};
19591971

1960-
// Propagate or create inputs / $$watchDelegates
1972+
// Propagate $$watchDelegates other then inputsWatchDelegate
19611973
useInputs = !parsedExpression.inputs;
19621974
if (watchDelegate && watchDelegate !== inputsWatchDelegate) {
19631975
fn.$$watchDelegate = watchDelegate;

‎test/ng/parseSpec.js

+69-73
Original file line numberDiff line numberDiff line change
@@ -2688,86 +2688,82 @@ describe('parser', function() {
26882688
expect($parse(':: ').literal).toBe(true);
26892689
}));
26902690

2691-
[true, false].forEach(function(isDeep) {
2692-
describe(isDeep ? 'deepWatch' : 'watch', function() {
2693-
it('should only become stable when all the properties of an object have defined values', inject(function($parse, $rootScope, log) {
2694-
var fn = $parse('::{foo: foo, bar: bar}');
2695-
$rootScope.$watch(fn, function(value) { log(value); }, isDeep);
2696-
2697-
expect(log.empty()).toEqual([]);
2698-
expect($rootScope.$$watchers.length).toBe(1);
2699-
2700-
$rootScope.$digest();
2701-
expect($rootScope.$$watchers.length).toBe(1);
2702-
expect(log.empty()).toEqual([{foo: undefined, bar: undefined}]);
2703-
2704-
$rootScope.foo = 'foo';
2705-
$rootScope.$digest();
2706-
expect($rootScope.$$watchers.length).toBe(1);
2707-
expect(log.empty()).toEqual([{foo: 'foo', bar: undefined}]);
2708-
2709-
$rootScope.foo = 'foobar';
2710-
$rootScope.bar = 'bar';
2711-
$rootScope.$digest();
2712-
expect($rootScope.$$watchers.length).toBe(0);
2713-
expect(log.empty()).toEqual([{foo: 'foobar', bar: 'bar'}]);
2714-
2715-
$rootScope.foo = 'baz';
2716-
$rootScope.$digest();
2717-
expect($rootScope.$$watchers.length).toBe(0);
2718-
expect(log.empty()).toEqual([]);
2719-
}));
2691+
it('should only become stable when all the properties of an object have defined values', inject(function($parse, $rootScope, log) {
2692+
var fn = $parse('::{foo: foo, bar: bar}');
2693+
$rootScope.$watch(fn, function(value) { log(value); }, true);
2694+
2695+
expect(log.empty()).toEqual([]);
2696+
expect($rootScope.$$watchers.length).toBe(1);
2697+
2698+
$rootScope.$digest();
2699+
expect($rootScope.$$watchers.length).toBe(1);
2700+
expect(log.empty()).toEqual([{foo: undefined, bar: undefined}]);
2701+
2702+
$rootScope.foo = 'foo';
2703+
$rootScope.$digest();
2704+
expect($rootScope.$$watchers.length).toBe(1);
2705+
expect(log.empty()).toEqual([{foo: 'foo', bar: undefined}]);
2706+
2707+
$rootScope.foo = 'foobar';
2708+
$rootScope.bar = 'bar';
2709+
$rootScope.$digest();
2710+
expect($rootScope.$$watchers.length).toBe(0);
2711+
expect(log.empty()).toEqual([{foo: 'foobar', bar: 'bar'}]);
2712+
2713+
$rootScope.foo = 'baz';
2714+
$rootScope.$digest();
2715+
expect($rootScope.$$watchers.length).toBe(0);
2716+
expect(log.empty()).toEqual([]);
2717+
}));
27202718

2721-
it('should only become stable when all the elements of an array have defined values', inject(function($parse, $rootScope, log) {
2722-
var fn = $parse('::[foo,bar]');
2723-
$rootScope.$watch(fn, function(value) { log(value); }, isDeep);
2719+
it('should only become stable when all the elements of an array have defined values', inject(function($parse, $rootScope, log) {
2720+
var fn = $parse('::[foo,bar]');
2721+
$rootScope.$watch(fn, function(value) { log(value); }, true);
27242722

2725-
expect(log.empty()).toEqual([]);
2726-
expect($rootScope.$$watchers.length).toBe(1);
2723+
expect(log.empty()).toEqual([]);
2724+
expect($rootScope.$$watchers.length).toBe(1);
27272725

2728-
$rootScope.$digest();
2729-
expect($rootScope.$$watchers.length).toBe(1);
2730-
expect(log.empty()).toEqual([[undefined, undefined]]);
2726+
$rootScope.$digest();
2727+
expect($rootScope.$$watchers.length).toBe(1);
2728+
expect(log.empty()).toEqual([[undefined, undefined]]);
27312729

2732-
$rootScope.foo = 'foo';
2733-
$rootScope.$digest();
2734-
expect($rootScope.$$watchers.length).toBe(1);
2735-
expect(log.empty()).toEqual([['foo', undefined]]);
2730+
$rootScope.foo = 'foo';
2731+
$rootScope.$digest();
2732+
expect($rootScope.$$watchers.length).toBe(1);
2733+
expect(log.empty()).toEqual([['foo', undefined]]);
27362734

2737-
$rootScope.foo = 'foobar';
2738-
$rootScope.bar = 'bar';
2739-
$rootScope.$digest();
2740-
expect($rootScope.$$watchers.length).toBe(0);
2741-
expect(log.empty()).toEqual([['foobar', 'bar']]);
2735+
$rootScope.foo = 'foobar';
2736+
$rootScope.bar = 'bar';
2737+
$rootScope.$digest();
2738+
expect($rootScope.$$watchers.length).toBe(0);
2739+
expect(log.empty()).toEqual([['foobar', 'bar']]);
27422740

2743-
$rootScope.foo = 'baz';
2744-
$rootScope.$digest();
2745-
expect($rootScope.$$watchers.length).toBe(0);
2746-
expect(log.empty()).toEqual([]);
2747-
}));
2741+
$rootScope.foo = 'baz';
2742+
$rootScope.$digest();
2743+
expect($rootScope.$$watchers.length).toBe(0);
2744+
expect(log.empty()).toEqual([]);
2745+
}));
27482746

2749-
it('should only become stable when all the elements of an array have defined values at the end of a $digest', inject(function($parse, $rootScope, log) {
2750-
var fn = $parse('::[foo]');
2751-
$rootScope.$watch(fn, function(value) { log(value); }, isDeep);
2752-
$rootScope.$watch('foo', function() { if ($rootScope.foo === 'bar') {$rootScope.foo = undefined; } });
2753-
2754-
$rootScope.foo = 'bar';
2755-
$rootScope.$digest();
2756-
expect($rootScope.$$watchers.length).toBe(2);
2757-
expect(log.empty()).toEqual([['bar'], [undefined]]);
2758-
2759-
$rootScope.foo = 'baz';
2760-
$rootScope.$digest();
2761-
expect($rootScope.$$watchers.length).toBe(1);
2762-
expect(log.empty()).toEqual([['baz']]);
2763-
2764-
$rootScope.bar = 'qux';
2765-
$rootScope.$digest();
2766-
expect($rootScope.$$watchers.length).toBe(1);
2767-
expect(log).toEqual([]);
2768-
}));
2769-
});
2770-
});
2747+
it('should only become stable when all the elements of an array have defined values at the end of a $digest', inject(function($parse, $rootScope, log) {
2748+
var fn = $parse('::[foo]');
2749+
$rootScope.$watch(fn, function(value) { log(value); }, true);
2750+
$rootScope.$watch('foo', function() { if ($rootScope.foo === 'bar') {$rootScope.foo = undefined; } });
2751+
2752+
$rootScope.foo = 'bar';
2753+
$rootScope.$digest();
2754+
expect($rootScope.$$watchers.length).toBe(2);
2755+
expect(log.empty()).toEqual([['bar'], [undefined]]);
2756+
2757+
$rootScope.foo = 'baz';
2758+
$rootScope.$digest();
2759+
expect($rootScope.$$watchers.length).toBe(1);
2760+
expect(log.empty()).toEqual([['baz']]);
2761+
2762+
$rootScope.bar = 'qux';
2763+
$rootScope.$digest();
2764+
expect($rootScope.$$watchers.length).toBe(1);
2765+
expect(log).toEqual([]);
2766+
}));
27712767
});
27722768
});
27732769

0 commit comments

Comments
 (0)