yiisoft/yii — pull request #2198 by acorncom (2013-03-12 23:34:06+00:00)
Title: Changed CFilter to use CVarDumper to dump vars

Adjusted to make sure of `CVarDumper`'s auto-catching of Yii object recursion.

One thing it's probably worth discussing. The previous setup (using `var_export`) would dump objects in more detail than the default `CVarDumper` setup.

I wonder about adding another `$dumpLevel` to CFilter and setting it to default to 100. `var_export` errors out if it runs into a recursive structure. Do we want to emulate that same behavior to keep from breaking BC?

I'm inclined to think this new behavior is better than the old, but it will differ slightly. But because it's for logging, it may not be a giant problem. Thoughts?