r86974 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r86973‎ | r86974 | r86975 >
Date:19:23, 26 April 2011
Author:maxsem
Status:ok
Tags:
Comment:
Oops, forgot to revert r86954 in r86973, not needed anymore
Modified paths:
  • /trunk/phase3/includes/api/ApiUserrights.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/api/ApiUserrights.php
@@ -96,11 +96,11 @@
9797 ApiBase::PARAM_REQUIRED => true
9898 ),
9999 'add' => array(
100 - ApiBase::PARAM_TYPE => array_values( User::getAllGroups() ),
 100+ ApiBase::PARAM_TYPE => User::getAllGroups(),
101101 ApiBase::PARAM_ISMULTI => true
102102 ),
103103 'remove' => array(
104 - ApiBase::PARAM_TYPE => array_values( User::getAllGroups() ),
 104+ ApiBase::PARAM_TYPE => User::getAllGroups(),
105105 ApiBase::PARAM_ISMULTI => true
106106 ),
107107 'token' => null,

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r86954Without this array_values(), user groups will be exported to JSON as object i...maxsem17:29, 26 April 2011
r86973Follow-up r86954: make one centralised call to array_values(), there are more...maxsem19:20, 26 April 2011

Status & tagging log