Page MenuHomePhabricator

remove $wgSecurePollSingleTransferableVoteEnabled?
Closed, ResolvedPublic1 Estimated Story Points

Description

What

  • remove $wgSecurePollSingleTransferableVoteEnabled from SecurePoll code, and replace it with always on behavior
  • remove $wgSecurePollSingleTransferableVoteEnabled from InitializeSettings.php

Why

  • easier for new devs working on SecurePoll who don't have to remember to turn this on when wanting to work on tickets related to STV
  • seems harmless to have this extra poll type displayed all the time in Special:SecurePoll/create. are there any wiki farms that have this poll type turned off or have a compelling reason to have this turned off?

Event Timeline

Novem_Linguae set the point value for this task to 1.Oct 10 2024, 6:20 PM

T282013: Introduce a feature flag for SecurePoll work on Single Transferable Vote [XS]

I'm guessing it was mostly introduced as a feature flag until it was deemed stable/whatever.

Doing this seems to go along with the general idea we have of trying to have "sensible" defaults.

Certainly could change the default, and then look to remove it in a later MW release, if it is deemed to have some use longer term...

Change #1115911 had a related patch set uploaded (by Mimurawil; author: Mimurawil):

[mediawiki/extensions/SecurePoll@master] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1115911

Change #1115917 had a related patch set uploaded (by Mimurawil; author: Mimurawil):

[operations/mediawiki-config@master] Remove flag wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1115917

Mimurawil changed the task status from Open to In Progress.Feb 1 2025, 1:04 AM

Change #1115911 merged by jenkins-bot:

[mediawiki/extensions/SecurePoll@master] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1115911

Change #1117171 had a related patch set uploaded (by Máté Szabó; author: Mimurawil):

[mediawiki/extensions/SecurePoll@wmf/1.44.0-wmf.14] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1117171

Change #1117172 had a related patch set uploaded (by Máté Szabó; author: Mimurawil):

[mediawiki/extensions/SecurePoll@wmf/1.44.0-wmf.15] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1117172

Change #1117172 merged by jenkins-bot:

[mediawiki/extensions/SecurePoll@wmf/1.44.0-wmf.15] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1117172

Change #1117171 merged by jenkins-bot:

[mediawiki/extensions/SecurePoll@wmf/1.44.0-wmf.14] Remove flag $wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1117171

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:15:59Z] <mszabo@deploy2002> Started scap sync-world: Backport for [[gerrit:1117172|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]], [[gerrit:1117171|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]]

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:20:52Z] <mszabo@deploy2002> mszabo: Backport for [[gerrit:1117172|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]], [[gerrit:1117171|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:29:45Z] <mszabo@deploy2002> Finished scap sync-world: Backport for [[gerrit:1117172|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]], [[gerrit:1117171|Remove flag $wgSecurePollSingleTransferableVoteEnabled (T376930)]] (duration: 13m 46s)

Change #1115917 merged by jenkins-bot:

[operations/mediawiki-config@master] Remove flag wgSecurePollSingleTransferableVoteEnabled

https://gerrit.wikimedia.org/r/1115917

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:33:15Z] <mszabo@deploy2002> Started scap sync-world: Backport for [[gerrit:1115917|Remove flag wgSecurePollSingleTransferableVoteEnabled (T376930)]]

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:36:54Z] <mszabo@deploy2002> mimurawil, mszabo: Backport for [[gerrit:1115917|Remove flag wgSecurePollSingleTransferableVoteEnabled (T376930)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-02-04T15:44:36Z] <mszabo@deploy2002> Finished scap sync-world: Backport for [[gerrit:1115917|Remove flag wgSecurePollSingleTransferableVoteEnabled (T376930)]] (duration: 11m 21s)

Novem_Linguae updated the task description. (Show Details)

Nothing is showing up in CodeSearch. I think this ticket is complete.