Hi,

I think the search module needs to use format_plural() instead of t(), patch attached.

Regards,
Tobias

CommentFileSizeAuthor
search_plural.patch948 bytestoddy

Comments

puradata’s picture

Search_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.

stevenpatz’s picture

Status: Needs review » Reviewed & tested by the community
webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed 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.

Status: Fixed » Closed (fixed)
Issue tags: -Quick fix

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