jenkins-bot has submitted this change and it was merged. Change subject: (bug 42639) Fix API action=options for multiselect prefs ......................................................................
(bug 42639) Fix API action=options for multiselect prefs Preferences options using HTMLForm's "multiselect" type are stored in the user preferences table as one key with a boolean value for each option in the multiselect. The validation code added in change I98df55f2 does not take this into account, and therefore considers all of these option keys invalid. This changeset fixes that, and adds a unit test to verify correct behavior. Change-Id: I137c74a6045c7b39e2119a8edde2705738879bc9 --- M includes/api/ApiOptions.php M tests/phpunit/includes/api/ApiOptionsTest.php 2 files changed, 74 insertions(+), 3 deletions(-) Approvals: Demon: Looks good to me, approved Matmarex: Looks good to me, but someone else must approve jenkins-bot: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.wikimedia.org/r/36494 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I137c74a6045c7b39e2119a8edde2705738879bc9 Gerrit-PatchSet: 4 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Anomie <[email protected]> Gerrit-Reviewer: Anomie <[email protected]> Gerrit-Reviewer: CSteipp <[email protected]> Gerrit-Reviewer: Catrope <[email protected]> Gerrit-Reviewer: Demon <[email protected]> Gerrit-Reviewer: Matmarex <[email protected]> Gerrit-Reviewer: Reedy <[email protected]> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
