Instead of making 10+ calls to str_replace(), use a single call + array.

This commit is contained in:
Colin Smith 2005-07-13 20:42:04 +00:00
parent e315cfa19a
commit ddce8ef2e4

View File

@ -424,39 +424,11 @@ function arp_get_mac_by_ip($ip) {
/* return a fieldname that is safe for xml usage */
function xml_safe_fieldname($fieldname) {
$fieldname = str_replace("/","",$fieldname);
$fieldname = str_replace("-","",$fieldname);
$fieldname = str_replace(" ","",$fieldname);
$fieldname = str_replace("!","",$fieldname);
$fieldname = str_replace("@","",$fieldname);
$fieldname = str_replace("#","",$fieldname);
$fieldname = str_replace("$","",$fieldname);
$fieldname = str_replace("%","",$fieldname);
$fieldname = str_replace("^","",$fieldname);
$fieldname = str_replace("&","",$fieldname);
$fieldname = str_replace("*","",$fieldname);
$fieldname = str_replace("(","",$fieldname);
$fieldname = str_replace(")","",$fieldname);
$fieldname = str_replace("_","",$fieldname);
$fieldname = str_replace("+","",$fieldname);
$fieldname = str_replace("=","",$fieldname);
$fieldname = str_replace("{","",$fieldname);
$fieldname = str_replace("}","",$fieldname);
$fieldname = str_replace("[","",$fieldname);
$fieldname = str_replace("]","",$fieldname);
$fieldname = str_replace("|","",$fieldname);
$fieldname = str_replace("\\","",$fieldname);
$fieldname = str_replace("/","",$fieldname);
$fieldname = str_replace("<","",$fieldname);
$fieldname = str_replace(">","",$fieldname);
$fieldname = str_replace("?","",$fieldname);
$fieldname = str_replace(":","",$fieldname);
$fieldname = str_replace(",","",$fieldname);
$fieldname = str_replace(".","",$fieldname);
$fieldname = str_replace("'","",$fieldname);
$fieldname = str_replace("\"","",$fieldname);
$fieldname = strtolower($fieldname);
return $fieldname;
$replace = array('/', '-', ' ', '!', '@', '#', '$', '%', '^', '&', '*', '(', ')',
'_', '+', '=', '{', '}', '[', ']', '|', '\', '/', '<', '>', '?',
':', ',', '.', '\'', '\\'
);
return strtolower(str_replace($replace, "", $fieldname));
}
?>