search

lekoala / silverstripe-debugbar

SilverStripe DebugBar module
MIT License
56 stars 27 forks source link

Bugfix: Only store DB queries in memory when enabled #122

Closed adrhumphreys closed 3 years ago

adrhumphreys commented 3 years ago

We shouldn't record and save in memory DB requests if the DebugBar is disabled otherwise long running tasks such as unit tests will slowly build up memory trying to store them.

Fixes https://github.com/lekoala/silverstripe-debugbar/issues/105

lekoala commented 3 years ago

Good catch. I think it's a safe change. I probably need to check why the CI build fails but i don't see how your change would cause that.