Use remote URI, not display URI, to match repositories
ClosedPublic
Actions

Authored by epriestley on Jan 28 2014, 2:42 AM.

Tags

None

Referenced Files

	F18852248: D8082.diff
	Fri, Oct 31, 7:42 AM

	F18851478: D8082.id18284.diff
	Fri, Oct 31, 1:37 AM

	F18846716: D8082.id.diff
	Wed, Oct 29, 8:43 PM

	F18843513: D8082.diff
	Tue, Oct 28, 10:49 PM

	F18798954: D8082.id18285.diff
	Fri, Oct 17, 10:14 AM

	F18777763: D8082.diff
	Sat, Oct 11, 5:06 AM

	F18633168: D8082.id.diff
	Sep 16 2025, 7:57 PM

	F18625463: D8082.diff
	Sep 15 2025, 10:52 PM

View All Files

Subscribers

aran

Details

Reviewers

btrahan
talshiri

Commits

Restricted Diffusion Commit
rPf2bc293a2aea: Use remote URI, not display URI, to match repositories

Summary

I derped this up, and it passed my tests because the two URIs are the same for hosted repositories.

Match repositories against their remote URI (like https://github.com/user/repo.git), not their display URI (like /diffusion/X/).

Test Plan

i r dums

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

smooch

Closed by commit rPf2bc293a2aea (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

storage/

PhabricatorRepository.php

6 lines

Diff 18285

View Options

src/applications/repository/storage/PhabricatorRepository.php

Use remote URI, not display URI, to match repositoriesClosedPublicActions