archive/phabricator
1
0
Fork 0
Code Issues 1 Pull Requests Packages Projects Releases Wiki Activity

Fix issue when a path is '/' in a package

Browse Source 
Summary:
when a path is '/' in defining a package, D1251 is generating
an extra '//'.

Test Plan: veryfied adding path '/', '/src' and '/src/' all worked.

Reviewers: nh, epriestley

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1266
This commit is contained in:
Jason Ge
2011-12-22 09:49:46 -08:00
parent 99a9c082f9
commit 682fb6f519
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/applications/owners/storage/package/PhabricatorOwnersPackage.php
View File

@@ -211,7 +211,9 @@ class PhabricatorOwnersPackage extends PhabricatorOwnersDAO {
if ($is_directory && substr($path, -1) != '/') { if ($is_directory && substr($path, -1) != '/') {
$path .= '/'; $path .= '/';
} }
if (substr($path, 0, 1) != '/') {
$path = '/'.$path; $path = '/'.$path;
}
if (empty($cur_paths[$repository_phid][$path]) && $valid) { if (empty($cur_paths[$repository_phid][$path]) && $valid) {
$obj = new PhabricatorOwnersPath(); $obj = new PhabricatorOwnersPath();
$obj->setPackageID($this->getID()); $obj->setPackageID($this->getID());