Fixed several string functions to be weakly typed, so you're allowed to pass the value 6 instead of '6'.

onnomarsman · onnomarsman · commit de6923f17766 · 2008-10-06T19:26:24.000Z
diff --git a/functions/strings/bin2hex.js b/functions/strings/bin2hex.js
@@ -1,10 +1,13 @@
 function bin2hex(s){
     // http://kevin.vanzonneveld.net
     // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
     // *     example 1: bin2hex('Kev');
     // *     returns 1: '4b6576'
 
-    var i, f = s.length, a = [];
+    var i, f = 0, a = [];
+    s += '';
+    f = s.length;
     for(i = 0; i<f; i++){
         a[i] = s.charCodeAt(i).toString(16);
     }
diff --git a/functions/strings/count_chars.js b/functions/strings/count_chars.js
@@ -2,11 +2,12 @@ function count_chars( str, mode ) {
     // http://kevin.vanzonneveld.net
     // +   original by: Ates Goral (http://magnetiq.com)
     // +    tweaked by: Jack
+    // +   bugfixed by: Onno Marsman
     // *     example 1: count_chars("Hello World!", 1);
     // *     returns 1: "Hd e!lWor"
 
     var histogram = new Object(), tmp_arr = new Array();
-    var key, i, code, mode, strl = 0; 
+    var key, i, code, mode, strl = 0;
     var argc = arguments.length;
 
     if (argc == 1) {
@@ -19,7 +20,9 @@ function count_chars( str, mode ) {
             histogram[i] = 0;
         }
     }
-    
+
+    str += '';
+
     strl = str.length;
     for (i = 0; i < strl; ++i) {
         code = str.charCodeAt(i);
diff --git a/functions/strings/html_entity_decode.js b/functions/strings/html_entity_decode.js
@@ -4,13 +4,14 @@ function html_entity_decode( string ) {
     // +      input by: ger
     // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
     // +    revised by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
     // %          note: table from http://www.the-art-of-web.com/html/character-codes/
     // *     example 1: html_entity_decode('Kevin &amp; van Zonneveld');
     // *     returns 1: 'Kevin & van Zonneveld'
-    
+
     var histogram = {}, histogram_r = {}, code = 0;
     var entity = chr = '';
-    
+
     histogram['34'] = 'quot';
     histogram['38'] = 'amp';
     histogram['60'] = 'lt';
@@ -111,19 +112,19 @@ function html_entity_decode( string ) {
     histogram['253'] = 'yacute';
     histogram['254'] = 'thorn';
     histogram['255'] = 'yuml';
-    
-    // Reverse table. Cause for maintainability purposes, the histogram is 
+
+    // Reverse table. Cause for maintainability purposes, the histogram is
     // identical to the one in htmlentities.
     for (code in histogram) {
         entity = histogram[code];
-        histogram_r[entity] = code; 
+        histogram_r[entity] = code;
     }
-    
-    return string.replace(/(\&([a-zA-Z]+)\;)/g, function(full, m1, m2){
+
+    return (string+'').replace(/(\&([a-zA-Z]+)\;)/g, function(full, m1, m2){
         if (m2 in histogram_r) {
             return String.fromCharCode(histogram_r[m2]);
         } else {
             return m2;
         }
-    });    
+    });
 }
diff --git a/functions/strings/htmlentities.js b/functions/strings/htmlentities.js
@@ -4,13 +4,14 @@ function htmlentities( string ){
     // +    revised by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
     // +   improved by: nobbler
     // +    tweaked by: Jack
+    // +   bugfixed by: Onno Marsman
     // %          note: table from http://www.the-art-of-web.com/html/character-codes/
     // *     example 1: htmlentities('Kevin & van Zonneveld');
     // *     returns 1: 'Kevin &amp; van Zonneveld'
-    
+
     var histogram = {}, code = 0, tmp_arr = [], i = 0;
     var stringl = 0;
-    
+
     histogram['34'] = 'quot';
     histogram['38'] = 'amp';
     histogram['60'] = 'lt';
@@ -111,7 +112,8 @@ function htmlentities( string ){
     histogram['253'] = 'yacute';
     histogram['254'] = 'thorn';
     histogram['255'] = 'yuml';
-    
+
+    string += '';
     stringl = string.length
     for (i = 0; i < stringl; ++i) {
         code = string.charCodeAt(i);
@@ -121,6 +123,6 @@ function htmlentities( string ){
             tmp_arr[i] = string.charAt(i);
         }
     }
-    
+
     return tmp_arr.join('');
 }
diff --git a/functions/strings/levenshtein.js b/functions/strings/levenshtein.js
@@ -1,10 +1,11 @@
 function levenshtein( str1, str2 ) {
     // http://kevin.vanzonneveld.net
     // +   original by: Carlos R. L. Rodrigues (http://www.jsfromhell.com)
+    // +   bugfixed by: Onno Marsman
     // *     example 1: levenshtein('Kevin van Zonneveld', 'Kevin van Sommeveld');
     // *     returns 1: 3
 
-    var s, l = (s = str1.split("")).length, t = (str2 = str2.split("")).length, i, j, m, n;
+    var s, l = (s = (str1+'').split("")).length, t = (str2 = (str2+'').split("")).length, i, j, m, n;
     if(!(l || t)) return Math.max(l, t);
     for(var a = [], i = l + 1; i; a[--i] = [i]);
     for(i = t + 1; a[0][--i] = i;);
diff --git a/functions/strings/ltrim.js b/functions/strings/ltrim.js
@@ -1,12 +1,13 @@
-function ltrim ( str, charlist ) {
-    // http://kevin.vanzonneveld.net
-    // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
-    // +      input by: Erkekjetter
-    // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
-    // *     example 1: ltrim('    Kevin van Zonneveld    ');
-    // *     returns 1: 'Kevin van Zonneveld    '
-
-    charlist = !charlist ? ' \s\xA0' : charlist.replace(/([\[\]\(\)\.\?\/\*\{\}\+\$\^\:])/g, '\$1');
-    var re = new RegExp('^[' + charlist + ']+', 'g');
-    return str.replace(re, '');
+function ltrim ( str, charlist ) {
+    // http://kevin.vanzonneveld.net
+    // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +      input by: Erkekjetter
+    // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
+    // *     example 1: ltrim('    Kevin van Zonneveld    ');
+    // *     returns 1: 'Kevin van Zonneveld    '
+
+    charlist = !charlist ? ' \s\xA0' : (charlist+'').replace(/([\[\]\(\)\.\?\/\*\{\}\+\$\^\:])/g, '\$1');
+    var re = new RegExp('^[' + charlist + ']+', 'g');
+    return (str+'').replace(re, '');
 }
diff --git a/functions/strings/ord.js b/functions/strings/ord.js
@@ -1,8 +1,9 @@
 function ord( string ) {
     // http://kevin.vanzonneveld.net
     // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
     // *     example 1: ord('K');
     // *     returns 1: 75
 
-    return string.charCodeAt(0);
+    return (string+'').charCodeAt(0);
 }
diff --git a/functions/strings/parse_str.js b/functions/strings/parse_str.js
@@ -3,6 +3,7 @@ function parse_str(str, array){
     // +   original by: Cagri Ekin
     // +   improved by: Michael White (http://getsprink.com)
     // +    tweaked by: Jack
+    // +   bugfixed by: Onno Marsman
     // *     example 1: parse_str('first=foo&second=bar');
     // *     returns 1: { first: 'foo', second: 'bar' }
     // *     example 2: parse_str('str_a=Jack+and+Jill+didn%27t+see+the+well.');
@@ -11,10 +12,10 @@ function parse_str(str, array){
     var glue1 = '=';
     var glue2 = '&';
 
-    var array2 = str.split(glue2);
+    var array2 = (str+'').split(glue2);
     var array3 = [];
     var array2l = 0, tmp = '', x = 0;
-    
+
     array2l = array2.length;
     for (x = 0; x<array2l; x++) {
         tmp = array2[x].split(glue1);
diff --git a/functions/strings/rtrim.js b/functions/strings/rtrim.js
@@ -1,12 +1,13 @@
-function rtrim ( str, charlist ) {
-    // http://kevin.vanzonneveld.net
-    // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
-    // +      input by: Erkekjetter
-    // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
-    // *     example 1: rtrim('    Kevin van Zonneveld    ');
-    // *     returns 1: '    Kevin van Zonneveld'
-
-    charlist = !charlist ? ' \s\xA0' : charlist.replace(/([\[\]\(\)\.\?\/\*\{\}\+\$\^\:])/g, '\$1');
-    var re = new RegExp('[' + charlist + ']+$', 'g');
-    return str.replace(re, '');
+function rtrim ( str, charlist ) {
+    // http://kevin.vanzonneveld.net
+    // +   original by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +      input by: Erkekjetter
+    // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
+    // *     example 1: rtrim('    Kevin van Zonneveld    ');
+    // *     returns 1: '    Kevin van Zonneveld'
+
+    charlist = !charlist ? ' \s\xA0' : (charlist+'').replace(/([\[\]\(\)\.\?\/\*\{\}\+\$\^\:])/g, '\$1');
+    var re = new RegExp('[' + charlist + ']+$', 'g');
+    return (str+'').replace(re, '');
 }
diff --git a/functions/strings/soundex.js b/functions/strings/soundex.js
@@ -3,6 +3,7 @@ function soundex(str) {
     // +   original by: Jonas Raoni Soares Silva (http://www.jsfromhell.com)
     // +    tweaked by: Jack
     // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
     // *     example 1: soundex('Kevin');
     // *     returns 1: 'K150'
     // *     example 2: soundex('Ellery');
@@ -12,9 +13,9 @@ function soundex(str) {
 
     var i, j, l, r, p = isNaN(p) ? 4 : p > 10 ? 10 : p < 4 ? 4 : p;
     var m = {BFPV: 1, CGJKQSXZ: 2, DT: 3, L: 4, MN: 5, R: 6};
-    var r = (s = str.toUpperCase().replace(/[^A-Z]/g, "").split("")).splice(0, 1);
+    var r = (s = (str+'').toUpperCase().replace(/[^A-Z]/g, "").split("")).splice(0, 1);
     var sl = 0;
-    
+
     sl = s.length;
     for (i = -1, l = sl; ++i < l;) {
         for (j in m) {
@@ -23,6 +24,6 @@ function soundex(str) {
             }
         }
     }
-    
+
     return r.length > p && (r.length = p), r.join("") + (new Array(p - r.length + 1)).join("0");
 }
diff --git a/functions/strings/str_ireplace.js b/functions/strings/str_ireplace.js
@@ -5,17 +5,19 @@ function str_ireplace ( search, replace, subject ) {
     // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
     // +    tweaked by: Jack
     // +   bugfixed by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
+    // +   bugfixed by: Onno Marsman
     // *     example 1: str_ireplace('l', 'l', 'HeLLo');
     // *     returns 1: 'Hello'
-    
+
     var i, k = '';
     var searchl = 0;
-    
+
+    search += '';
     searchl = search.length;
     if (!(replace instanceof Array)) {
         replace = new Array(replace);
         if (search instanceof Array) {
-            // If search is an array and replace is a string, 
+            // If search is an array and replace is a string,
             // then this replacement string is used for every value of search
             while (searchl > replace.length) {
                 replace[replace.length] = replace[0];
@@ -27,13 +29,13 @@ function str_ireplace ( search, replace, subject ) {
         search = new Array(search);
     }
     while (search.length>replace.length) {
-        // If replace has fewer values than search, 
+        // If replace has fewer values than search,
         // then an empty string is used for the rest of replacement values
         replace[replace.length] = '';
     }
 
     if (subject instanceof Array) {
-        // If subject is an array, then the search and replace is performed 
+        // If subject is an array, then the search and replace is performed
         // with every entry of subject , and the return value is an array as well.
         for (k in subject) {
             subject[k] = str_ireplace(search, replace, subject[k]);
diff --git a/functions/strings/str_rot13.js b/functions/strings/str_rot13.js
@@ -1,13 +1,14 @@
-function str_rot13( str ) {
-    // http://kevin.vanzonneveld.net
-    // +   original by: Jonas Raoni Soares Silva (http://www.jsfromhell.com)
-    // +   improved by: Ates Goral (http://magnetiq.com)
-    // *     example 1: str_rot13('Kevin van Zonneveld');
-    // *     returns 1: 'Xriva ina Mbaariryq'
-    // *     example 2: str_rot13('Xriva ina Mbaariryq');
-    // *     returns 2: 'Kevin van Zonneveld'
-
-    return str.replace(/[A-Za-z]/g, function (c) {
-        return String.fromCharCode((((c = c.charCodeAt(0)) & 223) - 52) % 26 + (c & 32) + 65);
-    });
+function str_rot13( str ) {
+    // http://kevin.vanzonneveld.net
+    // +   original by: Jonas Raoni Soares Silva (http://www.jsfromhell.com)
+    // +   improved by: Ates Goral (http://magnetiq.com)
+    // +   bugfixed by: Onno Marsman
+    // *     example 1: str_rot13('Kevin van Zonneveld');
+    // *     returns 1: 'Xriva ina Mbaariryq'
+    // *     example 2: str_rot13('Xriva ina Mbaariryq');
+    // *     returns 2: 'Kevin van Zonneveld'
+
+    return (str+'').replace(/[A-Za-z]/g, function (c) {
+        return String.fromCharCode((((c = c.charCodeAt(0)) & 223) - 52) % 26 + (c & 32) + 65);
+    });
 }
diff --git a/functions/strings/str_split.js b/functions/strings/str_split.js
@@ -2,9 +2,12 @@ function str_split ( f_string, f_split_length){
     // http://kevin.vanzonneveld.net
     // +     original by: Martijn Wieringa
     // +     improved by: Brett Zamir
+    // +     bugfixed by: Onno Marsman
     // *         example 1: str_split('Hello Friend', 3);
     // *         returns 1: ['Hel', 'lo ', 'Fri', 'end']
- 
+
+    f_string += '';
+
     if (f_split_length == undefined) {
         f_split_length = 1;
     }
diff --git a/functions/strings/strcasecmp.js b/functions/strings/strcasecmp.js
@@ -1,11 +1,12 @@
 function strcasecmp (f_string1, f_string2){
     // http://kevin.vanzonneveld.net
     // +     original by: Martijn Wieringa
+    // +     bugfixed by: Onno Marsman
     // *         example 1: strcasecmp('Hello', 'hello');
     // *         returns 1: 0
 
-    var string1 = f_string1.toLowerCase();
-    var string2 = f_string2.toLowerCase();
+    var string1 = (f_string1+'').toLowerCase();
+    var string2 = (f_string2+'').toLowerCase();
 
     if(string1 > string2) {
       return 1;
diff --git a/functions/xml/utf8_decode.js b/functions/xml/utf8_decode.js
@@ -4,16 +4,19 @@ function utf8_decode ( str_data ) {
     // +      input by: Aman Gupta
     // +   improved by: Kevin van Zonneveld (http://kevin.vanzonneveld.net)
     // +   improved by: Norman "zEh" Fuchs
-    // +   bugfixed by: hitwork  
+    // +   bugfixed by: hitwork
+    // +   bugfixed by: Onno Marsman
     // *     example 1: utf8_decode('Kevin van Zonneveld');
     // *     returns 1: 'Kevin van Zonneveld'
 
     var tmp_arr = [], i = ac = c1 = c2 = c3 = 0;
 
+    str_data += '';
+
     while ( i < str_data.length ) {
         c1 = str_data.charCodeAt(i);
         if (c1 < 128) {
-            tmp_arr[ac++] = String.fromCharCode(c1); 
+            tmp_arr[ac++] = String.fromCharCode(c1);
             i++;
         } else if ((c1 > 191) && (c1 < 224)) {
             c2 = str_data.charCodeAt(i+1);
@@ -26,6 +29,6 @@ function utf8_decode ( str_data ) {
             i += 3;
         }
     }
-    
+
     return tmp_arr.join('');
 }
diff --git a/functions/xml/utf8_encode.js b/functions/xml/utf8_encode.js
