Last modified: 2013-07-24 15:13:38 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 T49074, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 47074 - semantic queries - names of query options are transformed and cause breakage / decodeParameters()
semantic queries - names of query options are transformed and cause breakage ...
Status: UNCONFIRMED
Product: MediaWiki extensions
Classification: Unclassified
Semantic MediaWiki (Other open bugs)
REL1_20-branch
All All
: Unprioritized normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2013-04-10 11:57 UTC by Dima
Modified: 2013-07-24 15:13 UTC (History)
5 users (show)

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


Attachments

Description Dima 2013-04-10 11:57:11 UTC
Link to the search query is not working, because the spaces in parameter names are replaced by underscores


Error in decodeParameters()

$wgRequest->getValues();


["named_args"]=1

must be

["named args"]=1


Resolving:

446: $result = $wgRequest->getValues();

=>

446: foreach ($wgRequest->getValues() as $key => $value) {
447:        $result[str_replace( '_', ' ', $key)] = $value;
448: }
Comment 1 Andre Klapper 2013-07-24 12:37:19 UTC
Hi! Thanks for your report!

You are welcome to use Developer access
  https://www.mediawiki.org/wiki/Developer_access
to submit this code change as a Git branch directly into Gerrit:
  https://www.mediawiki.org/wiki/Git/Tutorial

Putting your branch in Git makes it easier to review it quickly.
Thanks again! We appreciate your contribution.
Comment 2 Andre Klapper 2013-07-24 15:13:38 UTC
Dima: Could ypu provide steps to reproduce the problem, please?

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


Navigation
Links