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

	Unknown Object (File)
	Wed, Sep 4, 10:47 PM

	Unknown Object (File)
	Sun, Sep 1, 9:07 AM

	Unknown Object (File)
	Sun, Sep 1, 1:57 AM

	Unknown Object (File)
	Sat, Aug 31, 1:22 PM

	Unknown Object (File)
	Sat, Aug 31, 11:29 AM

	Unknown Object (File)
	Sat, Aug 31, 5:51 AM

	Unknown Object (File)
	Thu, Aug 29, 11:44 AM

	Unknown Object (File)
	Wed, Aug 28, 6:14 AM

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