Remove all "FileHasObject" edge reads and writes
ClosedPublic
Actions

Authored by epriestley on May 12 2022, 10:25 PM.

Tags

None

Referenced Files

	F18993009: D21820.diff
	Wed, Nov 19, 4:56 AM

	F18877759: D21820.diff
	Thu, Nov 6, 11:52 AM

	F18852259: D21820.id.diff
	Fri, Oct 31, 7:44 AM

	F18817656: D21820.id.diff
	Oct 21 2025, 5:18 PM

	F18809507: D21820.id.diff
	Oct 19 2025, 2:38 PM

	F18803501: D21820.id52000.diff
	Oct 18 2025, 12:35 AM

	F18802863: D21820.id52028.diff
	Oct 17 2025, 8:42 PM

	F18787519: D21820.id52000.diff
	Oct 15 2025, 12:08 AM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: Restricted Maniphest Task
Commits: rPb872640c160a: Remove all "FileHasObject" edge reads and writes

Summary

Ref T13603. Removes all reads and writes to the other half of the legacy edge.

Test Plan

Verified I could still see file content accessible only through object attachment.
This fixes a unit test broken by removal of only half the edge access in the previous change.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley requested review of this revision.May 12 2022, 10:25 PM

epriestley created this revision.

Harbormaster completed remote builds in B25719: Diff 52000.May 12 2022, 10:25 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 19 2022, 8:21 PM

Closed by commit rPb872640c160a: Remove all "FileHasObject" edge reads and writes (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rPb872640c160a: Remove all "FileHasObject" edge reads and writes.

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

query/

PhabricatorFileQuery.php

34 lines

Diff 52028

View Options

src/applications/files/query/PhabricatorFileQuery.php

Remove all "FileHasObject" edge reads and writesClosedPublicActions