Hopiu/angular.js
1
0
Fork 0
mirror of https://github.com/Hopiu/angular.js.git synced 2026-03-17 07:40:22 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(ng-href): copy even if no binding

Browse source 
Closes# 850

fixed an issue where ng-href would not copy its content into href if it did not contain binding.
This commit is contained in:
Misko Hevery 2012-04-03 15:03:27 -07:00
parent 7e86eacf30
commit 2f5dba488e
2 changed files with 20 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
src/ng/directive/booleanAttrDirs.js
View file

@ -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);
});
}
};
}
};

9
test/ng/directive/booleanAttrDirSpecs.js
View file

@ -10,7 +10,14 @@ describe('boolean attr directives', function() {
it('should bind href', inject(function($rootScope, $compile) {
element = $compile('<a ng-href="{{url}}"></a>')($rootScope)
$rootScope.url = 'http://server'
$rootScope.url = 'http://server';
$rootScope.$digest();
expect(element.attr('href')).toEqual('http://server');
}));
it('should bind href even if no interpolation', inject(function($rootScope, $compile) {
element = $compile('<a ng-href="http://server"></a>')($rootScope)
$rootScope.$digest();
expect(element.attr('href')).toEqual('http://server');
}));