Should fix issue locutusjs#24: money_format wrongly formatting hundreds of thousands

kukawski · kukawski · commit 47d821c64b9b · 2011-12-08T09:26:25.000+01:00
diff --git a/functions/strings/money_format.js b/functions/strings/money_format.js
@@ -86,7 +86,7 @@ function money_format (format, number) {
             if (mon_grouping[0] < integer.length) {
                 for (var i = 0, idx = integer.length; i < mon_grouping.length; i++) {
                     idx -= mon_grouping[i]; // e.g., 3
-                    if (idx < 0) {
+                    if (idx <= 0) {
                         break;
                     }
                     if (filler && idx < fillnum) {

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ function money_format (format, number) {`
`86`	`86`	`if (mon_grouping[0] < integer.length) {`
`87`	`87`	`for (var i = 0, idx = integer.length; i < mon_grouping.length; i++) {`
`88`	`88`	`idx -= mon_grouping[i]; // e.g., 3`
`89`		`- if (idx < 0) {`
	`89`	`+ if (idx <= 0) {`
`90`	`90`	`break;`
`91`	`91`	`}`
`92`	`92`	`if (filler && idx < fillnum) {`