logger callback

[lalop]

ref #126

[girtsu]

Hi,
I just started to use Idiorm/Paris, and I really like its lightweightness if compared to other frameworks.
However I noticed a few inconveniences with the logging system. As this thread is currently discussing the logging, I will just add my thoughts here:

1. There is no possibility to reset the query log - for long running scripts it will consume a lot of memory at some point. Maybe there could be a method that clears log, just like there already is an option to clear the query cache.
   Another option would be to add an optional argument to the get_query_log method that allows to reset the log after each access.
2. Having a callback function for logging would be a really useful feature. Specifying callback could also turn off the internal logging, to avoid high memory usage.
3. It would be nice to have a config option that enables to log statements without replacing parameters with values. I see that the _log_query method contains a complicated code that replaces question marks with values. However, for some people it might be enough (or even more suitable for their needs) to log queries without values. It would also improve logging performance.

[lalop]

In my point, a good thing could to create a kind of new idiorm version to integrate some breaking change.

Like that we could remove the current logging system to a callback system, letting user keep an array of query if they want. Then we could pass to this callback a kind of query log if the prepared query, params, maybe the execution time ....

That 's not a big work but it's a big change and I don't know if a new version is expected.

[Surt]

+1

[badpenguin]

i wish to have a log callback providing a pointer to a class that implements an certain interface (i.e. like RedBean does)

[treffynnon]

There won't be a certain interface to implement just a callback that you
can use to pass the message whereever you choose.

[treffynnon]

Thank you for the pull request. All it needs are a couple of tests and some documentation. Then it can be merged!

[treffynnon]

Merge in commit 32b2220
Documented and tested in commit c842904