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.
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) kprint_r() and kdevel_print_object() was marked as deprecated. We should remove the usage of these functions from devel codebase
Comment | File | Size | Author |
---|---|---|---|
#9 | remove_usage_of-2703343-9.patch | 6.92 KB | willzyx |
| |||
#5 | remove_usage_of-2703343-5.patch | 1.5 KB | willzyx |
| |||
#3 | remove_usage_of-2703343-3.patch | 7.95 KB | lussoluca |
|
Comments
Comment #2
willzyx CreditAttribution: willzyx commentedComment #3
lussolucaComment #4
willzyx CreditAttribution: willzyx commentedLuca good work :)
Can we inject
devel.dumper
in DevelController and use::exportAsRenderable()
(where it makes sense) instead of use kpr()? I think it's much cleaner and makes the code more concisely and legiblyComment #5
willzyx CreditAttribution: willzyx commentedA firts step: completely remove kdevel_print_object and its usage
Comment #7
moshe weitzman CreditAttribution: moshe weitzman commentedComment #8
willzyx CreditAttribution: willzyx commentedthe patch that removes
kprint_r()
should take in account #4 and need a reroll :)Comment #9
willzyx CreditAttribution: willzyx commentedComment #10
moshe weitzman CreditAttribution: moshe weitzman commentedComment #12
willzyx CreditAttribution: willzyx commentedCommitted and pushed to 8.x