return [ $tagsToAdd, $tagsToRemove, $prevTags ];
}
- private static function getPrevTags( $rc_id = null, $rev_id = null, $log_id = null ) {
+ private static function getPrevTags( $rc_id = null, $log_id = null, $rev_id = null ) {
$conds = array_filter(
[
'ct_rc_id' => $rc_id,
$dbw->delete( 'change_tag_def', '*' );
$rcId = 123;
- ChangeTags::updateTags( [ 'tag1', 'tag2' ], [], $rcId );
+ $revId = 341;
+ ChangeTags::updateTags( [ 'tag1', 'tag2' ], [], $rcId, $revId );
$dbr = wfGetDB( DB_REPLICA );
$expected2 = [
(object)[
'ct_tag_id' => 1,
- 'ct_rc_id' => 123
+ 'ct_rc_id' => 123,
+ 'ct_rev_id' => 341
],
(object)[
'ct_tag_id' => 2,
- 'ct_rc_id' => 123
+ 'ct_rc_id' => 123,
+ 'ct_rev_id' => 341
],
];
- $res2 = $dbr->select( 'change_tag', [ 'ct_tag_id', 'ct_rc_id' ], '' );
+ $res2 = $dbr->select( 'change_tag', [ 'ct_tag_id', 'ct_rc_id', 'ct_rev_id' ], '' );
$this->assertEquals( $expected2, iterator_to_array( $res2, false ) );
$rcId = 124;
- ChangeTags::updateTags( [ 'tag1' ], [], $rcId );
-
- ChangeTags::updateTags( [ 'tag3' ], [], $rcId );
+ $revId = 342;
+ ChangeTags::updateTags( [ 'tag1' ], [], $rcId, $revId );
+ ChangeTags::updateTags( [ 'tag3' ], [], $rcId, $revId );
$dbr = wfGetDB( DB_REPLICA );
$expected2 = [
(object)[
'ct_tag_id' => 1,
- 'ct_rc_id' => 123
+ 'ct_rc_id' => 123,
+ 'ct_rev_id' => 341
],
(object)[
- 'ct_tag_id' => 2,
- 'ct_rc_id' => 123
+ 'ct_tag_id' => 1,
+ 'ct_rc_id' => 124,
+ 'ct_rev_id' => 342
],
(object)[
- 'ct_tag_id' => 1,
- 'ct_rc_id' => 124
+ 'ct_tag_id' => 2,
+ 'ct_rc_id' => 123,
+ 'ct_rev_id' => 341
],
(object)[
'ct_tag_id' => 3,
- 'ct_rc_id' => 124
+ 'ct_rc_id' => 124,
+ 'ct_rev_id' => 342
],
];
- $res2 = $dbr->select( 'change_tag', [ 'ct_tag_id', 'ct_rc_id' ], '' );
+ $res2 = $dbr->select( 'change_tag', [ 'ct_tag_id', 'ct_rc_id', 'ct_rev_id' ], '' );
$this->assertEquals( $expected2, iterator_to_array( $res2, false ) );
}