The If statement is:

if (preg_match('@[\[\{](\S+) TO (\S+)[\]\}]@', $range, $match)) {

It should be:

if (preg_match('@[\[\{](\S+) TO (\S+)[\]\}]@', $range['#value'], $match)) {
CommentFileSizeAuthor
#3 5.x-date-range-theme.patch1.12 KBdixon_
#2 date-range-theme.patch1.48 KBdixon_

Comments

nicholasthompson’s picture

nicholasthompson
Primary language English
commented

This also applies in DRUPAL-6--2

dixon_’s picture

dixon_
he/him
Primary language English
Location France
commented
Version: 5.x-2.x-dev » 6.x-2.x-dev
StatusFileSize
new date-range-theme.patch1.48 KB

Hi. Here is a patch that fixes this issue for the 6.x branch. I also noticed some incorrect usage of the function theme_apachesolr_breadcrumb_date_range(). It wasn't registered in hook_theme() and wasn't properly called from other functions.

I have successfully tested this patch, and I can confirm that this solves the problem.

/dixon_, NodeOne

dixon_’s picture

dixon_
he/him
Primary language English
Location France
commented
StatusFileSize
new 5.x-date-range-theme.patch1.12 KB

Here is also a patch for the 5.x branch. I haven't tested this patch as I'm not into Drupal 5 that much. I also don't know how theme functions are registered (if they are at all). So, someone will have to test this patch on a Drupal 5 site to see if it solves the problem.

/dixon_, NodeOne

robertdouglass’s picture

robertdouglass commented

Here's a related issue to the theme function usage which should fix that problem: http://drupal.org/node/551582#comment-2423632

robertdouglass’s picture

robertdouglass commented
Status: Needs review » Closed (duplicate)

I think #551582: Show value instead of key in CCK facets fixes this problem in 6.2. Porting it to 5.2. would fix it there, so closing this one as duplicate.