D25464.1737213467.diff
No OneTemporary
Actions

Size

1 KB

Referenced Files

None

Subscribers

None

D25464.1737213467.diff
View Options

	diff --git a/src/applications/files/document/PhabricatorPDFDocumentEngine.php b/src/applications/files/document/PhabricatorPDFDocumentEngine.php
	--- a/src/applications/files/document/PhabricatorPDFDocumentEngine.php
	+++ b/src/applications/files/document/PhabricatorPDFDocumentEngine.php
	@@ -14,14 +14,16 @@
	}

	protected function canRenderDocumentType(PhabricatorDocumentRef $ref) {
	- // Since we just render a link to the document anyway, we don't need to
	- // check anything fancy in config to see if the MIME type is actually
	- // viewable.
	+ $viewable_types = PhabricatorEnv::getEnvConfig('files.viewable-mime-types');
	+ $viewable_types = array_keys($viewable_types);

	- return $ref->hasAnyMimeType(
	- array(
	- 'application/pdf',
	- ));
	+ $pdf_types = array(
	+ 'application/pdf',
	+ );
	+
	+ return
	+ $ref->hasAnyMimeType($viewable_types) &&
	+ $ref->hasAnyMimeType($pdf_types);
	}

	protected function newDocumentContent(PhabricatorDocumentRef $ref) {