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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nbucknor’s picture

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

drunken monkey
he/him
Primary language German
Location Vienna, Austria
CreditAttribution: drunken monkey 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 CreditAttribution: 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 CreditAttribution: emorency commented
FileSize
max_inclusive-136311-4.patch660 bytes

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

emorency’s picture

emorency CreditAttribution: emorency commented
FileSize
max_inclusive-1363114-5.patch660 bytes

With the correct file name this time...

Shadlington’s picture

Shadlington CreditAttribution: 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 CreditAttribution: 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
CreditAttribution: drunken monkey commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed!

Status: Fixed » Closed (fixed)

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