diff options
| -rw-r--r-- | src/ng/directive/form.js | 4 | ||||
| -rw-r--r-- | test/ng/directive/formSpec.js | 29 |
2 files changed, 4 insertions, 29 deletions
diff --git a/src/ng/directive/form.js b/src/ng/directive/form.js index ec91f4b8..72f752be 100644 --- a/src/ng/directive/form.js +++ b/src/ng/directive/form.js @@ -306,13 +306,13 @@ var formDirectiveFactory = function(isNgForm) { alias = attr.name || attr.ngForm; if (alias) { - setter(scope, alias, controller, alias); + scope[alias] = controller; } if (parentFormCtrl) { formElement.bind('$destroy', function() { parentFormCtrl.$removeControl(controller); if (alias) { - setter(scope, alias, undefined, alias); + scope[alias] = undefined; } extend(controller, nullFormCtrl); //stop propagating child destruction handlers upwards }); diff --git a/test/ng/directive/formSpec.js b/test/ng/directive/formSpec.js index b5bbe535..f8e4cf63 100644 --- a/test/ng/directive/formSpec.js +++ b/test/ng/directive/formSpec.js @@ -83,11 +83,10 @@ describe('form', function() { }); - it('should support expression in form name', function() { + it('should allow form name to be an expression', function() { doc = $compile('<form name="obj.myForm"></form>')(scope); - expect(scope.obj).toBeDefined(); - expect(scope.obj.myForm).toBeTruthy(); + expect(scope['obj.myForm']).toBeTruthy(); }); @@ -326,30 +325,6 @@ describe('form', function() { }); - it('should deregister a child form whose name is an expression when its DOM is removed', function() { - doc = jqLite( - '<form name="parent">' + - '<div class="ng-form" name="child.form">' + - '<input ng:model="modelA" name="inputA" required>' + - '</div>' + - '</form>'); - $compile(doc)(scope); - scope.$apply(); - - var parent = scope.parent, - child = scope.child.form; - - expect(parent).toBeDefined(); - expect(child).toBeDefined(); - expect(parent.$error.required).toEqual([child]); - doc.children().remove(); //remove child - - expect(parent.child).toBeUndefined(); - expect(scope.child.form).toBeUndefined(); - expect(parent.$error.required).toBe(false); - }); - - it('should deregister a input when its removed from DOM', function() { doc = jqLite( '<form name="parent">' + |
