First, I have to say the Comment Search module is great stuff! I was expecting to write it myself, and here it was with a nice bow tied around it. Thanks Robert!

I'm finding my comment search results contain html tags, which I'm guessing is because they're escaped. Is this standard behavior, and if not, any idea on how to work around it?

CommentFileSizeAuthor
#2 solr_comments_escaped_tags.patch1.88 KBkcoop

Comments

robertdouglass’s picture

robertdouglass commented

Hmm. Perhaps you can compare the steps taken at index time and see if I do all the right things to the comment text as compared to the node text from the apachesolr_node_to_document function in apachesolr.index.inc. Thanks for looking into this.

kcoop’s picture

kcoop commented
StatusFileSize
new solr_comments_escaped_tags.patch1.88 KB

Yes, it was a simple fix. Duplication of the htmlspecialchars and html_entity_decode calls (implicit in apachesolr_clean_text). Here's a patch.

dakku’s picture

dakku commented
Status: Active » Reviewed & tested by the community

again, the above patch by kcoop seems to fix the issue. I am running this on my website here:
http://janaksingh.com/search/apachesolr_search/drupal?filters=type%3Acom...

Can this be committed to 6.2 please?

robertdouglass’s picture

robertdouglass commented
Status: Reviewed & tested by the community » Fixed

Applied, minus the code that limits the batch process to 1 ;)

Thanks.

kcoop’s picture

kcoop commented

Ack, I did it here too... whoops. :P

Status: Fixed » Closed (fixed)

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