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

Merge pull request #11889 from Snuffleupagus/_parsedAnnotations-move-catch

Browse Source 
Handle errors individually for each annotation in the `_parsedAnnotations` getter
This commit is contained in:
Tim van der Meij 2020-05-10 00:22:13 +02:00 committed by GitHub
commit 1ee63dc465
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 9 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
src/core/document.js
View File

@ -393,30 +393,24 @@ class Page {
const parsedAnnotations = this.pdfManager
.ensure(this, "annotations")
.then(() => {
const annotationRefs = this.annotations;
const annotationPromises = [];
for (let i = 0, ii = annotationRefs.length; i < ii; i++) {
for (const annotationRef of this.annotations) {
annotationPromises.push(
AnnotationFactory.create(
this.xref,
annotationRefs[i],
annotationRef,
this.pdfManager,
this.idFactory
)
).catch(function (reason) {
warn(`_parsedAnnotations: "${reason}".`);
return null;
})
);
}
return Promise.all(annotationPromises).then(
function (annotations) {
return annotations.filter(function isDefined(annotation) {
return !!annotation;
});
},
function (reason) {
warn(`_parsedAnnotations: "${reason}".`);
return [];
}
);
return Promise.all(annotationPromises).then(function (annotations) {
return annotations.filter(annotation => !!annotation);
});
});
return shadow(this, "_parsedAnnotations", parsedAnnotations);