Page MenuHomePhabricator

Render deleted files in Phragment as disabled
ClosedPublic

Authored by hach-que on Dec 7 2013, 6:40 AM.
Tags
None
Referenced Files
F18745549: D7737.id17483.diff
Fri, Oct 3, 7:50 AM
F18745525: D7737.id17501.diff
Fri, Oct 3, 7:44 AM
F18745164: D7737.diff
Fri, Oct 3, 5:32 AM
F18618405: D7737.id17502.diff
Mon, Sep 15, 12:01 AM
F18598712: D7737.diff
Sat, Sep 13, 5:40 AM
F18092026: D7737.id17502.diff
Aug 7 2025, 4:12 AM
F18077796: D7737.id.diff
Aug 4 2025, 7:30 PM
F18049070: D7737.diff
Aug 3 2025, 3:15 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T4205: Implement Phragment
Commits
Restricted Diffusion Commit
rPdabc7ea28dec: Render deleted files in Phragment as disabled
Summary

This updates Phragment so that fragments that are currently considered deleted have a disabled status and have an additional attribute 'Deleted'. It also places this effect on versions (in the history controller) that actually involve deleting the file.

Test Plan

Viewed deleted fragments and versions.

Diff Detail

Branch
render-deleted-files
Lint
Lint Passed
Unit
No Test Coverage

Event Timeline

Consider adding $fragment->isDeleted(), since I'd guess we'll need to represent this with slightly more finesse eventually (to be able to represente a file being converted into a directory and back in history).

hach-que updated this revision to Unknown Object (????).Dec 8 2013, 8:52 PM

Added isDeleted()