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.

CommentFileSizeAuthor
#1 htmlpurifier-filter_empty_text-1821186.patch557 bytesheddn
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn commented
Status: Active » Fixed
FileSize
htmlpurifier-filter_empty_text-1821186.patch557 bytes
ezyang’s picture

ezyang CreditAttribution: ezyang commented

You should do an explicit comparison against the empty string.

heddn’s picture

heddn
Primary language English
Location Nicaragua
CreditAttribution: heddn commented

Done and committed

Status: Fixed » Closed (fixed)

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