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

Fix an incorrect condition in BaseViewer.isPageVisible (PR 10217 follow-up)

Browse Source 
This was a blatant oversight in PR 10217, since there's obviously no `this.pageNumber` property anywhere in the `BaseViewer`. Luckily this shouldn't have caused any bugs, since the only call-site is also validating the `pageNumber` (but correctly that time).
This commit is contained in:
Jonas Jenwald 2019-12-04 13:38:07 +01:00
parent 514b500a6c
commit 06b1f619c6
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
web/base_viewer.js
View File

@ -935,7 +935,7 @@ class BaseViewer {
if (!this.pdfDocument) { if (!this.pdfDocument) {
return false; return false;
} }
if (this.pageNumber < 1 || pageNumber > this.pagesCount) { if (pageNumber < 1 || pageNumber > this.pagesCount) {
console.error( console.error(
`${this._name}.isPageVisible: "${pageNumber}" is out of bounds.`); `${this._name}.isPageVisible: "${pageNumber}" is out of bounds.`);
return false; return false;