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">' + | 
