The drupal 6 version does not hash the id attribute in a facet block. Now it generates invalid html id attributes with an @ sign. This prevents the "more" functionality from working.

CommentFileSizeAuthor
#3 facetapi-fix-for-delta-hash-1892192-3.patch574 bytesvoidberg
#1 hash-deltas.patch714 bytesvoidberg

Comments

voidberg’s picture

voidberg commented
Status: Active » Needs review
StatusFileSize
new hash-deltas.patch714 bytes

The attached patch fixes this issue.

Status: Needs review » Needs work

The last submitted patch, hash-deltas.patch, failed testing.

voidberg’s picture

voidberg commented
StatusFileSize
new facetapi-fix-for-delta-hash-1892192-3.patch574 bytes

Proper patch.

voidberg’s picture

voidberg commented
Status: Needs work » Needs review
cpliakas’s picture

cpliakas commented
Status: Needs review » Reviewed & tested by the community

Looks good to me. Thanks for the patch!

voidberg’s picture

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

Commited to 6.3 version of Facet API.

http://drupalcode.org/project/facetapi.git/commit/11a118d

cpliakas’s picture

cpliakas commented

Excellent. Thanks!

Status: Fixed » Closed (fixed)

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