Follow-up to #2191395: Make dpm() and friends pluggable (with Krumo, Kint, Ladybug etc)
in #2191395: Make dpm() and friends pluggable (with Krumo, Kint, Ladybug etc) was created a pluggable dumper system that allow contrib to extend the devel functionality and provide their own plugins for formatting and displaying the variables debugged through devel functions.
The following implementations of dumper have been created and are currently provided by devel:
- CromePhp
- FirePhp
- DoctrineDebug
- DrupalVariable
- Kint (kint module)
you was also talk of adding implementations for Ladybug and VarDumper.
IMHO devel should provide only the plugins management, plugins that works with libraries which are present in core (DoctrineDebug, DrupalVariable) and a default implementation, all other implementation should live in contrib modules (that means remove the CromePhp and FirePhp implementations).
We should decide before ship a stable release which dumper plugins ship with devel
Comment | File | Size | Author |
---|---|---|---|
#14 | devel-2703347-14.patch | 5.8 KB | moshe weitzman |
| |||
#10 | devel-2703347-9.patch | 5.51 KB | moshe weitzman |
#8 | devel-2703347-8.patch | 5.72 KB | moshe weitzman |
Comments
Comment #2
willzyx CreditAttribution: willzyx commentedComment #3
willzyx CreditAttribution: willzyx commentedComment #4
willzyx CreditAttribution: willzyx commentedComment #5
youx CreditAttribution: youx commentedHi willzyx,
I am glad you opened a discussion on that topic ;)
Coming from Symfony world, I would give a big +1 for the VarDumper component.
I understand that the library is out of the needs of devel, but it's usability makes it a must-have out of the box.
There is also the question of using the VarDumper with webprofiler module, there has been a discussion opened by lussoluca on that.
IMHO webprofiler is still missing some features like debug() - VarDumper based - profiling and exception stack trace to be even more useful.
By the way, could you tell me how to cleanly require an external library like VarDumper from a module if not in core ?
Comment #6
lussolucayoux ,
for vardumper you could try the https://www.drupal.org/project/vardumper module, it has a GIT branch level-integration that provides a plugin for Devel
Comment #7
youx CreditAttribution: youx commentedHi Lussoluca,
Thanks for the advice, I will give it a try.
Is there any status or started work for collecting exceptions stack trace in the profiler ?
Comment #8
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedAttached patch removes FirePHP, ChromePHP, and Drupal Variable (too simplistic).
Should the default by var-dumper, given that its dependency is already in our composer.json? Or maybe Kint becomes default if we move its dependency from suggest to require? Both of these are more capable that Doctrine.
Comment #10
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedFixing test
Comment #11
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedComment #12
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedComment #14
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedComment #15
lussolucaI'd love to see var-dumper as the default dumper in Devel
Comment #17
moshe weitzman CreditAttribution: moshe weitzman as a volunteer commentedOK, I'v merged this. Lets do the change of default in a future issue.