Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#2 | move_webprofiler_to_devel-2609242-2.patch | 439.76 KB | lussoluca |
|
Comments
Comment #2
lussolucaFirst 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.
Comment #3
moshe weitzman CreditAttribution: moshe weitzman at Acquia commentedThanks 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.
Comment #4
moshe weitzman CreditAttribution: moshe weitzman at Acquia commentedI'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.
Comment #5
lussolucaYes I agree with you. If we don't find any other available libraries we can just drop this feature.
Comment #6
moshe weitzman CreditAttribution: moshe weitzman at Acquia commentedCommitted! 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.
Comment #8
willzyx CreditAttribution: willzyx commentedIt 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!