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.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | advuser-5.x-2.x-dev.mitchh.patch | 2.33 KB | Anonymous (not verified) |
| #3 | advuser_filters.inc_types.patch | 1.92 KB | mitchh |
Comments
Comment #1
zmove commentedI have the same problem.
I have a list :
- public laboratory
- private laboratory
- distributor
The module only show 'distributor'
Comment #2
sign commentedComment #3
mitchh commentedHave 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.
Comment #4
Anonymous (not verified) commentedI'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.
Comment #5
mitchh commentedI 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
Comment #6
Anonymous (not verified) commentedUhm, 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?
Comment #7
mitchh commentedYes 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.
Comment #8
Anonymous (not verified) commentedThanks, 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.
Comment #9
Anonymous (not verified) commentedPatch committed to CVS.
Comment #10
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.