jonaswouters / XhprofBundle

XHProf bundle for Symfony 2
210 stars 49 forks source link

Do not enable profiler if xhprof is not installed on server + add extension required on composer #20

Closed stephpy closed 12 years ago

stephpy commented 12 years ago

Hi,

I added ext-xhprof on composer.json.

And i added a verification of php xhprof extension on DependencyInjection/Extension. Because we could have installed this bundle in an environment which has xhprof extension and then deploy it on an other environment. Calling composer install will be OK (coz .lock exists) and we'll have xhprof bundle. But if we dont' have xhprof extension, we'll have error due to call of xhprof functions.

jonaswouters commented 12 years ago

Sounds legit! Thanks