Last modified: 2011-06-23 20:23:12 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T24989, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 22989 - wfArrayToCGI breaks associative arrays
wfArrayToCGI breaks associative arrays
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
General/Unknown (Other open bugs)
unspecified
All All
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-03-28 19:24 UTC by Church of emacs
Modified: 2011-06-23 20:23 UTC (History)
3 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
uses array keys (517 bytes, patch)
2010-03-28 19:24 UTC, Church of emacs
Details

Description Church of emacs 2010-03-28 19:24:36 UTC
Created attachment 7252 [details]
uses array keys

Example: SemanticMediaWiki with a language != English
Special:Ask redirects to Special:<Localized Ask> and omits array keys. (e.g. foo[bar]=baz is turned into foo[]=baz)

See this mail:
http://www.mail-archive.com/semediawiki-devel@lists.sourceforge.net/msg01922.html

wfArrayToCGI is responsible for this.

One fix would be to always use the key, however I think this would turn foo[]=baz into foo[0]=baz (which is not a nice behavior, it should just stay the same). See patch.
Comment 1 Church of emacs 2010-03-29 17:28:01 UTC
Fixed the SemanticMediaWiki example with a workaround in r64351.

However, that does not resolve this bug, it might happen somewhere else as well.
Comment 2 p858snake 2011-04-30 00:10:00 UTC
*Bulk BZ Change: +Patch to open bugs with patches attached that are missing the keyword*
Comment 3 Matěj Grabovský 2011-06-23 15:25:42 UTC
Fixed in r90661.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links