@@ -44,10 +44,11 @@ SC.String = { */ fmt: function() { // first, replace any ORDERED replacements. + var args = arguments; var str = this.gsub(/%@([0-9]+)/, function(m) { - return (arguments[parseInt(m[1],0)-1] || '').toString(); + return (args[parseInt(m[1],0)-1] || '').toString(); }) ; - + // now, replace any remaining %@ items. Use this indexOf() method b/c // it is faster than split(). var ret = [] ; @@ -60,7 +61,7 @@ SC.String = { loc = idx + 2 ; // 2 to skip '%@'. // add in replacement. - var value = arguments[argIdx++] ; + var value = args[argIdx++] ; if (value && value.toString) value = value.toString() ; ret.push(value) ; } foundation/string.js 39f836a8b77a50b5ceed7c060cba63e9871992db Erich Ocean erich@atlasocean.com http://github.com/mauritslamers/sproutcore/commit/4f814eed40bc26d7838a5a923efc47d578aab9ed 4f814eed40bc26d7838a5a923efc47d578aab9ed 2008-09-30T01:30:20-07:00 2008-09-30T01:30:20-07:00 Fix [165] String.fmt ordered arguments are broken 4ae9fcf0855b00e388818e8f736ac02a8a4154da Erich Ocean erich@atlasocean.com