Last modified: 2014-09-23 23:33:32 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 T10262, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 8262 - Allow URL functions to concatenate query string from multiple arguments
Allow URL functions to concatenate query string from multiple arguments
Status: NEW
Product: MediaWiki
Classification: Unclassified
Parser (Other open bugs)
1.9.x
All All
: Low enhancement (vote)
: ---
Assigned To: Nobody - You can work on this!
: newparser
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2006-12-14 21:43 UTC by Carl Fürstenberg
Modified: 2014-09-23 23:33 UTC (History)
3 users (show)

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


Attachments
the patch as described (2.29 KB, patch)
2006-12-14 21:44 UTC, Carl Fürstenberg
Details

Description Carl Fürstenberg 2006-12-14 21:43:41 UTC
When you for example are using {{localurl:foo|query}} sometimes you have
multiple query variables. To make the interface more clean, I would suggest to
apply fortcomming patch to enable the format:
{{localurl:page|query1=a|query2=b}} (and for localurle, fullurl and fullurle),
to generate the url "index.php?title=Page&query1=a&query2=b"
Comment 1 Carl Fürstenberg 2006-12-14 21:44:43 UTC
Created attachment 2867 [details]
the patch as described
Comment 2 Rob Church 2007-02-02 11:51:20 UTC
Any chance of a bit of URL-encoding of values? :)
Comment 3 Waldir 2008-01-13 01:15:46 UTC
comment 15 on bug 5720 notes that this wouldnt be backward compatible.
Comment 4 Sumana Harihareswara 2012-01-11 21:58:01 UTC
Comment on attachment 2867 [details]
the patch as described

Marked as obsolete since patch no longer applies cleanly to Subversion trunk,
per Rusty Burchfield's automated testing
https://docs.google.com/spreadsheet/ccc?key=0Ah_71HHl7qa7dGtvSms3TGpHQU9NU2Y1VmNzUEUteWc
Comment 5 Sumana Harihareswara 2012-01-11 21:59:37 UTC
Carl, thank you for the patch.  I'm removing the need-review and patch
keywords since the MediaWiki codebase has changed so much since you submitted
the patch that the patch no longer applies cleanly to Subversion trunk.  I'm very sorry for the delay in review.

We're currently rewriting the parser, so I am cc'ing Gabriel Wicke, who's working on that effort.  If you're still interested in this issue, maybe you could come into the #mediawiki IRC channel https://www.mediawiki.org/wiki/MediaWiki_on_IRC and discuss the situation and possible approaches before revising your patch and reattaching it.  Thanks!

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


Navigation
Links