fixing bug on counter for unique value - closing issue #50

simogeo · Oct 17, 2019 · 07fdc71 · 07fdc71
1 parent a79d3ea
commit 07fdc71
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 27 deletions.
diff --git a/lib/geostats.js b/lib/geostats.js
@@ -174,8 +174,14 @@ var geostats = function(a) {
 		this.counter = new Array();
 
 		// we init counter with 0 value
-		for(i = 0; i < this.bounds.length -1; i++) {
-			this.counter[i]= 0;
+		if(this.is_uniqueValues == true) {
+			for(i = 0; i < this.bounds.length; i++) {
+				this.counter[i]= 0;
+			}
+		} else {
+			for(i = 0; i < this.bounds.length -1; i++) {
+				this.counter[i]= 0;
+			}
 		}
 
 		for(j=0; j < tmp.length; j++) {
@@ -184,7 +190,7 @@ var geostats = function(a) {
 			var cclass = this.getClass(tmp[j]);
 			this.counter[cclass]++;
 
-		}
+		}		
 
 	};
 
@@ -1127,8 +1133,11 @@ var geostats = function(a) {
 
 
 			if(this.is_uniqueValues == true) {
-				if(value == this.bounds[i])
+
+				if(value == this.bounds[i]) {
+					// console.log(value + ' - ' + this.bounds[i] + ' returned value : ' + i);
 					return i;
+				}
 			} else {
 				// parseFloat() is necessary
 				if(parseFloat(value) <= this.bounds[i + 1]) {