If you search on one profile field and then refine the search using another profile field an SQL error results in no matches.
Combining a user field and a profile field works OK. I have attached a patch for this.

CommentFileSizeAuthor
#7 advuser_filters.inc_refine.patch1.34 KBAnonymous (not verified)
#6 advuser_filters.inc_refine.patch1.23 KBAnonymous (not verified)
#1 advuser_filters.inc_refine.patch1005 bytesmitchh

Comments

mitchh’s picture

mitchh commented
StatusFileSize
new advuser_filters.inc_refine.patch1005 bytes

patch attached

mitchh’s picture

mitchh commented

This is still broken in the latest release (2008-May-16). Its a simple fix which also simplifies the SQL - why not put it in?

Anonymous’s picture

Anonymous (not verified) commented

I just haven't had the time yet. Beginning today I'm out of office and out of development cycles until June. This will be the first thing I look at when I get back.

mitchh’s picture

mitchh commented

OK, sorry.

Anonymous’s picture

Anonymous (not verified) commented
Status: Active » Needs work

Marking this a CNW because we can get rid of all the concatenation.

E.G.:

'join' => "LEFT JOIN {profile_values} ".$pvname." ON ".$pvname.".fid = " . $field->fid,

can be changed to

'join' => "LEFT JOIN {profile_values} $pvname ON $pvname.fid = $field->fid",

IMO, the changed code is easier to read.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs work » Needs review
StatusFileSize
new advuser_filters.inc_refine.patch1.23 KB

I've reworked the patch. Please review and comment.

Anonymous’s picture

Anonymous (not verified) commented
StatusFileSize
new advuser_filters.inc_refine.patch1.34 KB

Adjusted patch to meet a requirement in #262461: Patch to allow autocomplete by profile field in advuser filters.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Fixed

Patch has been committed.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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