I need to integrate Facetapi with Realname , someone else needs it? I'll post a patch ASAP.

CommentFileSizeAuthor
#2 facetapi-realname-integration-2085061-02.patch1.77 KBsoulfroys
#1 facetapi-realname-integration-2085061-01.patch1.74 KBsoulfroys
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

soulfroys’s picture

soulfroys
Primary language Portuguese, Brazil
CreditAttribution: soulfroys commented
Status: Active » Needs review
FileSize
facetapi-realname-integration-2085061-01.patch1.74 KB

Here's the patch. I would appreciate some feedback.

Thanks!

soulfroys’s picture

soulfroys
Primary language Portuguese, Brazil
CreditAttribution: soulfroys commented
FileSize
facetapi-realname-integration-2085061-02.patch1.77 KB

Security: I forgot to filter the output:

-        $realname = $record['realname'];
+        $realname = apachesolr_clean_text($record['realname']);
voidberg’s picture

voidberg CreditAttribution: voidberg commented
Assigned: Unassigned » voidberg
eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented
Issue summary: View changes

I see that you use function apachesolr_clean_text without checking that this module exists. And also facetapi can work with searchapi, but seems it's not supported.

eugene.ilyin’s picture

eugene.ilyin CreditAttribution: eugene.ilyin as a volunteer and at DrupalJedi commented
Status: Needs review » Needs work