@internetarchive/ia-book-actions 1.4.4-alpha18 → 1.4.4-alpha19

package/package.json

@@ -7,7 +7,7 @@
   },
   "author": "Internet Archive",
   "license": "AGPL-3.0-only",
-  "version": "1.4.4-alpha18",
+  "version": "1.4.4-alpha19",
   "main": "index.js",
   "module": "index.js",
   "publishConfig": {

package/src/ia-book-actions.js

@@ -55,6 +55,10 @@ export default class IABookActions extends LitElement {
         type: Function,
         attribute: false,
       },
+      reloadPageImages: {
+        type: Function,
+        attribute: false,
+      },
       barType: { type: String },
       sharedObserver: { attribute: false },
       disableActionGroup: { type: Boolean },
@@ -77,6 +81,7 @@ export default class IABookActions extends LitElement {
     this.width = 0;
     this.bwbPurchaseUrl = '';
     this.lendingBarPostInit = () => {};
+    this.reloadPageImages = () => {};
     this.barType = 'action'; // 'title'|'action'
     this.sharedObserver = undefined;
     this.disableActionGroup = false;
@@ -762,6 +767,12 @@ export default class IABookActions extends LitElement {
       // triggered this renewal.
       this.skipNextInitialTokenCall = true;
 
+      // A page image requested in the gap between expiry and this renewal
+      // landing can come back broken — browsers don't retry a failed <img>
+      // on their own. Retry once, right when access is confirmed good
+      // again, rather than leaving it broken until the patron reloads.
+      this.reloadPageImages();
+
       if (this.recoveringFromLoanExpiry) {
         // Only now is it safe to re-initialize BookReader — not for a
         // routine top-up, where that would be disruptive. Call it

package/test/ia-book-actions.test.js

@@ -958,6 +958,7 @@ describe('autoRenewExpiredLoan', () => {
     );
     await el.updateComplete;
     el.postInitComplete = true;
+    el.reloadPageImages = Sinon.spy();
 
     await el.localCache.set({
       key: 'foobar-loanTime',
@@ -985,6 +986,8 @@ describe('autoRenewExpiredLoan', () => {
     await el.updateComplete;
 
     expect(startTokenPollerSpy.calledOnceWith(true)).to.be.true;
+    // Retries any page image broken during the gap this renewal just closed.
+    expect(el.reloadPageImages.called).to.be.true;
   });
 
   it('calls lendingBarPostInit directly on a recovery renewal and skips the initial create_token', async () => {
@@ -1006,6 +1009,7 @@ describe('autoRenewExpiredLoan', () => {
     );
     await el.updateComplete;
     el.lendingBarPostInit = Sinon.spy();
+    el.reloadPageImages = Sinon.spy();
 
     await el.localCache.set({
       key: 'foobar-loanTime',
@@ -1030,6 +1034,7 @@ describe('autoRenewExpiredLoan', () => {
     // create_token success callback.
     expect(el.lendingBarPostInit.calledOnce).to.be.true;
     expect(el.postInitComplete).to.be.true;
+    expect(el.reloadPageImages.called).to.be.true;
 
     await el.updateComplete;
     await aTimeout(150); // let the 100ms token-poller-restart check run