Merge pull request Jakobovski#58 from zortnac/bugFix/dottedName

Dotted strings in form name attribute

Author: Jakobovski

dist/angular-validator.js

@@ -1,7 +1,7 @@
 angular.module('angularValidator', []);
 
-angular.module('angularValidator').directive('angularValidator', ['$injector',
-    function($injector) {
+angular.module('angularValidator').directive('angularValidator', ['$injector', '$parse',
+    function($injector, $parse) {
         return {
             restrict: 'A',
             link: function(scope, element, attrs, fn) {
@@ -15,7 +15,7 @@ angular.module('angularValidator').directive('angularValidator', ['$injector',
                 // This is the the scope form model
                 // All validation states are contained here
                 var form_name = DOMForm.attributes['name'].value;
-                var scopeForm = scope[form_name];
+                var scopeForm = $parse(form_name)(scope);
 
                 // Set the default submitted state to false
                 scopeForm.submitted = false;

dist/angular-validator.min.js

@@ -1,7 +1,7 @@
 angular.module('angularValidator', []);
 
-angular.module('angularValidator').directive('angularValidator', ['$injector',
-    function($injector) {
+angular.module('angularValidator').directive('angularValidator', ['$injector', '$parse',
+    function($injector, $parse) {
         return {
             restrict: 'A',
             link: function(scope, element, attrs, fn) {
@@ -15,7 +15,7 @@ angular.module('angularValidator').directive('angularValidator', ['$injector',
                 // This is the the scope form model
                 // All validation states are contained here
                 var form_name = DOMForm.attributes['name'].value;
-                var scopeForm = scope[form_name];
+                var scopeForm = $parse(form_name)(scope);
 
                 // Set the default submitted state to false
                 scopeForm.submitted = false;

src/angular-validator.js

@@ -4,6 +4,35 @@
 
    var scope, compile;
 
+   describe('angularValidator with dotted name', function () {
+     var htmlForm, element, linkIt;
+
+     beforeEach(inject(function ($rootScope, $compile) {
+       scope = $rootScope.$new();
+       scope.object = {};
+
+       htmlForm = angular.element(
+           '<form name="object.form" angular-validator>' +
+           '<input ng-model="model.firstName" validate-on="dirty" name="firstName" type="text" required/>' +
+           '</form>'
+       );
+
+       linkIt = function () {
+         element = $compile(htmlForm)(scope);
+         scope.$digest();
+       };
+     }));
+
+     it('should not throw an error during linking', function () {
+       expect(linkIt).not.toThrow();
+     });
+
+     it('should correctly parse the dotted form name, evidenced by what\'s on scope', function () {
+       linkIt();
+       expect(scope.object.form).toBeDefined();
+     });
+   });
+   
   describe('angularValidator without form invalid message', function () {
    beforeEach(inject(function($rootScope, $compile) {
      scope = $rootScope.$new();