Hello Reedy, Jack Phoenix, Legoktm, jenkins-bot,

I'd like you to do a code review.  Please visit

    https://gerrit.wikimedia.org/r/349278

to review the following change.


Change subject: Revert "Use $dbr->buildLike() instead of manual LIKE"
......................................................................

Revert "Use $dbr->buildLike() instead of manual LIKE"

This breaks SQL wildcards.

This reverts commit 2a63769238f19bc8d3533f31f4c226e9cce78856.

Change-Id: I473fcabf147b24b266ba6556d32a3990274da4ab
---
M Nuke_body.php
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Nuke 
refs/changes/78/349278/1

diff --git a/Nuke_body.php b/Nuke_body.php
index 6cfe5ef..8458db5 100644
--- a/Nuke_body.php
+++ b/Nuke_body.php
@@ -268,7 +268,7 @@
 
                $pattern = $this->getRequest()->getText( 'pattern' );
                if ( !is_null( $pattern ) && trim( $pattern ) !== '' ) {
-                       $where[] = 'rc_title ' . $dbr->buildLike( $pattern );
+                       $where[] = 'rc_title LIKE ' . $dbr->addQuotes( $pattern 
);
                }
                $group = implode( ', ', $what );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/349278
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I473fcabf147b24b266ba6556d32a3990274da4ab
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Nuke
Gerrit-Branch: master
Gerrit-Owner: Mattflaschen <[email protected]>
Gerrit-Reviewer: Jack Phoenix <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Reedy <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to