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

jannic · jannic · commit a19528910f7f · 2016-04-05T18:28:52.000+02: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
@@ -3120,7 +3120,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
@@ -2712,6 +2712,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
`@@ -3120,7 +3120,7 @@ function $CompileProvider($provide, $$sanitizeUriProvider) {`
`3120`	`3120`	`destination[scopeName] = attrs[attrName] = void 0;`
`3121`	`3121`	`}`
`3122`	`3122`	`attrs.$observe(attrName, function(value) {`
`3123`		`- if (isString(value)) {`
	`3123`	`+ if (isString(value) \|\| isBoolean(value)) {`
`3124`	`3124`	`var oldValue = destination[scopeName];`
`3125`	`3125`	`recordChanges(scopeName, value, oldValue);`
`3126`	`3126`	`destination[scopeName] = value;`