The level of information sent to the watchdog logs is just way to excessive. Every call to xc_log_info() writes something to watchdog, a large proportion of which are xc_search_form_alter: foo_form_id. In addition, whenever cron runs these messages are also sent in an email to the site admin! I left this module enabled overnight and it ended up spamming a mailing list every time cron ran! These messages might be useful for debugging but on a production site they're just noise and cause the log files to fill up too quickly and drown out other, more important, log messages.
xc_log_info() is called too many times throughout the code for me to remove it and I imagine it is useful when debugging. So I've patched the module for my site so that the function only logs messages if $verbose is set to TRUE. Currently $verbose value defaults to FALSE. It might be useful to make this into a system variable and expose it as a checkbox in the config somewhere, but I wasn't sure where to put that, so I'll leave it up to you.
| Comment | File | Size | Author |
|---|---|---|---|
| xc_log_info_verbose.patch | 600 bytes | stella |