diff options
| author | Umherirrender <umherirrender_de.wp@web.de> | 2024-04-21 14:24:21 +0200 |
|---|---|---|
| committer | Umherirrender <umherirrender_de.wp@web.de> | 2024-04-21 14:24:21 +0200 |
| commit | fea5c2f68736c1d7e6dfe1ffd3179527a25e5df4 (patch) | |
| tree | c02ae1a62f19ce19f5ee58662eeacc3b24165908 /maintenance/migrateExternallinks.php | |
| parent | 401c78b10e0aa5ef49f2a3894756cb2287c7666f (diff) | |
| download | mediawikicore-fea5c2f68736c1d7e6dfe1ffd3179527a25e5df4.tar.gz mediawikicore-fea5c2f68736c1d7e6dfe1ffd3179527a25e5df4.zip | |
Use expression builder to avoid raw sql via BETWEEN operator
Replace BETWEEN with >= and <= operator
Change-Id: Ic21b6f4cc11c773c967d9d4c5f20e762c2ff9629
Diffstat (limited to 'maintenance/migrateExternallinks.php')
| -rw-r--r-- | maintenance/migrateExternallinks.php | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/maintenance/migrateExternallinks.php b/maintenance/migrateExternallinks.php index b07868b80fcd..e6e7cff878b2 100644 --- a/maintenance/migrateExternallinks.php +++ b/maintenance/migrateExternallinks.php @@ -69,7 +69,7 @@ class MigrateExternallinks extends LoggedUpdateMaintenance { private function handleBatch( $lowId ) { $batchSize = $this->getBatchSize(); - // BETWEEN is inclusive, let's subtract one. + // range is inclusive, let's subtract one. $highId = $lowId + $batchSize - 1; $dbw = $this->getPrimaryDB(); $updated = 0; @@ -78,7 +78,8 @@ class MigrateExternallinks extends LoggedUpdateMaintenance { ->from( 'externallinks' ) ->where( [ 'el_to_domain_index' => '', - "el_id BETWEEN $lowId AND $highId" + $dbw->expr( 'el_id', '>=', $lowId ), + $dbw->expr( 'el_id', '<=', $highId ), ] ) ->limit( $batchSize ) ->caller( __METHOD__ ) |