Change Grunt tasks to match new directory structure

-Fix JSHint to check language files (it wasn't checking those before)
-Fix JSHint errors
-Add task to minify CSS
-Bump dependencies
-Reorganize tasks. Default task is now a dev task which watches directory for changes and
does tests/builds upon changes, but not minification; "build-production" task bumps version,
does tests, and minifies output

Author: ray-print

Gruntfile.js

@@ -1,27 +1,23 @@
-
-const SRC_DIR = './form-validator/src/';
-const MODULE_DIR = './form-validator/';
-const LANG_DIR = './form-validator/lang/';
+//TODO: During next major version bump change to /dist. Leaving at ./form-validator for backwards
+//compabilitiy
+const DIST_DIR = './form-validator';
 const MAIN_PLUGIN_FILE = 'form-validator/jquery.form-validator.min.js';
-const JS_EXTENSION = '.js';
-const DEV_EXTENSION = '.dev.js';
+const SRC_DIR = './src';
+const MAIN_DIR = '/main/';
+const MODULE_DIR = '/modules/';
+const LANG_DIR = '/lang/';
+const CSS_FILE = 'theme-default.css';
+
 
 var fs = require('fs'),
   filesToBuild = {
     uglify: {},
     concat: {
       main:{
-        src:[SRC_DIR+'core-validators.js'],
+        src: [SRC_DIR + MAIN_DIR+'core-validators.js'],
         dest: MAIN_PLUGIN_FILE
       }
-    },
-    devFiles: []
-  },
-  isJavascriptFile = function(fileName) {
-    return fileName.substr(-3) == JS_EXTENSION;
-  },
-  isDevFile = function(fileName) {
-    return fileName.substr(-1 * DEV_EXTENSION.length) == DEV_EXTENSION;
+    }
   },
   readFile = function (file) {
     return fs.readFileSync(file, 'utf-8');
@@ -31,31 +27,40 @@ var fs = require('fs'),
   };
 
 module.exports = function (grunt) {
-
   // Gather up all module and language files
   [MODULE_DIR, LANG_DIR].forEach(function (path) {
-    fs.readdirSync(path).forEach(function (fileName) {
-      if (isDevFile(fileName)) {
-        var name = fileName.substr(0, fileName.length - DEV_EXTENSION.length),
-          fullPath = path + name + JS_EXTENSION;
-
-        filesToBuild.uglify[fullPath] = [fullPath];
-        filesToBuild.concat['file' + name] = {
-          src: [path + fileName],
-          dest: path + name + JS_EXTENSION
+    var srcPath = SRC_DIR + path;
+    var distPath = DIST_DIR + path;
+    if (path === MODULE_DIR) {
+      distPath = DIST_DIR + '/';
+    }
+
+    fs.readdirSync(srcPath).forEach(function (fileName) {
+        var fullPath = srcPath + fileName;
+        filesToBuild.uglify[distPath + fileName] = fullPath;
+        filesToBuild.concat[fullPath] = {
+          src: [fullPath],
+          dest: distPath + fileName
         };
-        filesToBuild.devFiles.push(path + fileName);
-      }
     });
   });
+
   // Gather up all source files that will added to minified core library
-  fs.readdirSync(SRC_DIR).forEach(function (fileName) {
-    var fullPath = SRC_DIR + fileName;
-    if (isJavascriptFile(fileName) && filesToBuild.concat.main.src.indexOf(fullPath) == -1) {
+  fs.readdirSync(SRC_DIR + MAIN_DIR).forEach(function (fileName) {
+    var fullPath = SRC_DIR + MAIN_DIR + fileName;
+    if (filesToBuild.concat.main.src.indexOf(fullPath) === -1) {
       filesToBuild.concat.main.src.unshift(fullPath);
     }
   });
 
+  filesToBuild.cssFiles = [];
+  filesToBuild.cssFiles.push({
+    dest: DIST_DIR,
+    src: CSS_FILE,
+    cwd: SRC_DIR,
+    expand: true
+  });
+
   // Add options for concat and uglify
   filesToBuild.concat.options = {
     banner: "<%= meta.banner %>"
@@ -87,11 +92,17 @@ module.exports = function (grunt) {
     // Concat definitions.
     concat: filesToBuild.concat,
 
+    cssmin: {
+        target: {
+          files: filesToBuild.cssFiles
+        }
+    },
     // Lint definitions
     jshint: {
-      files: [MODULE_DIR+"*"+DEV_EXTENSION, SRC_DIR+"*.js"],
+      files: [SRC_DIR + '/*'],
       options: {
-        jshintrc: ".jshintrc"
+        jshintrc: ".jshintrc",
+        ignores: [SRC_DIR + '/' + CSS_FILE]
       }
     },
 
@@ -102,8 +113,8 @@ module.exports = function (grunt) {
     // Better than calling grunt a million times
     // (call 'grunt watch')
     watch: {
-      files: [SRC_DIR+'/*', LANG_DIR+'/*', MODULE_DIR+'/*'],
-      tasks: ['build'],
+      files: [SRC_DIR + '/**'],
+      tasks: ['test'],
       options : { nospawn : true }
     },
 
@@ -143,7 +154,8 @@ module.exports = function (grunt) {
 
     grunt.log.writeln('* Moving from version ' + currentVersion + ' to ' + newVersion);
 
-    replaceInFile('package.json', '"version": "' + currentVersion + '"', '"version": "' + newVersion + '"');
+    replaceInFile('package.json', '"version": "' + currentVersion + '"',
+      '"version": "' + newVersion + '"');
     replaceInFile('formvalidator.jquery.json', '"version": "' + currentVersion + '"', '"version": "' + newVersion + '"');
 
     // Set new version globally (can later on be used by concat/uglify)
@@ -158,10 +170,10 @@ module.exports = function (grunt) {
   grunt.loadNpmTasks("grunt-contrib-watch");
   grunt.loadNpmTasks('grunt-contrib-connect');
   grunt.loadNpmTasks('grunt-contrib-qunit');
-
-  grunt.registerTask("build", ["version", "concat", "uglify"]);
-  grunt.registerTask('test', ['concat', 'jshint', 'qunit']);
-  grunt.registerTask('test-uglify', ['concat', 'uglify', 'jshint', 'qunit']);
-  grunt.registerTask("default", ["test", "build"]);
-
+  grunt.loadNpmTasks('grunt-contrib-cssmin');
+  grunt.registerTask("build-production", ["version", "cssmin", "test", "uglify"]);
+  grunt.registerTask('test', ['concat', 'cssmin','jshint', 'qunit']);
+  grunt.registerTask("default", ["test", "watch"]);
+  //TODO: add clean task, don't minify CSS in dev build, ?remove volo (its {version} is busted anyway)
+  //Add unminified CSS to prod build
 };

package.json

@@ -30,9 +30,10 @@
     "grunt-cli": "~0.1.13",
     "grunt-contrib-concat": "^0.5.1",
     "grunt-contrib-connect": "^0.11.2",
-    "grunt-contrib-jshint": "^0.11.0",
+    "grunt-contrib-cssmin": "~0.14.0",
+    "grunt-contrib-jshint": "~1.0.0",
     "grunt-contrib-qunit": "^0.7.0",
-    "grunt-contrib-uglify": "^0.8.0",
+    "grunt-contrib-uglify": "~0.11.1",
     "grunt-contrib-watch": "^0.6.1",
     "jquery": "^2.1.4",
     "numeral": "~1.5.3",

src/lang/it.js

@@ -9,7 +9,6 @@
  * @version 2.2.163
  */
 (function($, window) {
-
   'use strict';
 
   $(window).bind('validatorsLoaded', function() {
@@ -18,7 +17,7 @@
       errorTitle: 'Impossibile inviare il modulo!',
       requiredField: 'Campo obbligatorio',
       requiredFields: 'Non sono stati compilati tutti i campi richiesti',
-      badTime: "L'ora scelta non è valida",
+      badTime: 'L\'ora scelta non è valida',
       badEmail: 'Questo indirizzo email non è valido',
       badTelephone: 'Il numero di telefono imputato non è valido',
       badSecurityAnswer: 'La risposta alla domanda di sicurezza è errata',
@@ -29,7 +28,7 @@
       lengthTooShortStart: 'La lunghezza della risposta deve essere maggiore di ',
       notConfirmed: 'Los valores proporcionados no pudieron ser confirmados',
       badDomain: 'Il dominio inserito non è corretto.',
-      badUrl: "L' URL inserito non è valido",
+      badUrl: 'L\' URL inserito non è valido',
       badCustomVal: 'I valori inseriti non sono validi',
       andSpaces: ' e spazi ',
       badInt: 'Il numero inserito non è valido',
@@ -48,13 +47,13 @@
       groupCheckedEnd: ' opzione/i',
       badCreditCard: 'Il numero di carta di credito non risulta valido',
       badCVV: 'CVV non valido',
-      wrongFileDim: "La dimensione dell'immagine non è valida,",
-      imageTooTall: "il lato alto dell'immagine non può essere maggiore di ",
-      imageTooWide: "il lato lungo dell'immagine non può essere maggiore di",
-      imageTooSmall: "L'immagine è troppo piccola",
+      wrongFileDim: 'La dimensione dell\'immagine non è valida,',
+      imageTooTall: 'il lato alto dell\'immagine non può essere maggiore di',
+      imageTooWide: 'il lato lungo dell\'immagine non può essere maggiore di',
+      imageTooSmall: 'L\'immagine è troppo piccola',
       min: 'min.',
       max: 'máx.',
-      imageRatioNotAccepted : "La proporzione dell' immagine (altezza x larghezza) non è valida"
+      imageRatioNotAccepted : 'La proporzione dell\' immagine (altezza x larghezza) non è valida'
     };
 
   });

src/lang/ru.js

@@ -8,8 +8,7 @@
  * @license MIT
  */
 (function($, window) {
-
-  "use strict";
+  'use strict';
 
   $(window).bind('validatorsLoaded', function() {