Change Details

Now that {T210206} is done, this extension should migrate away from building and passing around raw SQL to expression builders. It improves readability and security of the code and is more aligned with industry practices easing onboarding. For more information check T210206 and T350075. Calls to Database::addQuotes(), ::buildLike(), ::makeList() indicate that raw SQL is being built and passed around. Relevant codesearches: * [x] ~~[[ https://codesearch.wmcloud.org/search/?q=buildLike&files=&excludeFiles=&repos=Extension%3AORES | buildLike (0) ]]~~ * [x] [[ https://codesearch.wmcloud.org/search/?q=addQuotes&files=&excludeFiles=&repos=Extension%3AORES | addQuotes (6) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1004348 | gerrit:1004348]]) * [ ] [[ https://codesearch.wmcloud.org/search/?q=makeList&files=&excludeFiles=&repos=Extension%3AORES | makeList (4) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1007485 | gerrit:1007485]])

Now that {T210206} is done, this extension should migrate away from building and passing around raw SQL to expression builders. It improves readability and security of the code and is more aligned with industry practices easing onboarding. For more information check T210206 and T350075. Calls to Database::addQuotes(), ::buildLike(), ::makeList() indicate that raw SQL is being built and passed around. Relevant codesearches: * [x] ~~[[ https://codesearch.wmcloud.org/search/?q=buildLike&files=&excludeFiles=&repos=Extension%3AORES | buildLike (0) ]]~~ * [x] [[ https://codesearch.wmcloud.org/search/?q=addQuotes&files=&excludeFiles=&repos=Extension%3AORES | addQuotes (6) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1004348 | gerrit:1004348]]) * [x] [[ https://codesearch.wmcloud.org/search/?q=makeList&files=&excludeFiles=&repos=Extension%3AORES | makeList (4) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1007485 | gerrit:1007485]])

Now that {T210206} is done, this extension should migrate away from building and passing around raw SQL to expression builders. It improves readability and security of the code and is more aligned with industry practices easing onboarding. For more information check T210206 and T350075. Calls to Database::addQuotes(), ::buildLike(), ::makeList() indicate that raw SQL is being built and passed around. Relevant codesearches: * [x] ~~[[ https://codesearch.wmcloud.org/search/?q=buildLike&files=&excludeFiles=&repos=Extension%3AORES | buildLike (0) ]]~~ * [x] [[ https://codesearch.wmcloud.org/search/?q=addQuotes&files=&excludeFiles=&repos=Extension%3AORES | addQuotes (6) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1004348 | gerrit:1004348]]) * [ * [x] [[ https://codesearch.wmcloud.org/search/?q=makeList&files=&excludeFiles=&repos=Extension%3AORES | makeList (4) ]] ([[https://gerrit.wikimedia.org/r/c/mediawiki/extensions/ORES/+/1007485 | gerrit:1007485]])