From ebe9178ce69d89a273e5fb886aaa8eecd4c098c8 Mon Sep 17 00:00:00 2001
From: "Diego F.(EW7)" <diegoefe@gmail.com>
Date: Tue, 20 Dec 2016 14:00:39 -0300
Subject: [PATCH] =?UTF-8?q?=C3=B3rden=20y=20warnings?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 lib/txt-to-sql.js | 27 ++++++++++++---------------
 test/test.js      |  8 +++++---
 web/txt-to-sql.js | 27 ++++++++++++---------------
 3 files changed, 29 insertions(+), 33 deletions(-)

diff --git a/lib/txt-to-sql.js b/lib/txt-to-sql.js
index b14d91e..34b5822 100644
--- a/lib/txt-to-sql.js
+++ b/lib/txt-to-sql.js
@@ -24,14 +24,6 @@ function filling(columnLength, val) { return val.length>=columnLength?'':new Arr
 function padLeft(columnLength, val) { return val+filling(columnLength, val); }
 function padRight(columnLength, val) { return filling(columnLength,val)+val; }
 
-function evaluateColumn(column, rows, regex) {
-    for(var row=0; row<rows.length; ++row) {
-        if(rows[row][column] && ! rows[row][column].match(regex)) {
-            return false;
-        }
-    }
-    return true;
-}
 var booleanRegExps = [
     {t:/^1$/,       f:/^0$/},
     {t:/^1$/,       f:/^2$/},
@@ -49,23 +41,28 @@ function isBoolean(column, rows) {
             if(vals.length>2) { return false; }
         }
     }
-    if(vals.length==2) {
+    if(vals.length===2) {
         return booleanRegExps.some(function(rgx) {
-            //console.log("vals", vals, '->', rgx)
-            //console.log(' ',vals[0], vals[0].match(rgx.t)?"OK":"nop")
-            //console.log(' ',vals[1], vals[1].match(rgx.f)?"OK":"nop")
-            return (vals[0].match(rgx.t) && vals[1].match(rgx.f))
-                    || (vals[1].match(rgx.t) && vals[0].match(rgx.f));
+            return (vals[0].match(rgx.t) && vals[1].match(rgx.f)) ||
+                   (vals[1].match(rgx.t) && vals[0].match(rgx.f));
         });
     } else {
         return vals.some(function(val) {
             return booleanRegExps.some(function(rgx) {
-                return val.match(rgx.f) || val.match(rgx.t)
+                return val.match(rgx.f) || val.match(rgx.t);
             });
         });
     }
 }
 
+function evaluateColumn(column, rows, regex) {
+    for(var row=0; row<rows.length; ++row) {
+        if(rows[row][column] && ! rows[row][column].match(regex)) {
+            return false;
+        }
+    }
+    return true;
+}
 function isInteger(column, rows) {
     return evaluateColumn(column, rows, /^-?[0-9]{1,5}$/);
 }
diff --git a/test/test.js b/test/test.js
index 4581813..c5cc6eb 100644
--- a/test/test.js
+++ b/test/test.js
@@ -289,16 +289,18 @@ describe("datatype validation", function(){
         var b = txtToSql.typeValidations['boolean'].checkArray;
         expect(b1('null')).to.be.ok(); // coverage
         expect(b1('')).to.not.be.ok(); // coverage
-        // good
-        expect(b(0, [['1'],['1'],['0'],['1']])).to.be.ok();
+        // good, 1 value
         expect(b(0, [['uno']])).to.be.ok();
         expect(b(0, [[''],['uno']])).to.be.ok();
+        expect(b(0, [['true'],['true'],['true']])).to.be.ok();
+        expect(b(0, [['pi'],['pi'],['pi']])).to.be.ok();
+        // good, 2 values
+        expect(b(0, [['1'],['1'],['0'],['1']])).to.be.ok();
         expect(b(0, [['1'],['0'],[''],['0'],['1']])).to.be.ok();
         expect(b(0, [[''],['1'],['0'],['0'],['1'],['']])).to.be.ok();
         expect(b(0, [['1'],['2'],['2'],['1'],['']])).to.be.ok();
         expect(b(0, [['si'],['si']])).to.be.ok();
         expect(b(0, [['no'],['no'],['si'],['no'],['si']])).to.be.ok();
-        expect(b(0, [['true'],['true'],['true']])).to.be.ok();
         expect(b(0, [['true'],['true'],['false']])).to.be.ok();
         // bad
         expect(b(0, [['3'],['1'],['0']])).to.not.be.ok();
diff --git a/web/txt-to-sql.js b/web/txt-to-sql.js
index b14d91e..34b5822 100644
--- a/web/txt-to-sql.js
+++ b/web/txt-to-sql.js
@@ -24,14 +24,6 @@ function filling(columnLength, val) { return val.length>=columnLength?'':new Arr
 function padLeft(columnLength, val) { return val+filling(columnLength, val); }
 function padRight(columnLength, val) { return filling(columnLength,val)+val; }
 
-function evaluateColumn(column, rows, regex) {
-    for(var row=0; row<rows.length; ++row) {
-        if(rows[row][column] && ! rows[row][column].match(regex)) {
-            return false;
-        }
-    }
-    return true;
-}
 var booleanRegExps = [
     {t:/^1$/,       f:/^0$/},
     {t:/^1$/,       f:/^2$/},
@@ -49,23 +41,28 @@ function isBoolean(column, rows) {
             if(vals.length>2) { return false; }
         }
     }
-    if(vals.length==2) {
+    if(vals.length===2) {
         return booleanRegExps.some(function(rgx) {
-            //console.log("vals", vals, '->', rgx)
-            //console.log(' ',vals[0], vals[0].match(rgx.t)?"OK":"nop")
-            //console.log(' ',vals[1], vals[1].match(rgx.f)?"OK":"nop")
-            return (vals[0].match(rgx.t) && vals[1].match(rgx.f))
-                    || (vals[1].match(rgx.t) && vals[0].match(rgx.f));
+            return (vals[0].match(rgx.t) && vals[1].match(rgx.f)) ||
+                   (vals[1].match(rgx.t) && vals[0].match(rgx.f));
         });
     } else {
         return vals.some(function(val) {
             return booleanRegExps.some(function(rgx) {
-                return val.match(rgx.f) || val.match(rgx.t)
+                return val.match(rgx.f) || val.match(rgx.t);
             });
         });
     }
 }
 
+function evaluateColumn(column, rows, regex) {
+    for(var row=0; row<rows.length; ++row) {
+        if(rows[row][column] && ! rows[row][column].match(regex)) {
+            return false;
+        }
+    }
+    return true;
+}
 function isInteger(column, rows) {
     return evaluateColumn(column, rows, /^-?[0-9]{1,5}$/);
 }