Page MenuHomePhabricator

Create file controller in Phragment
AbandonedPublic

Authored by hach-que on Apr 2 2014, 8:35 AM.
Tags
None
Referenced Files
F15474620: D8672.id20561.diff
Sun, Apr 6, 10:26 AM
F15455788: D8672.id20561.diff
Sun, Mar 30, 4:51 AM
F15446101: D8672.id24555.diff
Thu, Mar 27, 5:11 PM
F15445547: D8672.id.diff
Thu, Mar 27, 2:31 PM
F15441501: D8672.diff
Wed, Mar 26, 6:32 PM
F15420080: D8672.diff
Fri, Mar 21, 11:10 AM
F15419091: D8672.id24555.diff
Fri, Mar 21, 3:25 AM
F15378043: D8672.id20561.diff
Thu, Mar 13, 12:19 PM

Details

Summary

This provides a controller similar to the ZIP controller, except that it's meant for us on fragments that you want to download as a file. This controller provides a consistent URL (that can optionally contain a snapshot) which will be correct when and if a fragment gets updated, unlike the raw file link.

It also adds "Download Snapshot" on the snapshot controller, which links to the file controller with the snapshot name included.

Test Plan

Downloaded file-based fragments from the new controller.

Diff Detail

Repository
rP Phabricator
Branch
phragment-redirect-link
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:41PHL1Unknown Symbol
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:49PHL1Unknown Symbol
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:53PHL1Unknown Symbol
Unit
No Test Coverage

Event Timeline

hach-que retitled this revision from to Create file controller in Phragment.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que edited edge metadata.

Rebase on master

Might as well abandon this as we're no longer using it in production (and is probably still years away on the upstream roadmap).