Fix: Handle failed document preload for deleted representations (#968)

box · Mar 27, 2019 · 0a1d0ef · 0a1d0ef
1 parent f5c4de4
commit 0a1d0ef
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/lib/viewers/doc/DocPreloader.js b/src/lib/viewers/doc/DocPreloader.js
@@ -13,8 +13,7 @@ import {
     PDFJS_WIDTH_PADDING_PX,
     PDFJS_HEIGHT_PADDING_PX
 } from '../../constants';
-import api from '../../api';
-import { setDimensions } from '../../util';
+import { setDimensions, fetchRepresentationAsBlob } from '../../util';
 
 const EXIF_COMMENT_TAG_NAME = 'UserComment'; // Read EXIF data from 'UserComment' tag
 const EXIF_COMMENT_REGEX = /pdfWidth:([0-9.]+)pts,pdfHeight:([0-9.]+)pts,numPages:([0-9]+)/;
@@ -81,7 +80,7 @@ class DocPreloader extends EventEmitter {
         this.containerEl = containerEl;
 
         // Need to load image as a blob to read EXIF
-        return api.get(preloadUrlWithAuth, { type: 'blob' }).then((imgBlob) => {
+        return fetchRepresentationAsBlob(preloadUrlWithAuth).then((imgBlob) => {
             if (this.checkDocumentLoaded()) {
                 return;
             }

diff --git a/src/lib/viewers/doc/__tests__/DocPreloader-test.js b/src/lib/viewers/doc/__tests__/DocPreloader-test.js
@@ -50,7 +50,7 @@ describe('lib/viewers/doc/DocPreloader', () => {
 
         it('should set up preload DOM structure and bind image load handler', () => {
             const imgSrc = 'https://someblobimgsrc/';
-            sandbox.stub(api, 'get').returns(Promise.resolve({}));
+            sandbox.stub(util, 'fetchRepresentationAsBlob').returns(Promise.resolve({}));
             sandbox.stub(URL, 'createObjectURL').returns(imgSrc);
             sandbox.stub(docPreloader, 'bindDOMListeners');