Differential D21835

Give "FileAttachment" policy support and a query object
ClosedPublic
Actions

Authored by epriestley on May 23 2022, 5:24 PM.

Tags

None

Referenced Files

	F12194148: D21835.diff
	Sun, Sep 24, 4:24 PM

	F12192704: D21835.diff
	Sat, Sep 23, 2:52 AM

	F12191910: D21835.id52038.diff
	Fri, Sep 22, 8:17 PM

	F12191443: D21835.diff
	Fri, Sep 22, 11:12 AM

	F12184777: D21835.diff
	Fri, Sep 15, 10:56 PM

	F12183258: D21835.diff
	Thu, Sep 14, 9:22 AM

	F12171540: D21835.diff
	Fri, Sep 8, 7:01 AM

	F12162473: D21835.diff
	Wed, Sep 6, 8:18 PM

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13682: Improve Remarkup file reference attachment behaviors after removal of "attach by default"
Commits: rP5033ef6f88b3: Give "FileAttachment" policy support and a query object

Summary

Ref T13682. This supports an "Attached Files" curtain UI element.

Test Plan

See next change.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley requested review of this revision.May 23 2022, 5:24 PM

epriestley created this revision.

Harbormaster completed remote builds in B25737: Diff 52037.May 23 2022, 5:24 PM

Fix unit test.

Harbormaster completed remote builds in B25738: Diff 52038.May 23 2022, 5:33 PM

Also fix indirect reference attachment/scrambling test.

Harbormaster completed remote builds in B25739: Diff 52039.May 23 2022, 5:49 PM

Further improve unit test coverage and test "reference doesn't grant view permission".

Harbormaster completed remote builds in B25740: Diff 52040.May 23 2022, 6:11 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 23 2022, 9:08 PM

Closed by commit rP5033ef6f88b3: Give "FileAttachment" policy support and a query object (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP5033ef6f88b3: Give "FileAttachment" policy support and a query object.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

applications/

files/

query/

PhabricatorFileAttachmentQuery.php

110 lines

PhabricatorFileQuery.php

16 lines

storage/

PhabricatorFileAttachment.php

57 lines

__tests__/

PhabricatorFileTestCase.php

61 lines

transactions/

editengine/

PhabricatorEditEngine.php

20 lines

editor/

PhabricatorApplicationTransactionEditor.php

7 lines

Diff 52044

src/__phutil_library_map__.php

Loading...

src/applications/files/query/PhabricatorFileAttachmentQuery.php

Loading...

src/applications/files/query/PhabricatorFileQuery.php

Loading...

src/applications/files/storage/PhabricatorFileAttachment.php

Loading...

src/applications/files/storage/tests/PhabricatorFileTestCase.php

Loading...

src/applications/transactions/editengine/PhabricatorEditEngine.php

Loading...

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...