Page MenuHomePhabricator
Differential D11586

Policy - remove loadRepository() method from ArcanistProjects
ClosedPublic
Actions

Authored by btrahan on Jan 30 2015, 11:04 PM.
Tags
None
Referenced Files
F18105341: D11586.diff
Sun, Aug 10, 5:43 PM
F18088563: D11586.diff
Wed, Aug 6, 8:26 AM
F17923048: D11586.id.diff
Wed, Jul 30, 7:40 AM
F17916391: D11586.diff
Tue, Jul 29, 10:05 PM
F17733855: D11586.id27989.diff
Jul 20 2025, 9:18 AM
F17732385: D11586.id27880.diff
Jul 20 2025, 4:38 AM
F17731289: D11586.diff
Jul 20 2025, 12:58 AM
Unknown Object (File)
Jun 28 2025, 9:39 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T7094: Clean up T603
Commits
Restricted Diffusion Commit
rPf58dce681905: Policy - remove loadRepository() method from ArcanistProjects
Summary

Ref T7094. This loadRepository() method bypassed policy unnecessarily. kill it.

Test Plan

basically un-tested since arcanist projects are deprecated and the main callsites were in releeph. conduit end point still works though!

Diff Detail

Repository
rP Phabricator
Branch
T7094_5
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4194
Build 4207: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 27880.Jan 30 2015, 11:04 PM
btrahan added a task: T7094: Clean up T603.
btrahan retitled this revision from to Policy - remove loadRepository() method from ArcanistProjects.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptJan 30 2015, 11:04 PM
epriestley accepted this revision.Jan 31 2015, 2:24 PM
epriestley edited edge metadata.

yeahhhhh

This revision is now accepted and ready to land.Jan 31 2015, 2:24 PM
Closed by commit rPf58dce681905: Policy - remove loadRepository() method from ArcanistProjects (authored by btrahan, committed by btrahan). · Explain WhyFeb 2 2015, 9:58 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
arcanist/
conduit/
8 lines
releeph/
controller/
branch/
4 lines
product/
9 lines
2 lines
view/
branch/
21 lines
repository/
storage/
8 lines

Diff 27880

View Options

src/applications/arcanist/conduit/ArcanistProjectInfoConduitAPIMethod.php

Loading...
View Options

src/applications/releeph/controller/branch/ReleephBranchNamePreviewController.php

Loading...
View Options

src/applications/releeph/controller/product/ReleephProductCreateController.php

Loading...
View Options

src/applications/releeph/controller/product/ReleephProductEditController.php

Loading...
View Options

src/applications/releeph/view/branch/ReleephBranchTemplate.php

Loading...
View Options

src/applications/repository/storage/PhabricatorRepositoryArcanistProject.php

Loading...
Phabricator · Built by Phacility