allowed 0 (zero) value for total/progress initialization #11

Author: AndiDittrich

.gitignore

@@ -6,4 +6,5 @@ gfx/*
 PublicHtml/*
 node_modules*
 .tmp*
-intern_*
+intern_*
+dev.js

CHANGES.md

@@ -1,3 +1,6 @@
+### 1.5.0 ###
+* Added: **0** values for total/progress initialization are allowed - feature requested by [jfmmm on GitHub](https://github.com/AndiDittrich/Node.CLI-Progress/issues/11) #11
+
 ### 1.4.0 ###
 * Added: **Preset/Theme support**. Different bar-styles can be loaded from internal library (in addition to full customization)
 * Added: Dependency **colors** for colorized progress bars 

lib/Bar.js

@@ -71,7 +71,8 @@ Bar.prototype.render = function(){
     var s = this.format;
 
     // calculate the normalized current progress
-    var progress = this.value/this.total;
+    // handle NaN Errors
+    var progress = (this.value/this.total) || 1.0;
 
     // limiter
     progress = Math.min(Math.max(progress, 0.0), 1.0);
@@ -150,7 +151,8 @@ Bar.prototype.formatTime = function (t, roundTo) {
 Bar.prototype.start = function(total, startValue){
     // set initial values
     this.value = startValue || 0;
-    this.total = total || 100;
+    this.total = (typeof total !== 'undefined' && total >= 0) ? total : 100;
+
     this.startTime = Date.now();
     this.lastDrawnString = '';
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "cli-progress",
-  "version": "1.4.1",
+  "version": "1.5.0",
   "description": "Easy to use Progress-Bar for Command-Line/Terminal Applications",
   "keywords": [
     "cli",