If I take the $user from the global scope I can get all the info I need without passing the user variables to the function. This makes the call to the function look a lot cleaner since it only includes what type it is and any extra information that should be saved with it. I think I'm going to go with this, but what do you others think of doing it this way?
Example:
code:
// ---------------
// Log this action
admin_log("APPROVEPOSTS", "Approved: $totalapproved; Threads: $threadtotal");
