Build: Fix the jscs upgrade, disable some rules for now

Author: jzaefferer

.jscsrc

@@ -15,5 +15,7 @@
 	"excludeFiles": [
 		"node_modules/**",
 		"dist/**"
-	]
+	],
+	"maximumLineLength": null,
+	"requireCamelCaseOrUpperCaseIdentifiers": null
 }

Gruntfile.js

@@ -195,7 +195,7 @@ grunt.loadNpmTasks("grunt-contrib-uglify");
 grunt.loadNpmTasks("grunt-contrib-concat");
 grunt.loadNpmTasks("grunt-contrib-compress");
 grunt.loadNpmTasks("grunt-contrib-watch");
-grunt.loadNpmTasks("grunt-jscs-checker");
+grunt.loadNpmTasks("grunt-jscs");
 grunt.loadNpmTasks("grunt-contrib-copy");
 grunt.loadNpmTasks("grunt-text-replace");
 

src/additional/dateITA.js

@@ -23,9 +23,9 @@ $.validator.addMethod("dateITA", function(value, element) {
 		adata, gg, mm, aaaa, xdata;
 	if ( re.test(value)) {
 		adata = value.split("/");
-		gg = parseInt(adata[0],10);
-		mm = parseInt(adata[1],10);
-		aaaa = parseInt(adata[2],10);
+		gg = parseInt(adata[0], 10);
+		mm = parseInt(adata[1], 10);
+		aaaa = parseInt(adata[2], 10);
 		xdata = new Date(aaaa, mm - 1, gg, 12, 0, 0, 0);
 		if ( ( xdata.getUTCFullYear() === aaaa ) && ( xdata.getUTCMonth () === mm - 1 ) && ( xdata.getUTCDate() === gg ) ) {
 			check = true;

src/additional/iban.js

@@ -9,7 +9,7 @@ $.validator.addMethod("iban", function(value, element) {
 	}
 
 	// remove spaces and to upper case
-	var iban = value.replace(/ /g,"").toUpperCase(),
+	var iban = value.replace(/ /g, "").toUpperCase(),
 		ibancheckdigits = "",
 		leadingZeroes = true,
 		cRest = "",
@@ -21,7 +21,7 @@ $.validator.addMethod("iban", function(value, element) {
 	}
 
 	// check the country code and find the country specific format
-	countrycode = iban.substring(0,2);
+	countrycode = iban.substring(0, 2);
 	bbancountrypatterns = {
 		"AL": "\\d{8}[\\dA-Z]{16}",
 		"AD": "\\d{8}[\\dA-Z]{12}",
@@ -105,7 +105,7 @@ $.validator.addMethod("iban", function(value, element) {
 	}
 
 	// now check the checksum, first convert to digits
-	ibancheck = iban.substring(4,iban.length) + iban.substring(0,4);
+	ibancheck = iban.substring(4, iban.length) + iban.substring(0, 4);
 	for (i = 0; i < ibancheck.length; i++) {
 		charAt = ibancheck.charAt(i);
 		if (charAt !== "0") {

src/additional/vinUS.js

@@ -23,7 +23,7 @@ $.validator.addMethod("vinUS", function(v) {
 
 	for (i = 0; i < 17; i++) {
 		f = FL[i];
-		d = v.slice(i,i + 1);
+		d = v.slice(i, i + 1);
 		if (i === 8) {
 			cdv = d;
 		}

src/core.js

@@ -496,7 +496,7 @@ $.extend( $.validator, {
 					.focus()
 					// manually trigger focusin event; without it, focusin handler isn't called, findLastActive won't have anything to find
 					.trigger( "focusin" );
-				} catch( e ) {
+				} catch ( e ) {
 					// ignore IE throwing errors when focusing hidden elements
 				}
 			}
@@ -613,7 +613,7 @@ $.extend( $.validator, {
 						this.formatAndAdd( element, rule );
 						return false;
 					}
-				} catch( e ) {
+				} catch ( e ) {
 					if ( this.settings.debug && window.console ) {
 						console.log( "Exception occurred when checking element " + element.id + ", check the '" + rule.method + "' method.", e );
 					}

test/methods.js

@@ -999,9 +999,9 @@ function fillFormWithValuesAndExpect(formSelector, inputValues, expected) {
 test("require_from_group", function() {
 	$("#productInfo").validate({
 		rules: {
-			partnumber: { require_from_group: [ 2,".productInfo" ] },
-			description: { require_from_group: [ 2,".productInfo" ] },
-			discount: { require_from_group: [ 2,".productInfo" ] }
+			partnumber: { require_from_group: [ 2, ".productInfo" ] },
+			description: { require_from_group: [ 2, ".productInfo" ] },
+			discount: { require_from_group: [ 2, ".productInfo" ] }
 		}
 	});
 
@@ -1018,9 +1018,9 @@ test("require_from_group", function() {
 test("require_from_group preserve other rules", function() {
 	$("#productInfo").validate({
 		rules: {
-			partnumber: { require_from_group: [ 2,".productInfo" ] },
-			description: { require_from_group: [ 2,".productInfo" ] },
-			color: { require_from_group: [ 2,".productInfo" ] },
+			partnumber: { require_from_group: [ 2, ".productInfo" ] },
+			description: { require_from_group: [ 2, ".productInfo" ] },
+			color: { require_from_group: [ 2, ".productInfo" ] },
 			supplier: { required: true }
 		}
 	});
@@ -1037,9 +1037,9 @@ test("require_from_group preserve other rules", function() {
 test("skip_or_fill_minimum", function() {
 	$("#productInfo").validate({
 		rules: {
-			partnumber:  { skip_or_fill_minimum: [ 2,".productInfo" ] },
-			description: { skip_or_fill_minimum: [ 2,".productInfo" ] },
-			color:       { skip_or_fill_minimum: [ 2,".productInfo" ] }
+			partnumber:  { skip_or_fill_minimum: [ 2, ".productInfo" ] },
+			description: { skip_or_fill_minimum: [ 2, ".productInfo" ] },
+			color:       { skip_or_fill_minimum: [ 2, ".productInfo" ] }
 		}
 	});
 
@@ -1052,9 +1052,9 @@ test("skip_or_fill_minimum", function() {
 test("skip_or_fill_minimum preserve other rules", function() {
 	$("#productInfo").validate({
 		rules: {
-			partnumber:  { skip_or_fill_minimum: [ 2,".productInfo" ] },
-			description: { skip_or_fill_minimum: [ 2,".productInfo" ] },
-			color:       { skip_or_fill_minimum: [ 2,".productInfo" ] },
+			partnumber:  { skip_or_fill_minimum: [ 2, ".productInfo" ] },
+			description: { skip_or_fill_minimum: [ 2, ".productInfo" ] },
+			color:       { skip_or_fill_minimum: [ 2, ".productInfo" ] },
 			supplier: { required: true }
 		}
 	});