diff options
| author | Misko Hevery | 2012-04-03 15:03:27 -0700 |
|---|---|---|
| committer | Misko Hevery | 2012-04-03 16:02:20 -0700 |
| commit | 2f5dba488e855bcdbb9304aa809efcb9de7b43e9 (patch) | |
| tree | 73312f69811d1257b76eaf986ff5c6628adb5898 /src | |
| parent | 7e86eacf301934335c22908ec6dbd1a083d88fab (diff) | |
| download | angular.js-2f5dba488e855bcdbb9304aa809efcb9de7b43e9.tar.bz2 | |
fix(ng-href): copy even if no binding
Closes# 850
fixed an issue where ng-href would not copy its content into href if it did not contain binding.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ng/directive/booleanAttrDirs.js | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/src/ng/directive/booleanAttrDirs.js b/src/ng/directive/booleanAttrDirs.js index f0182610..01134998 100644 --- a/src/ng/directive/booleanAttrDirs.js +++ b/src/ng/directive/booleanAttrDirs.js @@ -302,13 +302,21 @@ forEach(['src', 'href'], function(attrName) { var normalized = directiveNormalize('ng-' + attrName); ngAttributeAliasDirectives[normalized] = function() { return { - priority: 100, + priority: 99, // it needs to run after the attributes are interpolated compile: function(tpl, attr) { return function(scope, element, attr) { - attr.$$observers[attrName] = []; - attr.$observe(normalized, function(value) { + var value = attr[normalized]; + if (value == undefined) { + // undefined value means that the directive is being interpolated + // so just register observer + attr.$$observers[attrName] = []; + attr.$observe(normalized, function(value) { + attr.$set(attrName, value); + }); + } else { + // value present means that no interpolation, so copy to native attribute. attr.$set(attrName, value); - }); + } }; } }; |
