revi-archive/phorge-phorge
1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-26 08:42:41 +01:00
Code Issues Releases Wiki Activity

Fix URI escaping, which should actually be "%s", not "%p"

Browse source 
See <https://discourse.phabricator-community.org/t/projects-workboards-links-issue/2896>.

The documentation on `urisprintf()` isn't very clear here, I'll update it in
a followup. "%p" is for cases like encoding a branch name (which may contain
slashes) as a single path component in a URI.
This commit is contained in:
epriestley 2019-07-04 11:00:04 -07:00
parent 02315c4c48
commit 45f4211541
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
src/applications/project/state/PhabricatorWorkboardViewState.php
View file

@ -103,7 +103,7 @@ final class PhabricatorWorkboardViewState
public function newWorkboardURI($path = null) { public function newWorkboardURI($path = null) {
$project = $this->getProject(); $project = $this->getProject();
$uri = urisprintf('%p%p', $project->getWorkboardURI(), $path); $uri = urisprintf('%s%s', $project->getWorkboardURI(), $path);
return $this->newURI($uri); return $this->newURI($uri);
} }