mirror of
https://we.phorge.it/source/phorge.git
synced 2024-12-19 03:50:54 +01:00
Remove some ad-hoc loading of repositories from Releeph
Summary: Ref T3551. Since we now require repositories in order to perform policy checks, things that did loads properly don't need to load this data explicitly. Test Plan: Edited a product, cut a new branch. Reviewers: btrahan Reviewed By: btrahan Subscribers: epriestley Maniphest Tasks: T3551 Differential Revision: https://secure.phabricator.com/D8769
This commit is contained in:
parent
f4c8a34abe
commit
47245b1b59
3 changed files with 2 additions and 6 deletions
|
@ -31,11 +31,10 @@ final class ReleephBranchCreateController extends ReleephProductController {
|
|||
$symbolic_name = $request->getStr('symbolicName');
|
||||
|
||||
if (!$cut_point) {
|
||||
$repository = $product->loadPhabricatorRepository();
|
||||
$repository = $product->getRepository();
|
||||
switch ($repository->getVersionControlSystem()) {
|
||||
case PhabricatorRepositoryType::REPOSITORY_TYPE_SVN:
|
||||
break;
|
||||
|
||||
case PhabricatorRepositoryType::REPOSITORY_TYPE_GIT:
|
||||
$cut_point = $product->getTrunkBranch();
|
||||
break;
|
||||
|
|
|
@ -139,9 +139,7 @@ final class ReleephProductEditController extends ReleephProductController {
|
|||
id(new AphrontFormStaticControl())
|
||||
->setLabel(pht('Repository'))
|
||||
->setValue(
|
||||
$product
|
||||
->loadPhabricatorRepository()
|
||||
->getName()))
|
||||
$product->getRepository()->getName()))
|
||||
->appendChild(
|
||||
id(new AphrontFormStaticControl())
|
||||
->setLabel(pht('Arc Project'))
|
||||
|
|
|
@ -7,7 +7,6 @@ final class ReleephProjectQuery
|
|||
private $ids;
|
||||
private $phids;
|
||||
|
||||
private $needRepositories;
|
||||
private $needArcanistProjects;
|
||||
|
||||
private $order = 'order-id';
|
||||
|
|
Loading…
Reference in a new issue