Page MenuHomePhabricator

Convert PreferentialBallot to use OOUI
Closed, ResolvedPublic3 Estimated Story Points

Description

As part of T273044: Convert SecurePoll VotePage to use OOUI, this included ballot will need to return an OOUI object. You should check the getForm and getQuestionForm functions. VotePage.php calls getForm which calls getQuestionForm.

Details

Related Changes in Gerrit:

Event Timeline

Change 674550 had a related patch set uploaded (by STran; owner: STran):
[mediawiki/extensions/SecurePoll@master] Use OOUI in PreferentialBallot

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

• Niharika set the point value for this task to 3.Mar 24 2021, 4:49 PM

Change 674550 merged by jenkins-bot:

[mediawiki/extensions/SecurePoll@master] Use OOUI in PreferentialBallot

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

dom_walden subscribed.

I checked that I can submit a Schulze ballot and the vote is correctly recorded in the database.

I notice the validation messages are different. So far, I haven't noticed any difference in what the validation will allow/not allow. It expects an integer between 1 and 999, and it allows the same rank to be repeated across options (same as before this change).

For example, before this change, when trying to submit without all required questions answered:

before_required.png (902×655 px, 69 KB)

After this change:

after_required.png (644×496 px, 25 KB)

Before this change when entering an invalid answer:

before_invalid_answer.png (752×547 px, 33 KB)

After this change:

after_invalid_answer.png (624×499 px, 25 KB)

Test Environment: https://vote.wikimedia.beta.wmflabs.org SecurePoll 2.0.0 (69d5f55b) 07:38, 1 April 2021