dépôts
/
lhc
/
web
/
wiklou.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Do not assume rc_patrolled is binary in ChangesListSpecialPage
[lhc/web/wiklou.git]
/
includes
/
specialpage
/
ChangesListSpecialPage.php
diff --git
a/includes/specialpage/ChangesListSpecialPage.php
b/includes/specialpage/ChangesListSpecialPage.php
index
5aa1c6b
..
b9d20be
100644
(file)
--- a/
includes/specialpage/ChangesListSpecialPage.php
+++ b/
includes/specialpage/ChangesListSpecialPage.php
@@
-121,7
+121,11
@@
abstract class ChangesListSpecialPage extends SpecialPage {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
- $conds[] = 'rc_user = 0';
+ $actorMigration = ActorMigration::newMigration();
+ $actorQuery = $actorMigration->getJoin( 'rc_user' );
+ $tables += $actorQuery['tables'];
+ $join_conds += $actorQuery['joins'];
+ $conds[] = $actorMigration->isAnon( $actorQuery['fields']['rc_user'] );
},
'isReplacedInStructuredUi' => true,
},
'isReplacedInStructuredUi' => true,
@@
-135,7
+139,11
@@
abstract class ChangesListSpecialPage extends SpecialPage {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
- $conds[] = 'rc_user != 0';
+ $actorMigration = ActorMigration::newMigration();
+ $actorQuery = $actorMigration->getJoin( 'rc_user' );
+ $tables += $actorQuery['tables'];
+ $join_conds += $actorQuery['joins'];
+ $conds[] = $actorMigration->isNotAnon( $actorQuery['fields']['rc_user'] );
},
'isReplacedInStructuredUi' => true,
]
},
'isReplacedInStructuredUi' => true,
]
@@
-220,8
+228,10
@@
abstract class ChangesListSpecialPage extends SpecialPage {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
- $user = $ctx->getUser();
- $conds[] = 'rc_user_text != ' . $dbr->addQuotes( $user->getName() );
+ $actorQuery = ActorMigration::newMigration()->getWhere( $dbr, 'rc_user', $ctx->getUser() );
+ $tables += $actorQuery['tables'];
+ $join_conds += $actorQuery['joins'];
+ $conds[] = 'NOT(' . $actorQuery['conds'] . ')';
},
'cssClassSuffix' => 'self',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
},
'cssClassSuffix' => 'self',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
@@
-236,8
+246,11
@@
abstract class ChangesListSpecialPage extends SpecialPage {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
- $user = $ctx->getUser();
- $conds[] = 'rc_user_text = ' . $dbr->addQuotes( $user->getName() );
+ $actorQuery = ActorMigration::newMigration()
+ ->getWhere( $dbr, 'rc_user', $ctx->getUser(), false );
+ $tables += $actorQuery['tables'];
+ $join_conds += $actorQuery['joins'];
+ $conds[] = $actorQuery['conds'];
},
'cssClassSuffix' => 'others',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
},
'cssClassSuffix' => 'others',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
@@
-477,7
+490,7
@@
abstract class ChangesListSpecialPage extends SpecialPage {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
'queryCallable' => function ( $specialClassName, $ctx, $dbr, &$tables, &$fields, &$conds,
&$query_options, &$join_conds
) {
- $conds[] = 'rc_patrolled
= 1
';
+ $conds[] = 'rc_patrolled
!= 0
';
},
'cssClassSuffix' => 'unpatrolled',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
},
'cssClassSuffix' => 'unpatrolled',
'isRowApplicableCallable' => function ( $ctx, $rc ) {
@@
-844,7
+857,7
@@
abstract class ChangesListSpecialPage extends SpecialPage {
*/
protected function outputTimeout() {
$this->getOutput()->addHTML(
*/
protected function outputTimeout() {
$this->getOutput()->addHTML(
- '<div class="mw-changeslist-timeout">' .
+ '<div class="mw-changeslist-
empty mw-changeslist-
timeout">' .
$this->msg( 'recentchanges-timeout' )->parse() .
'</div>'
);
$this->msg( 'recentchanges-timeout' )->parse() .
'</div>'
);
@@
-1702,22
+1715,27
@@
abstract class ChangesListSpecialPage extends SpecialPage {
return;
}
return;
}
+ $actorMigration = ActorMigration::newMigration();
+ $actorQuery = $actorMigration->getJoin( 'rc_user' );
+ $tables += $actorQuery['tables'];
+ $join_conds += $actorQuery['joins'];
+
// 'registered' but not 'unregistered', experience levels, if any, are included in 'registered'
if (
in_array( 'registered', $selectedExpLevels ) &&
!in_array( 'unregistered', $selectedExpLevels )
) {
// 'registered' but not 'unregistered', experience levels, if any, are included in 'registered'
if (
in_array( 'registered', $selectedExpLevels ) &&
!in_array( 'unregistered', $selectedExpLevels )
) {
- $conds[] =
'rc_user != 0'
;
+ $conds[] =
$actorMigration->isNotAnon( $actorQuery['fields']['rc_user'] )
;
return;
}
if ( $selectedExpLevels === [ 'unregistered' ] ) {
return;
}
if ( $selectedExpLevels === [ 'unregistered' ] ) {
- $conds[] =
'rc_user = 0'
;
+ $conds[] =
$actorMigration->isAnon( $actorQuery['fields']['rc_user'] )
;
return;
}
$tables[] = 'user';
return;
}
$tables[] = 'user';
- $join_conds['user'] = [ 'LEFT JOIN',
'rc_user
= user_id' ];
+ $join_conds['user'] = [ 'LEFT JOIN',
$actorQuery['fields']['rc_user'] . '
= user_id' ];
if ( $now === 0 ) {
$now = time();
if ( $now === 0 ) {
$now = time();
@@
-1747,7
+1765,7
@@
abstract class ChangesListSpecialPage extends SpecialPage {
if ( in_array( 'unregistered', $selectedExpLevels ) ) {
$selectedExpLevels = array_diff( $selectedExpLevels, [ 'unregistered' ] );
if ( in_array( 'unregistered', $selectedExpLevels ) ) {
$selectedExpLevels = array_diff( $selectedExpLevels, [ 'unregistered' ] );
- $conditions[] =
'rc_user = 0'
;
+ $conditions[] =
$actorMigration->isAnon( $actorQuery['fields']['rc_user'] )
;
}
if ( $selectedExpLevels === [ 'newcomer' ] ) {
}
if ( $selectedExpLevels === [ 'newcomer' ] ) {
@@
-1769,7
+1787,7
@@
abstract class ChangesListSpecialPage extends SpecialPage {
} elseif ( $selectedExpLevels === [ 'experienced', 'learner' ] ) {
$conditions[] = $aboveNewcomer;
} elseif ( $selectedExpLevels === [ 'experienced', 'learner', 'newcomer' ] ) {
} elseif ( $selectedExpLevels === [ 'experienced', 'learner' ] ) {
$conditions[] = $aboveNewcomer;
} elseif ( $selectedExpLevels === [ 'experienced', 'learner', 'newcomer' ] ) {
- $conditions[] =
'rc_user != 0'
;
+ $conditions[] =
$actorMigration->isNotAnon( $actorQuery['fields']['rc_user'] )
;
}
if ( count( $conditions ) > 1 ) {
}
if ( count( $conditions ) > 1 ) {