From 93e7f630c10b7c0585bcb290b1c4217673a71720 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 20 May 2020 11:30:03 +0200 Subject: [PATCH] Remove unnecessary empty string fallback from the `getPDFFileNameFromURL` call in `web/pdf_document_properties.js` (PR 10114 follow-up) Given that the `getPDFFileNameFromURL` helper function has a specific code-path for handling non-string inputs, this empty string fallback really isn't necessary at the call-site in `web/pdf_document_properties.js`. --- web/pdf_document_properties.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/pdf_document_properties.js b/web/pdf_document_properties.js index f5c5c5cf3..8b38fb08e 100644 --- a/web/pdf_document_properties.js +++ b/web/pdf_document_properties.js @@ -132,7 +132,7 @@ class PDFDocumentProperties { return Promise.all([ info, metadata, - contentDispositionFilename || getPDFFileNameFromURL(this.url || ""), + contentDispositionFilename || getPDFFileNameFromURL(this.url), this._parseFileSize(this.maybeFileSize), this._parseDate(info.CreationDate), this._parseDate(info.ModDate),