check_plain() issue fixed in 6.x not in 5.x

stplanken - July 5, 2009 - 15:16
Project:Page Title
Version:5.x-2.x-dev
Component:Code
Category:bug report
Priority:minor
Assigned:Unassigned
Status:duplicate
Description

See: http://drupal.org/node/456256 "" in title"

I have this problem on my D5.x site as well: page titles incorrectly appear with the HTML for quotation marks etc.

Changing line 430 from return check_plain(strip_tags($title)); to return strip_tags($title); solved it for me. (Not sure if I should have modified the status of the original report or not -- apologies and thanks for a great module.)

#1

ron_s - July 9, 2009 - 14:57

I can confirm this fix worked for me too. Thanks for sharing.

#2

kiamlaluno - July 9, 2009 - 18:32

To pass the title string to check_plain() is something that also Drupal core code does; see aggregator_page_source(), book_admin_edit(), contact_user_page().

#3

chriso - November 11, 2009 - 02:24
Status:active» duplicate

I'm marking this as a duplicate of #124169: HTML entities double-escaped in titles -- as there is active discussion over there about it.

FWIW the fix above works for me too.

 
 

Drupal is a registered trademark of Dries Buytaert.