Page MenuHomePhabricator
Differential D19239

Add filesize limits for document rendering engines and support partial/complete rendering
ClosedPublic
Actions

Authored by epriestley on Mar 19 2018, 7:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 5, 8:28 PM
Unknown Object (File)
Fri, Nov 1, 4:58 AM
Unknown Object (File)
Wed, Oct 23, 7:42 AM
Unknown Object (File)
Oct 20 2024, 11:16 AM
Unknown Object (File)
Oct 19 2024, 1:53 PM
Unknown Object (File)
Oct 1 2024, 5:41 PM
Unknown Object (File)
Oct 1 2024, 5:09 AM
Unknown Object (File)
Sep 26 2024, 12:21 AM
View All Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13105: Plans: Rich presentation and diff rendering pipelines for various file types
Commits
rP4aafce686202: Add filesize limits for document rendering engines and support partial/complete…
Summary

Depends on D19238. Ref T13105. Give document engines some reasonable automatic support for degrading gracefully when someone tries to hexdump a 100MB file or similar.

Also, make "Video" sort above "Audio" for files which could be rendered either way.

Test Plan

Viewed audio, video, image, and other files. Adjusted limits and saw full, partial, and fallback/error rendering.

Diff Detail

Repository
rP Phabricator
Branch
doc3
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 19882
Build 26946: Run Core Tests
Build 26945: arc lint + arc unit

Event Timeline

epriestley created this revision.Mar 19 2018, 7:40 PM
Harbormaster completed remote builds in B19882: Diff 46063.Mar 19 2018, 7:41 PM
epriestley requested review of this revision.Mar 19 2018, 7:41 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 19 2018, 10:18 PM
Closed by commit rP4aafce686202: Add filesize limits for document rendering engines and support partial/complete… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 46063

View Options

resources/celerity/map.php

Loading...
View Options

src/applications/files/controller/PhabricatorFileInfoController.php

Loading...
View Options

src/applications/files/document/PhabricatorAudioDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorDocumentRef.php

Loading...
View Options

src/applications/files/document/PhabricatorHexdumpDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorImageDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorVideoDocumentEngine.php

Loading...
View Options

src/applications/files/document/PhabricatorVoidDocumentEngine.php

Loading...
View Options

webroot/rsrc/css/phui/phui-property-list-view.css

Loading...
View Options

webroot/rsrc/js/application/files/behavior-document-engine.js

Loading...
View Options

webroot/rsrc/js/phuix/PHUIXActionView.js

Loading...
Phabricator · Built by Phacility