Hi,
I think the search module needs to use format_plural() instead of t(), patch attached.
Regards,
Tobias
| Comment | File | Size | Author |
|---|---|---|---|
| search_plural.patch | 948 bytes | toddy |
Hi,
I think the search module needs to use format_plural() instead of t(), patch attached.
Regards,
Tobias
| Comment | File | Size | Author |
|---|---|---|---|
| search_plural.patch | 948 bytes | toddy |
Comments
Comment #1
puradata commentedSearch_plural bug patch works great. I have some questions
1. The patch fixes a case that never happens because the minimum number of
characters for a search item is 3. Why is this 3 google allows 1?
2. Currently variable_set(‘'minimum_word_size') is never called causing
variable_get('minimum_word_size') to always to return the second argument.
3. Shouldn’t there be a define for default MINIMUM_WORD_SIZE instead of using the
number 3.
Comment #2
stevenpatzComment #3
webchickCommitted to HEAD. Thanks! puradata, maybe start a separate issue to discuss this variable further and whether or not it makes sense the way it is currently.