Opened 7 years ago
Last modified 3 years ago
#29222 new Bug
Substr on NULL values returns incorrect results with pattern lookups.
Description (last modified by ) ¶
feb683c4c2c5ecfb61e4cb490c3e357450c0c0e8 revealed an unrelated issue on Oracle. SUBSTR(NULL, x, y)
returns NULL
on Oracle which can be concatenate with other strings, hence if we use it with pattern lookups then all rows match an query, e.g.
__startswith=Substr(sth, x, y)
->LIKE SUBSTR(sth, x, y) || '%'
->LIKE '%'
,__endswith=Substr(sth, x, y)
->LIKE '%' || SUBSTR(sth, x, y)
->LIKE '%'
,__contains=Substr(sth, x, y)
->LIKE '%' || SUBSTR(sth, x, y) || '%'
->LIKE '%%'
,
which is unexpected.
"Although Oracle treats zero-length character strings as nulls, concatenating a zero-length character string with another operand always results in the other operand, so null can result only from the concatenation of two null strings. However, this may not continue to be true in future versions of Oracle Database. To concatenate an expression that might be null, use the NVL function to explicitly convert the expression to a zero-length string." (see https://docs.oracle.com/en/database/oracle/oracle-database/21/sqlrf/Concatenation-Operator.html#GUID-08C10738-706B-4290-B7CD-C279EBC90F7E).
According to the ticket's flags, the next step(s) to move this issue forward are:
- To provide a patch by sending a pull request. Claim the ticket when you start working so that someone else doesn't duplicate effort. Before sending a pull request, review your work against the patch review checklist. Check the "Has patch" flag on the ticket after sending a pull request and include a link to the pull request in the ticket comment when making that update. The usual format is:
[https://github.com/django/django/pull/#### PR]
.
Change History (5)
comment:1 by , 7 years ago
Cc: | added |
---|
comment:2 by , 7 years ago
Description: | modified (diff) |
---|
comment:3 by , 7 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:4 by , 7 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:5 by , 3 years ago
Description: | modified (diff) |
---|---|
Owner: | removed |
Status: | assigned → new |