pat/thinking-sphinx — pull request #461 by nicholasklick (2013-02-09 04:10:32+00:00)
Title: Modified ThinkingSphinx::Search::Query.to_s to handle null conditions and empty conditions

What are your thoughts on this?

I modified to_s to ignore conditions that are either nil or empty.

def to_s
(star_keyword(keywords) + ' ' + conditions.keys.collect { |key|
"@#{key} #{star_keyword conditions[key], key}" unless ( conditions[key].nil? || conditions[key].empty? )
}.join(' ')).strip
end

What is returned in both these cases is similar to a query such as Model.search String.new or Model.search nil.

Thoughts?