Page MenuHomePhabricator

Convert ApprovalBallot to use OOUI
Closed, ResolvedPublic

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

Event Timeline

Change 672726 had a related patch set uploaded (by Wikitrent; owner: Wikitrent):
[mediawiki/extensions/SecurePoll@master] WIP: Update ApprovalBallot.php to OOUI

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

Change 672726 merged by jenkins-bot:
[mediawiki/extensions/SecurePoll@master] Update ApprovalBallot.php to OOUI

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

dom_walden subscribed.

For an election of type "Approval vote", I tested submitting votes and checking the votes were recorded correctly in the database.

I inspected the HTML of the voting form, compared to how it used to be. The checkbox <input>s name and value attributes are unaffected, so we should not have affected how votes are recorded.

I also checked that the <label> corresponds to the correct <input> (even when the questions/options are randomised), so users are voting for the right thing.

I tested voting without JS as well.

Test Environment: https://vote.wikimedia.beta.wmflabs.org SecurePoll 2.0.0 (97412a0) 07:26, 19 March 2021.