Merge pull request #589 from nextcloud/fix/loading-after-v2.13.216

Fix unknown variable after upgrade from v2.13.216
nextcloud · Apr 8, 2022 · 8749b66 · 8749b66
2 parents 258e860 + 23d937e
commit 8749b66
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 6 deletions.
diff --git a/js/files_pdfviewer-workersrc.js b/js/files_pdfviewer-workersrc.js
diff --git a/js/files_pdfviewer-workersrc.js.map b/js/files_pdfviewer-workersrc.js.map
diff --git a/src/workersrc.js b/src/workersrc.js
@@ -42,13 +42,16 @@ const canDownload = urlParams.get('canDownload')
 // "PDFViewerApplication" and "PDFViewerApplicationOptions" are globally set and
 // before "PDFViewerApplication.initialize" is executed.
 function initializeCustomPDFViewerApplication() {
+	const head = document.getElementsByTagName('head')[0]
+
 	// Preferences override options, so they must be disabled for
 	// "externalLinkTarget" to take effect.
 	PDFViewerApplicationOptions.set('disablePreferences', true)
-	PDFViewerApplicationOptions.set('externalLinkTarget', pdfjsLib.LinkTarget.BLANK)
+	// TODO https://github.com/mozilla/pdf.js/pull/14424#issuecomment-1092947792
+	PDFViewerApplicationOptions.set('externalLinkTarget', 2)
 	PDFViewerApplicationOptions.set('isEvalSupported', false)
-	PDFViewerApplicationOptions.set('workerSrc', document.getElementsByTagName('head')[0].getAttribute('data-workersrc'))
-	PDFViewerApplicationOptions.set('cMapUrl', document.getElementsByTagName('head')[0].getAttribute('data-cmapurl'))
+	PDFViewerApplicationOptions.set('workerSrc', head.getAttribute('data-workersrc'))
+	PDFViewerApplicationOptions.set('cMapUrl', head.getAttribute('data-cmapurl'))
 	PDFViewerApplicationOptions.set('enablePermissions', true)
 
 	if (canDownload === '0') {