fix($compile): handle boolean attributes in @ bindings

jannic · gkalpak · commit 499e1b2adf27 · 2016-04-09T16:53:41.000+03:00
Commit db5e0ff handles initial values of boolean attributes. The same change needs to be applied inside the attrs.$observe() call. Closes #14070
diff --git a/src/ng/compile.js b/src/ng/compile.js
@@ -3135,7 +3135,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
               destination[scopeName] = attrs[attrName] = void 0;
             }
             attrs.$observe(attrName, function(value) {
-              if (isString(value)) {
+              if (isString(value) || isBoolean(value)) {
                 var oldValue = destination[scopeName];
                 recordChanges(scopeName, value, oldValue);
                 destination[scopeName] = value;
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
@@ -2711,6 +2711,25 @@ describe('$compile', function() {
                 expect(checkedVal).toEqual(true);
               });
             });
+
+            it('should handle updates to @ bindings on BOOLEAN attributes', function() {
+              var componentScope;
+              module(function($compileProvider) {
+                $compileProvider.directive('test', function() {
+                  return {
+                    scope: {checked: '@'},
+                    link: function(scope, element, attrs) {
+                      componentScope = scope;
+                      attrs.$set('checked', true);
+                    }
+                  };
+                });
+              });
+              inject(function($compile, $rootScope) {
+                $compile('<test></test>')($rootScope);
+                expect(componentScope.checked).toBe(true);
+              });
+            });
           });
 
 

Original file line number	Diff line number	Diff line change
`@@ -3135,7 +3135,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {`
`3135`	`3135`	`destination[scopeName] = attrs[attrName] = void 0;`
`3136`	`3136`	`}`
`3137`	`3137`	`attrs.$observe(attrName, function(value) {`
`3138`		`- if (isString(value)) {`
	`3138`	`+ if (isString(value) \|\| isBoolean(value)) {`
`3139`	`3139`	`var oldValue = destination[scopeName];`
`3140`	`3140`	`recordChanges(scopeName, value, oldValue);`
`3141`	`3141`	`destination[scopeName] = value;`