Paths

Table of Contentst

Differential D8072

Update DifferentialDiff: add repositoryPHID, drop parentRevisionID
ClosedPublic
Actions

Authored by epriestley on Jan 26 2014, 5:18 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 25, 10:24 PM

	Unknown Object (File)
	Thu, Dec 12, 5:54 AM

	Unknown Object (File)
	Sun, Dec 8, 6:07 AM

	Unknown Object (File)
	Wed, Dec 4, 1:04 AM

	Unknown Object (File)
	Sun, Dec 1, 7:11 AM

	Unknown Object (File)
	Sat, Nov 30, 6:00 AM

	Unknown Object (File)
	Nov 26 2024, 1:09 AM

	Unknown Object (File)
	Nov 21 2024, 4:33 PM

Subscribers

Details

Reviewers

Commits

Restricted Diffusion Commit
rP014a873773eb: Update DifferentialDiff: add repositoryPHID, drop parentRevisionID

Summary

Moves away from ArcanistProjects:

Adds storage for diffs to be directly associated with a repository (instead of indirectly, through arcanist projects). Not really populated yet.
Drops parentRevisionID, which is obsoleted by the "Depends On" edge. This is not exposed in the UI anywhere and doesn't do anything. Resolves TODO.

Test Plan

Ran storage upgrades, browsed around, lots of grep.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

Also deprecate authorPHID, which isn't respected and is inauthentic.
Also deprecate repositoryUUID, which is redundant. Keep some traces of it around for compatibility.
Add UUID lookup, which is useful for SVN and can completely obsolete the older mechanism.

btrahan accepted this revision.Jan 26 2014, 11:10 PM

Closed by commit rP014a873773eb (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20140126.diff.1.parentrevisionid.sql

2 lines

20140126.diff.2.repositoryphid.sql

2 lines

src/

applications/

differential/

conduit/

ConduitAPI_differential_creatediff_Method.php

44 lines

storage/

DifferentialDiff.php

3 lines

repository/

conduit/

ConduitAPI_repository_query_Method.php

6 lines

Diff 18271

resources/sql/autopatches/20140126.diff.1.parentrevisionid.sql

Loading...

resources/sql/autopatches/20140126.diff.2.repositoryphid.sql

Loading...

src/applications/differential/conduit/ConduitAPI_differential_creatediff_Method.php

Loading...

src/applications/differential/storage/DifferentialDiff.php

Loading...

src/applications/repository/conduit/ConduitAPI_repository_query_Method.php

Loading...