Page MenuHomePhabricator

ApiSandbox incorrectly converts spaces from URL parameters
Closed, ResolvedPublic

Description

It seems to only convert the first one; someone missed a g flag somewhere?

Steps to reproduce:

  1. Visit this URL
  2. Press make request, and notice that the request succeeds. Note that spaces in the titles parameter in the URL have been correctly converted to pluses
  3. Reload the page, press make request again, and notice that the request fails due to all but the first plus not being converted back to a space.

Event Timeline

Change 273208 had a related patch set uploaded (by Anomie):
ApiSandbox: Fix String.replace call

https://gerrit.wikimedia.org/r/273208

I already submitted a patch for this when I claimed it, but the bot obviously ignored it for some reason...

https://gerrit.wikimedia.org/r/#/c/273186/

Change 273186 had a related patch set uploaded (by Anomie):
ApiSandbox: Convert all pluses in URL to spaces, not just the first one

https://gerrit.wikimedia.org/r/273186

Change 273208 abandoned by Anomie:
ApiSandbox: Fix String.replace call

Reason:
I45607c1e already exists.

https://gerrit.wikimedia.org/r/273208

I already submitted a patch for this when I claimed it, but the bot obviously ignored it for some reason...

The footer is "Bug:", not "Task:". You can see the bot picked it up when I adjusted that (and misattributed it to me since it goes by "uploader" rather than "author").

Change 273186 merged by jenkins-bot:
ApiSandbox: Convert all pluses in URL to spaces, not just the first one

https://gerrit.wikimedia.org/r/273186