Merge pull request locutusjs#156 from glenara/patch-5

Update str_replace.js

Author: kvz

functions/strings/str_replace.js

@@ -23,7 +23,11 @@ function str_replace(search, replace, subject, count) {
   // bugfixed by: Glen Arason (http://CanadianDomainRegistry.ca)
   //   example 3: str_replace(Array('S','F'),'x','ASDFASDF');
   //   returns 3: 'AxDxAxDx'
-
+  // bugfixed by: Glen Arason (http://CanadianDomainRegistry.ca) Corrected count
+  //   example 4: str_replace(['A','D'], ['x','y'] , 'ASDFASDF' , 'cnt');
+  //   returns 4: 'xSyFxSyF' // cnt = 0 (incorrect before fix)
+  //   returns 4: 'xSyFxSyF' // cnt = 4 (correct after fix)
+  
   var i = 0,
     j = 0,
     temp = '',
@@ -62,9 +66,9 @@ function str_replace(search, replace, subject, count) {
       s[i] = (temp)
         .split(f[j])
         .join(repl);
-      if (count && s[i] !== temp) {
-        this.window[count] += (temp.length - s[i].length) / f[j].length;
-      }
+      if (count) {
+        this.window[count] += ((temp.split(f[j])).length - 1);
+      } 
     }
   }
   return sa ? s : s[0];