Changing all indentation to 2 spaces as per style guide. #6

Author: sagrawal31

.jshintrc

@@ -7,11 +7,11 @@
     "curly": true,
     "eqeqeq": true,
     "immed": true,
-    "indent": 4,
+    "indent": 2,
     "latedef": true,
     "newcap": true,
     "noarg": true,
-    "quotmark": "double",
+    "quotmark": "single",
     "regexp": true,
     "undef": true,
     "unused": true,

ChangeLog.md

@@ -2,4 +2,4 @@
 
 ## 0.0.1
 
-First version of the basic validtion in Angular with Bootstrap like jQuery.
+First version of the basic validation in Angular with Bootstrap like jQuery.

Gruntfile.js

@@ -1,49 +1,51 @@
 /* global module:false */
 
-"use strict";
+'use strict';
 
 module.exports = function (grunt) {
-    var yeomanConfig;
-    require("matchdep").filterDev("grunt-*").forEach(grunt.loadNpmTasks);
-    yeomanConfig = {
-        src: "src",
-        dist: "dist"
-    };
-    return grunt.initConfig({
-        yeoman: yeomanConfig,
-        uglify: {
-            build: {
-                src: ["<%=yeoman.src %>/app.js", "<%=yeoman.src %>/**/*.js"],
-                dest: "<%=yeoman.dist %>/bootstrap-angular-validation.min.js"
-            }
-        },
-        bump: {
-            options: {
-                files: ["package.json"],
-                updateConfigs: [],
-                commit: true,
-                commitMessage: "Release v%VERSION%",
-                commitFiles: ["package.json"],
-                createTag: true,
-                tagName: "v%VERSION%",
-                tagMessage: "Version %VERSION%",
-                push: true,
-                pushTo: "origin",
-                gitDescribeOptions: "--tags --always --abbrev=1 --dirty=-d",
-                globalReplace: false,
-                prereleaseName: false,
-                metadata: "",
-                regExp: false
-            }
-        },
-        watch: {
-            scripts: {
-                files: ["<%=yeoman.src %>/**/*.js"],
-                tasks: ["uglify"],
-                options: {
-                    spawn: false
-                }
-            }
+  var yeomanConfig;
+  require('matchdep').filterDev('grunt-*').forEach(grunt.loadNpmTasks);
+
+  yeomanConfig = {
+    src: 'src',
+    dist: 'dist'
+  };
+
+  return grunt.initConfig({
+    yeoman: yeomanConfig,
+    uglify: {
+      build: {
+        src: ['<%=yeoman.src %>/app.js', '<%=yeoman.src %>/**/*.js'],
+        dest: '<%=yeoman.dist %>/bootstrap-angular-validation.min.js'
+      }
+    },
+    bump: {
+      options: {
+        files: ['package.json'],
+        updateConfigs: [],
+        commit: true,
+        commitMessage: 'Release v%VERSION%',
+        commitFiles: ['package.json'],
+        createTag: true,
+        tagName: 'v%VERSION%',
+        tagMessage: 'Version %VERSION%',
+        push: true,
+        pushTo: 'origin',
+        gitDescribeOptions: '--tags --always --abbrev=1 --dirty=-d',
+        globalReplace: false,
+        prereleaseName: false,
+        metadata: '',
+        regExp: false
+      }
+    },
+    watch: {
+      scripts: {
+        files: ['<%=yeoman.src %>/**/*.js'],
+        tasks: ['uglify'],
+        options: {
+          spawn: false
         }
-    }, grunt.registerTask("default", ["uglify"]));
+      }
+    }
+  }, grunt.registerTask('default', ['uglify']));
 };

src/app.js

@@ -1,40 +1,40 @@
 /* global document */
 
-"use strict";
+'use strict';
 
-angular.module("bootstrap.angular.validation", []).config(["$httpProvider", function ($httpProvider) {
-    var interceptor = ["$rootScope", "BsValidationService", function($rootScope, bsValidationService) {
-        return {
-            "response": function(response) {
-                /**
-                 * Intercept all ng-included templates in a form and add validation-errors directive. See form
-                 * directive in this component itself.
-                 */
-                if (bsValidationService.checkNgIncludedURL(response.config.url)) {
-                    /**
-                     * Empty "div" is being used to temporarily store the received HTML from the server to find out
-                     * all the child form elements to add the "bs-validation" directive. And then later the innerHTML
-                     * i.e. the received HTML can easily be extracted using the html() and the temporary "div" will
-                     * be discarded.
-                     *
-                     * http://stackoverflow.com/a/652771/2405040
-                     */
-                    var parsedJQLiteElements = angular.element(document.createElement("div"));
-                    parsedJQLiteElements.append(response.data);
-                    bsValidationService.addDirective(parsedJQLiteElements);
+angular.module('bootstrap.angular.validation', []).config(['$httpProvider', function ($httpProvider) {
+  var interceptor = ['$rootScope', 'BsValidationService', function ($rootScope, bsValidationService) {
+    return {
+      'response': function (response) {
+        /**
+         * Intercept all ng-included templates in a form and add validation-errors directive. See form
+         * directive in this component itself.
+         */
+        if (bsValidationService.checkNgIncludedURL(response.config.url)) {
+          /**
+           * Empty 'div' is being used to temporarily store the received HTML from the server to find out
+           * all the child form elements to add the 'bs-validation' directive. And then later the innerHTML
+           * i.e. the received HTML can easily be extracted using the html() and the temporary 'div' will
+           * be discarded.
+           *
+           * http://stackoverflow.com/a/652771/2405040
+           */
+          var parsedJQLiteElements = angular.element(document.createElement('div'));
+          parsedJQLiteElements.append(response.data);
+          bsValidationService.addDirective(parsedJQLiteElements);
 
-                    /*
-                     * Get the modified HTML template which is now having the "bs-validation" directive.
-                     * (Not iterating over the every jQlite element otherwise it will append a string "undefined"
-                     * for every new empty line in the HTML)
-                     */
-                    response.data = parsedJQLiteElements.html();
-                }
+          /*
+           * Get the modified HTML template which is now having the 'bs-validation' directive.
+           * (Not iterating over the every jQlite element otherwise it will append a string 'undefined'
+           * for every new empty line in the HTML)
+           */
+          response.data = parsedJQLiteElements.html();
+        }
 
-                return response;
-            }
-        };
-    }];
+        return response;
+      }
+    };
+  }];
 
-    $httpProvider.interceptors.push(interceptor);
+  $httpProvider.interceptors.push(interceptor);
 }]);

src/directives/validation.directive.js

@@ -70,7 +70,7 @@ angular.module('bootstrap.angular.validation').directive('bsValidation', [
             }, displayOrHideError);
 
             $scope.$watchCollection(function() {
-              return ngModelController.$valid
+              return ngModelController.$valid;
             }, displayOrHideError);
 
             function resolveMessage(key) {
@@ -118,7 +118,7 @@ angular.module('bootstrap.angular.validation').directive('bsValidation', [
             function addErrorMessage(){
               var message = resolveMessage(Object.keys(ngModelController.$error)[0]);
               var errorElement = formGroupElement.findOne('.' + errorElementClass);
-              if (errorElement.length === 0) { errorContainer(message) }
+              if (errorElement.length === 0) { errorContainer(message); }
               errorElement.html(iconMarkup + message).removeClass('ng-hide');
             }
 

src/utils.js

@@ -13,8 +13,8 @@
  * var element = domElement.findOne(".form");
  * var element = domElement.findOne("#form");
  */
-angular.element.prototype.findOne = function(selector) {
-    return angular.element(this[0].querySelector(selector));
+angular.element.prototype.findOne = function (selector) {
+  return angular.element(this[0].querySelector(selector));
 };
 
 /**
@@ -29,8 +29,8 @@ angular.element.prototype.findOne = function(selector) {
  * var elementList = domElement.findAll(".form");
  * var elementList = domElement.findAll("#form");
  */
-angular.element.prototype.findAll = function(selector) {
-    return angular.element(this[0].querySelectorAll(selector));
+angular.element.prototype.findAll = function (selector) {
+  return angular.element(this[0].querySelectorAll(selector));
 };
 
 /**
@@ -45,36 +45,36 @@ angular.element.prototype.findAll = function(selector) {
  * var parents = domElement.parents(".form");
  * var parents = domElement.parents("#form");
  */
-angular.element.prototype.parents = function(selector) {
-    // At least IE6
-    var isIE = /*@cc_on!@*/false || !!document.documentMode;
+angular.element.prototype.parents = function (selector) {
+  // At least IE6
+  var isIE = /*@cc_on!@*/false || !!document.documentMode;
 
-    var element = this[0];
-    var parents = [];
+  var element = this[0];
+  var parents = [];
 
-    // Traverse element's ancestor tree until the upto the root and pushing all the ancestors to an array.
-    while (element.parentNode.parentNode) {
-        element = element.parentNode;
+  // Traverse element's ancestor tree until the upto the root and pushing all the ancestors to an array.
+  while (element.parentNode.parentNode) {
+    element = element.parentNode;
 
-        // Get the ancestors of element filtered by a selector (if any)
-        if (selector) {
-            if (isIE) {
-                /*
-                 * For IE versions less then 9. Since Microsoft introduces msMatchesSelector method from 9.
-                 * @see Browser compatibility in https://developer.mozilla.org/en-US/docs/Web/API/Element/matches
-                 */
-                if (element.msMatchesSelector(selector)) {
-                    parents.push(element);
-                }
-            } else {
-                if (element.matches(selector)) {
-                    parents.push(element);
-                }
-            }
-        } else {
-            // If no selector is defined then get all the ancestors with no filtering
-            parents.push(element);
+    // Get the ancestors of element filtered by a selector (if any)
+    if (selector) {
+      if (isIE) {
+        /*
+         * For IE versions less then 9. Since Microsoft introduces msMatchesSelector method from 9.
+         * @see Browser compatibility in https://developer.mozilla.org/en-US/docs/Web/API/Element/matches
+         */
+        if (element.msMatchesSelector(selector)) {
+          parents.push(element);
         }
+      } else {
+        if (element.matches(selector)) {
+          parents.push(element);
+        }
+      }
+    } else {
+      // If no selector is defined then get all the ancestors with no filtering
+      parents.push(element);
     }
-    return angular.element(parents);
+  }
+  return angular.element(parents);
 };