Drop-down list field searches don't work
Adelle - January 16, 2007 - 18:08
| Project: | Advanced User |
| Version: | 5.x-2.x-dev |
| Component: | User interface |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
Cannot search fields that are of the drop-down list type. Advuser will only list some of the options within the list and not all of them.
#1
I have the same problem.
I have a list :
- public laboratory
- private laboratory
- distributor
The module only show 'distributor'
#2
#3
Have attached patch which fixes profile field searches for checkboxes, selection drop downs.
Also textareas (multiline text) caused problems by showing a text area for entering the search term which obliterates any divs beneath it. This is also fixed.
#4
I'm not sure how this helps? I'm trying to correct coding standards errors as well. Please separate the string concantenation . (period) with white space on both sides. Also wrap the comments at the 78 column.
Are your profile field names created as separate tables? That is what this patch appears to be. I have no problem selecting on a multi text column.
#5
I think this is possibly a response to my other patch, which does indeed address a different problem. I uploaded two at the same time, which may have caused the confusion.
The only concantenation in here without spaces around the periods is in the original source, which is why I assumed it was acceptable style and used it in my other patch at http://drupal.org/node/255777
#6
Uhm, I'm not sure what I was looking at when I made my comment at #4. Mitch I reworked your patch into a switch statement. Does this look reasonable?
#7
Yes looks tidier. Need to decide what to do with textarea case. In my application I just put up a normal textfield, because people are generally entering a snippet which they are looking for in the longer multiline text. I think this a sensible default- who wants to type in multiline text to do an exact match? What do other people think?
That just leaves freeform list, URL and date types, which I think are all doable but I dont have time right now.
I think the patch you were looking at is at the link above. This fixes a problem with the SQL when you search on more than profile field by refining a search. It also simplifies the SQL by removing an unnecssary clause.
#8
Thanks, I'll plan to commit the patch today. I've already got a fix for the display of textarea and date fields. I know textarea selection, even multiline is working. The issue I have currently is the display of the grippie graphic below the text box for the resizing, it is displaying a single vertical bar instead.
#9
Patch committed to CVS.
#10
Automatically closed -- issue fixed for two weeks with no activity.