Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Krumo calls substr() which breaks UTF-8 data. The patch replaces it to drupal_substr().
Comment | File | Size | Author |
---|---|---|---|
devel-krumo_substr.patch | 510 bytes | maximpodorov | |
Comments
Comment #1
moshe weitzman CreditAttribution: moshe weitzman commentedIs unicode.inc always loaded when this code is called? Also, we're not really doing edits to krumo class. we need a replacement.
Comment #2
pcambraHere's the main discussion for the krumo stuff #1853112: Replacement for Krumo?
Comment #3
maximpodorov CreditAttribution: maximpodorov commentedI understand that the real solution is getting rid of Krumo, but the current version of Devel just makes my site unusable for admins, since Krumo tries to cut large UTF-8 texts, and this produces such warnings:
Invalid UTF-8 sequence in argument in json_encode().
Comment #4
DamienMcKennaIf Drupal provides a replacement for a standard function, it should be used instead. IMHO it's worth using a quick bandaid, especially when there's no end in sight for the other issue. RTBC.
Comment #6
salvisBeating a dead horse here, but I agree with DamienMcKenna.
Thanks, maximpodorov and DamienMcKenna!