theme_apachesolr_mlt_recommendation_block themes a list of links. The links run through l() which escapes html elements and html entities. Since the results passed to theme_apachesolr_mlt_recommendation_block are already have there html entities encoded, doing this ends up with entities showing up on the page. I think its best to reverse the html encoding before passing it onto check_plain.

CommentFileSizeAuthor
#4 mlt_double.patch1.46 KBrobertdouglass
fix-apachesolr-recommendations.patch717 bytesjosh waihi

Comments

janusman’s picture

janusman commented
Status: Active » Needs review

I think this got lost because of the status. Changing status to needs review.

Scott Reynolds’s picture

Scott Reynolds commented
Status: Needs review » Needs work

instead of decode then encoding, why don't you just tell l() html = true and then l won't check_plain().

Scott Reynolds’s picture

Scott Reynolds commented

This is the change..

$links[] = l($title, $result->path, array('html' => TRUE));
robertdouglass’s picture

robertdouglass commented
Status: Needs work » Fixed
StatusFileSize
new mlt_double.patch1.46 KB

Committed to both DRUPAL-6--1 and DRUPAL-6--2. Thanks.

Status: Fixed » Closed (fixed)

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