jordiserra discovered this issue when working with the Search API Ranges module. However, I don't think a patch was ever submitted. So I'm submitting it now.

http://drupal.org/node/1320790#comment-5164472

CommentFileSizeAuthor
#5 max_inclusive-1363114-5.patch660 bytesemorency
#4 max_inclusive-136311-4.patch660 bytesemorency
query_type_term.inc_.patch682 bytesnbucknor

Comments

nbucknor’s picture

nbucknor commented
Status: Active » Needs review
drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented

Thanks for submitting the patch!
I don't really know which behaviour could be seen as the „expected“ one, but if no-one objects in the next few days I'm inclined to follow your opinion there.

emorency’s picture

emorency commented

For SOLR at least, the specification is that it should be a "<="...

See http://wiki.apache.org/solr/SolrQuerySyntax:

"field:[* TO 100] finds all field values less than or equal to 100"

emorency’s picture

emorency commented
StatusFileSize
new max_inclusive-136311-4.patch660 bytes

This git patch applies on 7.x.1.x

emorency’s picture

emorency commented
StatusFileSize
new max_inclusive-1363114-5.patch660 bytes

With the correct file name this time...

Shadlington’s picture

Shadlington commented

#5 works for me :D
I too think that should be the default behaviour

warning: contrib/search_api_facetapi/plugins/facetapi/query_type_term.inc has type 100755, expected 100644

mollux’s picture

mollux commented
Status: Needs review » Reviewed & tested by the community

works and solves the problem

drunken monkey’s picture

drunken monkey
he/him
Primary language German
Location Vienna, Austria
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed!

Status: Fixed » Closed (fixed)

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