Make sense to me. Since this hack affects so many files, you might want to run it by Rick to see if he anticipates making any changes that could impact it. (or maybe even convince him to incorporate it as a feature
).
Another thought:
error_log("$time,$IP,$Operation,$Username,$Number,$Status,$Description\n",The Username and Description fields could potentially contain commas, so it might be a good idea to escape any commas that are present in those fields. The function addcslashes() looks like a convenient way of doing that, since it has a companion function stripcslashes(). I think it would be used like this:
$Username_escaped = addcslashes($Username, ',');
$Description_escaped = addcslashes($Description, ',');
