Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Move the getPage call in PDFDocumentProperties class

Browse Source 
This allows us to remove an ESLint disable-statement for `arrow-body-style`, without affecting readability of the code, and fetching the metadata and the page in parallel should be a *tiny* bit more efficient as well.
This commit is contained in:
Jonas Jenwald 2024-12-01 12:25:16 +01:00
parent 8f08ca2150
commit e5f744da04
1 changed files with 8 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
web/pdf_document_properties.js
View File

@ -112,12 +112,13 @@ class PDFDocumentProperties {
} }
// Get the document properties. // Get the document properties.
const { const [
info, { info, /* metadata, contentDispositionFilename, */ contentLength },
/* metadata, */ pdfPage,
/* contentDispositionFilename, */ ] = await Promise.all([
contentLength, this.pdfDocument.getMetadata(),
} = await this.pdfDocument.getMetadata(); this.pdfDocument.getPage(currentPageNumber),
]);
const [ const [
fileName, fileName,
@ -131,10 +132,7 @@ class PDFDocumentProperties {
this.#parseFileSize(contentLength), this.#parseFileSize(contentLength),
this.#parseDate(info.CreationDate), this.#parseDate(info.CreationDate),
this.#parseDate(info.ModDate), this.#parseDate(info.ModDate),
// eslint-disable-next-line arrow-body-style this.#parsePageSize(getPageSizeInches(pdfPage), pagesRotation),
this.pdfDocument.getPage(currentPageNumber).then(pdfPage => {
return this.#parsePageSize(getPageSizeInches(pdfPage), pagesRotation);
}),
this.#parseLinearization(info.IsLinearized), this.#parseLinearization(info.IsLinearized),
]); ]);