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

jannic · jannic · commit cdff6d368f75 · 2016-02-17T21:29:14.000+01:00
Commit db5e0ff handles initial values of boolean attributes. The same change needs to be applied inside the attrs.$observe() call.
diff --git a/src/ng/compile.js b/src/ng/compile.js
@@ -2994,7 +2994,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {
               destination[scopeName] = attrs[attrName] = void 0;
             }
             attrs.$observe(attrName, function(value) {
-              if (isString(value)) {
+              if (isString(value) || isBoolean(value)) {
                 destination[scopeName] = value;
               }
             });
diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js
@@ -2713,6 +2713,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('<input test>')($rootScope);
+                expect(componentScope.checked).toEqual(true);
+              });
+            });
           });
 
 

Original file line number	Diff line number	Diff line change
`@@ -2994,7 +2994,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {`
`2994`	`2994`	`destination[scopeName] = attrs[attrName] = void 0;`
`2995`	`2995`	`}`
`2996`	`2996`	`attrs.$observe(attrName, function(value) {`
`2997`		`- if (isString(value)) {`
	`2997`	`+ if (isString(value) \|\| isBoolean(value)) {`
`2998`	`2998`	`destination[scopeName] = value;`
`2999`	`2999`	`}`
`3000`	`3000`	`});`