Fixing case where rounding results in the wrong exponent, fixes #103

avoidwork · Sep 25, 2019 · b326ec5 · b326ec5
1 parent 5b362af
commit b326ec5
Show file tree

Hide file tree

Showing 10 changed files with 83 additions and 71 deletions.
diff --git a/lib/filesize.es6.js b/lib/filesize.es6.js
@@ -3,7 +3,7 @@
  *
  * @copyright 2019 Jason Mulligan <[email protected]>
  * @license BSD-3-Clause
- * @version 4.2.0
+ * @version 4.2.1
  */
 (function (global) {
 	const b = /^(b|B)$/,
@@ -97,6 +97,12 @@
 			}
 
 			result[0] = Number(val.toFixed(e > 0 ? round : 0));
+
+			if (result[0] === ceil && e < 8 && descriptor.exponent === void 0) {
+				result[0] = 1;
+				e++;
+			}
+
 			result[1] = base === 10 && e === 1 ? bits ? "kb" : "kB" : symbol[standard][bits ? "bits" : "bytes"][e];
 
 			if (unix) {

diff --git a/lib/filesize.es6.min.js b/lib/filesize.es6.min.js
diff --git a/lib/filesize.es6.min.js.map b/lib/filesize.es6.min.js.map
diff --git a/lib/filesize.js b/lib/filesize.js
@@ -5,7 +5,7 @@
  *
  * @copyright 2019 Jason Mulligan <[email protected]>
  * @license BSD-3-Clause
- * @version 4.2.0
+ * @version 4.2.1
  */
 (function (global) {
   var b = /^(b|B)$/,
@@ -115,6 +115,12 @@
       }
 
       result[0] = Number(val.toFixed(e > 0 ? round : 0));
+
+      if (result[0] === ceil && e < 8 && descriptor.exponent === void 0) {
+        result[0] = 1;
+        e++;
+      }
+
       result[1] = base === 10 && e === 1 ? bits ? "kb" : "kB" : symbol[standard][bits ? "bits" : "bytes"][e];
 
       if (unix) {

diff --git a/lib/filesize.min.js b/lib/filesize.min.js
diff --git a/lib/filesize.min.js.map b/lib/filesize.min.js.map