Issue with enabling BetaFeatures
Closed, DuplicatePublic
Actions

Assigned To

None

Authored By

	Andrew-WMDE
	Oct 5 2017, 11:42 AM

Description

When attempting to enable a BetaFeature (e.g. TwoColConflict) from the user's preferences panel upon clicking save the page reloads with the once selected BetaFeature now deselected and still disabled.

This phenomenon can be seen here during selenium tests for TwoColConflict: https://integration.wikimedia.org/ci/job/mwext-mw-selenium-jessie/6581/artifact/log/Two%20column%20edit%20conflict%20help%3A%20RevisionSlider%20tutorial%20is%20not%20present%20after%20it%20was%20dismissed%20once.mp4

It also occurs on my local development environment shown below:

Mediawiki Version: 1.31.0-alpha (1bb9a22) 03:01, 4 October 2017
BetaFeatures Version: 0.1 (edfd490) 20:31, 30 September 2017
TwoColConflict Version: 0.0.1 (a30c5f3) 09:23, 4 October 2017
https://github.com/addshore/mediawiki-docker-dev

The issue seems to only occur on fresh installations of mediawiki, therefore, it doesn't appear to be causing problems on the production server. Since, I use mediawiki-docker-dev for my local environment I periodically use new installations and thats when the issue was first spotted. This issue does not appear to be TwoColConflict specific and occurs with other BetaFeatures as well.

Related Objects

Mentioned In: T170055: Impossible to turn off certain BetaFeatures when "Automatically enable all new beta features" is selected
T177508: Can't disable a beta feature on Special:Preferences, gets re-enabled again after saving preferences
Mentioned Here: T177478: Unable to change user settings
rEBETedfd49043f3a: Localisation updates from https://translatewiki.net.
rMW1bb9a223d268: Inject dependencies into Shell\Command
rESCCa30c5f34d7f9: Show no page found when not enabled for simulate page

Event Timeline

Andrew-WMDE created this task.Oct 5 2017, 11:42 AM

Restricted Application added a project: TCB-Team (now WMDE-TechWish). · View Herald TranscriptOct 5 2017, 11:42 AM

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Tobi_WMDE_SW added a project: MediaWiki-Core-Preferences.Oct 5 2017, 11:45 AM

+1 I can reproduce this, but only after I pulled core.

Andrew-WMDE updated the task description. (Show Details)Oct 5 2017, 11:52 AM

A quick git bisect tells me that the following commit in core is where the issue starts for me:

7ab57ba290f670847f981d5fc2c79339f1d4844e is the first bad commit
commit 7ab57ba290f670847f981d5fc2c79339f1d4844e
Author: Chad Horohoe <chadh@wikimedia.org>
Date:   Tue Oct 3 14:26:18 2017 -0700

    Stop stubbing StubUserLang

    Stub objects are confusing as heck and are a performance optimization
    that really aren't fit for the modern era. They were designed to avoid
    loading the actual code from the disk back in the days when bytecode
    caching wasn't always gonna be there.

    It's 2017. If you're using HHVM, you've got a bytecode cache. If you're
    using any reasonably recent version of PHP then you've got the opcode
    caching enabled by default in basically every distro-related build.

    Nothing actually relies on this object being a stub (that'd be silly),
    so only references are basically things force unstubbing (also kind of
    silly) the object. Once remaining code referencing this in extensions
    are all cleaned up then we can remove the class itself.

    Change-Id: I15df24aeeb729e8e764792daa933377f35042fab

https://gerrit.wikimedia.org/r/#/c/382067/

Addshore added subscribers: • demon, MaxSem.Oct 5 2017, 12:35 PM

Cause is likely the same as in T177478, it basically affects all user preferences.

Mainframe98 mentioned this in T177508: Can't disable a beta feature on Special:Preferences, gets re-enabled again after saving preferences.Oct 5 2017, 3:52 PM

• Elitre subscribed.Oct 5 2017, 4:04 PM

Addshore closed this task as a duplicate of T177478: Unable to change user settings.Oct 5 2017, 4:51 PM

Tobi_WMDE_SW moved this task from Proposed to Demoed on the WMDE-TechWish board.Oct 10 2017, 9:29 AM

Tobi_WMDE_SW moved this task from Incoming to Edit Conflict Handling on the TCB-Team (now WMDE-TechWish) board.Oct 10 2017, 9:36 AM

Quiddity mentioned this in T170055: Impossible to turn off certain BetaFeatures when "Automatically enable all new beta features" is selected.Feb 4 2018, 5:54 PM

thiemowmde removed a project: TCB-Team (now WMDE-TechWish).Jan 7 2022, 3:25 PM