Paths

Table of Contentst

Differential D15744

Add missing RepositoryURI table + run storage adjustments in tests
ClosedPublic
Actions

Authored by epriestley on Apr 18 2016, 11:46 AM.

Tags

None

Referenced Files

	F18835387: D15744.diff
	Sun, Oct 26, 3:33 PM

	F18827216: D15744.id37941.diff
	Fri, Oct 24, 8:31 AM

	F18780759: D15744.id.diff
	Sun, Oct 12, 7:35 PM

	F18772237: D15744.id37941.diff
	Thu, Oct 9, 1:18 AM

	F18711482: D15744.id.diff
	Mon, Sep 29, 4:12 AM

	F18676428: D15744.diff
	Sep 25 2025, 7:19 PM

	F18673805: D15744.diff
	Sep 25 2025, 12:09 PM

	F18671240: D15744.id37943.diff
	Sep 25 2025, 3:12 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10366: General support for multiple URIs for a repository
T10830: phabricator_repository.repository_uri Missing

Commits

rP9352ed8abbd2: Add missing RepositoryURI table + run storage adjustments in tests

Summary

Fixes T10830. Ref T10366. I wasn't writing to this table yet so I didn't build it, but the fact that bin/storage adjust would complain slipped my mind.

Add the table.
Make the tests run adjust. This is a little slow (a few extra seconds) but we could eventually move some steps like this to run server-side only.

Test Plan

Ran bin/storage upgrade -f, got a clean adjust.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 37941.Apr 18 2016, 11:46 AM

epriestley retitled this revision from to Add missing RepositoryURI table + run storage adjustments in tests.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: chad.

epriestley added tasks: T10830: phabricator_repository.repository_uri Missing, T10366: General support for multiple URIs for a repository.

epriestley mentioned this in T10830: phabricator_repository.repository_uri Missing.Apr 18 2016, 11:54 AM

chad accepted this revision.Apr 18 2016, 1:43 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Apr 18 2016, 1:43 PM

Closed by commit rP9352ed8abbd2: Add missing RepositoryURI table + run storage adjustments in tests (authored by epriestley, committed by epriestley). · Explain WhyApr 18 2016, 2:54 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T10748: Implement `diffusion.repository.edit`, for creating and editing repositories via the API.Apr 19 2016, 9:06 PM

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20160418.repouri.1.sql

14 lines

src/

applications/

repository/

storage/

PhabricatorRepositoryURI.php

2 lines

infrastructure/

testing/

fixture/

PhabricatorStorageFixtureScopeGuard.php

2 lines

Diff 37943

resources/sql/autopatches/20160418.repouri.1.sql

Loading...

src/applications/repository/storage/PhabricatorRepositoryURI.php

Loading...

src/infrastructure/testing/fixture/PhabricatorStorageFixtureScopeGuard.php

Loading...