EDIT: icontains case not covered when dealing with StringArrayProperties

Question

EDIT: icontains case not covered when dealing with StringArrayProperties

tonjo opened this issue 10 years ago · comments

Row 353 in query.py:

elif condition.operator in (OPERATORS.CONTAINS, OPERATORS.ICONTAINS):
    if isinstance(field._property, StringProperty):

EDIT:
The icontains check should cover the StringArrayProperty case.

Temporary workaround is quite easy: search the .lower() version of the string,
provided the StringArray contains lowercase strings.

Matt Luongo · Answer 1 · Mon Feb 17 2014 05:38:49 GMT+0800 (China Standard Time)

@tonjo when there's no index, this can be solved pretty easily in Cypher. Maybe icontains against an array property should always go straight to the Cypher case, instead of trying to be clever with index usage?

tonjo · Answer 2 · Tue Feb 18 2014 20:00:08 GMT+0800 (China Standard Time)

Not sure I understood this. Are you suggesting to solve it directly with a Cypher query?
Besides, field is indexed, and as workaround I'm cycling through the StringArray.