r62140 MediaWiki - Code Review archive
Repository:
MediaWiki
Revision:
<
r62139
|
r62140
|
r62141
>
Date:
22:01, 8 February 2010
Author:
catrope
Status:
ok
Tags:
Comment:
OptIn: Show "Leave Beta" link if a user has ANY feature enabled (rather than only displaying it if they have ALL of them). Also don't consider the skin preference for this.
Modified paths:
/trunk/extensions/UsabilityInitiative/OptIn/SpecialOptIn.php
(modified) (
history
)
Diff
[
purge
]
Index: trunk/extensions/UsabilityInitiative/OptIn/SpecialOptIn.php
—
—
@@ -25,11 +25,11 @@
26
26
return false;
27
27
28
28
foreach ( $wgOptInPrefs as $pref => $value ) {
29
- if ( $user->getOption( $pref ) != $value ) {
30
- return false;
29
+ if ( $pref != 'skin' && $user->getOption( $pref ) == $value ) {
30
+ return true;
31
31
}
32
32
}
33
- return true;
33
+ return false;
34
34
}
35
35
36
36
public static function checkToken() {
Follow-up revisions
Revision
Commit summary
Author
Date
r62141
wmf-deployment: Merge
r62140
(OptIn fix) from trunk
catrope
22:05, 8 February 2010
Status & tagging log
13:31, 23 February 2010
😂
(
talk
|
contribs
)
changed the
status
of r62140
[
removed:
new
added:
ok]