Skip to content

Commit 09ba20e

Browse files
jenkins-botGerrit Code Review
authored andcommitted
Merge "Use SelectQueryBuilder in ReplicaMasterAwareRecordIdsAcquirer"
2 parents c3cf4e1 + 550ed41 commit 09ba20e

File tree

1 file changed

+8
-14
lines changed

1 file changed

+8
-14
lines changed

lib/includes/Store/Sql/Terms/Util/ReplicaMasterAwareRecordIdsAcquirer.php

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -277,23 +277,17 @@ private function findExistingRecords( IDatabase $db, array $neededRecords ): arr
277277
* an optional argument to self::acquireIds() for instance, the current solution
278278
* in here can be a fallback when that isn't given.
279279
*/
280-
$selectColumns = array_keys( $neededRecords[0] );
281-
$selectColumns[] = $this->idColumn;
282-
283-
$existingRows = $db->select(
284-
$this->table,
285-
$selectColumns,
286-
$db->makeList( $recordsSelectConditions, IDatabase::LIST_OR ),
287-
__METHOD__
288-
);
280+
$existingRows = $db->newSelectQueryBuilder()
281+
->select( array_keys( $neededRecords[0] ) )
282+
->select( $this->idColumn )
283+
->from( $this->table )
284+
->where( $db->makeList( $recordsSelectConditions, IDatabase::LIST_OR ) )
285+
->caller( __METHOD__ )
286+
->fetchResultSet();
289287

290288
$existingRecords = [];
291289
foreach ( $existingRows as $row ) {
292-
$existingRecord = [];
293-
foreach ( $selectColumns as $column ) {
294-
$existingRecord[$column] = $row->$column;
295-
}
296-
$existingRecords[] = $existingRecord;
290+
$existingRecords[] = (array)$row;
297291
}
298292

299293
return $existingRecords;

0 commit comments

Comments
 (0)