Until #1821178: Performance tune text_field_load() gets put into core, I'd suggest we do something locally. I noticed something strange with really short entries in a node's 'value'. So short that there is no 'summary'. It still passed the text value of the body to htmlpurifer. This is unnecessary, since the text is an empty string. I'd like to suggest we exit _htmlpurifier_process_text early.

#1 htmlpurifier-filter_empty_text-1821186.patch557 bytesheddn
Members fund testing for the Drupal project. Drupal Association Learn more


heddn’s picture

Status: Active » Fixed
557 bytes
ezyang’s picture

You should do an explicit comparison against the empty string.

heddn’s picture

Done and committed

Status: Fixed » Closed (fixed)

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