Per the WMF response to the Media Viewer RfC on Commons:
Media Viewer should be enabled by default for anonymous users, but disabled for logged-in users.
Version: wmf-deployment
Severity: enhancement
See Also:
https://bugzilla.wikimedia.org/show_bug.cgi?id=69080
gerritadmin wrote:
Change 154395 had a related patch set uploaded by Gergő Tisza:
Add an option to disable by default for logged-in users
gerritadmin wrote:
Change 154396 had a related patch set uploaded by Gergő Tisza:
Disable MediaViewer by default for logged-in users on Commons
gerritadmin wrote:
Change 154396 merged by jenkins-bot:
Disable MediaViewer by default for logged-in users on Commons
gerritadmin wrote:
Change 155443 had a related patch set uploaded by MarkTraceur:
Add an option to disable by default for logged-in users
gerritadmin wrote:
Change 154395 merged by jenkins-bot:
Add an option to disable by default for logged-in users
gerritadmin wrote:
Change 155443 merged by MarkTraceur:
Add an option to disable by default for logged-in users
Note: Also ran a query manually to set the opt-in preference if the user disabled the gadget. It was like 50 people.
The change of default caused some junk to be written in the user_properties table (details in bug 69942):
select up_value, count(*) from user_properties where up_property = 'multimediaviewer-enable' group by up_value;
+----------+----------+
| up_value | count(*) |
+----------+----------+
| 1560 | |
| 0 | 2 |
| 1 | 97 |
| true | 17 |
+----------+----------+
Fixed in https://gerrit.wikimedia.org/r/#/c/156004/ , needs DB cleanup.
The 'toggle' preference type sets an up_value of '1' for enabled, '' for disabled, but the row is omitted if the setting is the same as the default. Commons has MediaViewer disabled as default for logged-in users, so the cleanup query is
UPDATE user_properties SET up_value = '1' WHERE up_property = 'multimediaviewer-enable' AND up_value = 'true';
DELETE FROM user_properties WHERE up_property = 'multimediaviewer-enable' AND up_value IN ( '', '0' );
(In reply to Tisza Gergő from comment #8)
Fixed in https://gerrit.wikimedia.org/r/#/c/156004/ , needs DB cleanup.
Tisza: Who's going to do the DB cleanup (and when)?
(In reply to Andre Klapper from comment #10)
Tisza: Who's going to do the DB cleanup (and when)?
Someone from the Multimedia team will do this. Probably this week, although it is not particularly urgent (functionally there is no difference between the correct and incorrect values, but it's ugly and will probably mess up stats scripts).
tgr@terbium:~$ mwscript sql.php --wiki=commonswiki clean-mmv-user-property.sql
Query OK, 17 row(s) affected
Query OK, 1555 row(s) affected
...
mysql:research@s4-analytics-slave.eqiad.wmnet [commonswiki]> select up_value, count(*) from user_properties where up_property = 'multimediaviewer-enable' group by up_value;
+----------+----------+
| up_value | count(*) |
+----------+----------+
| 3 | |
| 1 | 223 |
+----------+----------+
2 rows in set (0.01 sec)
So, um. MV still seems to be setting '' values. Probably I should redo the optout patch properly, with $wgDefaultUserOptions['multimediaviewer-enable'] set to 1 instead of true. I'll do that once I don't have anything important to do.
Since this is not really related to the original request anymore, I moved it to bug 72557.