The regex expressions in CargoSQLQuery.php which check for the use of multi-value fields in #cargo_query calls are over-inclusive in that they match the field "book" and the field "bookworm" when the only field that would allow for multiple values is "book". Additionally the matches are case sensitive while table and column names in most databases are case-insensitive.
Description
Description
Details
Details
Event Timeline
Comment Actions
Change 257266 had a related patch set uploaded (by Ed Hoo):
[Cargo] Fix T120583: #cargo_query HOLDS check used in multi-value fields
Comment Actions
Change 257266 abandoned by Ed Hoo:
[Cargo] Fix T120583: #cargo_query HOLDS check used in multi-value fields
Reason:
Replaced by
https://gerrit.wikimedia.org/r/#/c/258753/
https://gerrit.wikimedia.org/r/#/c/260531/
https://gerrit.wikimedia.org/r/#/c/260951/