Problem/Motivation

Webprofiler and Devel overlaps on many points, so let's merge them together.

Proposed resolution

Include Webprofiler as Devel sub-module

Remaining tasks

  • Review patch
  • Add documentation
  • ...

User interface changes

in #2572009: Remove query logger and footer info we remove the query logger from Devel, here we add the Webprofiler toolbar and dashboard.

API changes

None.

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lussoluca created an issue. See original summary.

lussoluca’s picture

Status: Active » Needs review
FileSize
439.76 KB

First version just to see if all works correctly. I've left "webprofiler" as module name and I've moved the configuration settings of webprofiler as a sub-tab of devel configuration.

moshe weitzman’s picture

Thanks for doing this. Once this is green, I will commit this issue and #2572009: Remove query logger and footer info. Then we should update the devel project page and edit webprofiler project page to point here.

moshe weitzman’s picture

I'd love to get rid of the highlight.js task of the README in favor of some lines in our composer.json as per https://www.drupal.org/node/1398772.

lussoluca’s picture

Yes I agree with you. If we don't find any other available libraries we can just drop this feature.

moshe weitzman’s picture

Status: Needs review » Fixed

Committed! Thanks to @lussoluca for a great module, and to Wellnet for sponsoring many hours of work. This is going to make many drupal devels happy.

willzyx’s picture

It is really great to see webprofiler being part of Devel, I'm an happy developer :)
Thanks Luca for the hard work you've done on webprofiler!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.