Paths

Table of Contentst

Support `bin/remove destroy Fnnn` for files
ClosedPublic
Actions

Authored by epriestley on Aug 19 2014, 10:32 PM.

Tags

None

Referenced Files

	F14064056: D10300.diff
	Mon, Nov 18, 9:38 PM

	F14048404: D10300.id24804.diff
	Thu, Nov 14, 7:41 AM

	F14037990: D10300.diff
	Sun, Nov 10, 8:44 PM

	F14028169: D10300.id24804.diff
	Fri, Nov 8, 12:21 PM

	F14026286: D10300.id24804.diff
	Fri, Nov 8, 12:51 AM

	F14013628: D10300.id24804.diff
	Sat, Nov 2, 8:07 AM

	F13989262: D10300.id.diff
	Mon, Oct 21, 7:21 PM

	F13978233: D10300.id24800.diff
	Oct 18 2024, 10:14 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan
chad

Commits

Restricted Diffusion Commit
rPe8ece70ee0f2: Support `bin/remove destroy Fnnn` for files

Summary

Straightforward (this is the one object type we do let you delete from the web UI) implemetation of PhabricatorDestructibleInterface.

Test Plan

Used bin/remove destroy to destory several files. Used --trace to verify they wiped file data.

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 24800.Aug 19 2014, 10:32 PM

epriestley retitled this revision from to Support `bin/remove destroy Fnnn` for files.

epriestley updated this object.

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

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley. · View Herald TranscriptAug 19 2014, 10:32 PM

Harbormaster completed remote builds in B2278: Diff 24800.Aug 19 2014, 10:33 PM

chad accepted this revision.Aug 19 2014, 10:39 PM

chad added a reviewer: chad.

This revision is now accepted and ready to land.Aug 19 2014, 10:39 PM

Closed by commit rPe8ece70ee0f2 (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

applications/

files/

storage/

PhabricatorFile.php

14 lines

Diff 24804

View Options

src/applications/files/storage/PhabricatorFile.php

Support `bin/remove destroy Fnnn` for filesClosedPublicActions