fix: StackPrefetch should use the imageLoadPoolManager (#1433)

sedghi · swederik · web-flow · commit fdb0e47e4260 · 2021-10-01T09:51:23.000+02:00
* fix: StackPrefetch should use the imageLoadPoolManager

* bump cornerstone-core version

Co-authored-by: swederik &lt;erik.sweed@gmail.com&gt;
diff --git a/package.json b/package.json
@@ -54,7 +54,7 @@
     ]
   },
   "peerDependencies": {
-    "cornerstone-core": "^2.5.0"
+    "cornerstone-core": "^2.6.0"
   },
   "devDependencies": {
     "@babel/core": "^7.5.0",
diff --git a/src/stackTools/stackPrefetch.js b/src/stackTools/stackPrefetch.js
@@ -9,6 +9,8 @@ const logger = getLogger('stackTools:stackPrefetch');
 
 const toolName = 'stackPrefetch';
 const requestType = 'prefetch';
+const priority = 0;
+const addToBeginning = true;
 
 let configuration = {
   maxImagesToPrefetch: Infinity,
@@ -229,18 +231,32 @@ function prefetch(element) {
       imageIdsToPrefetch.push(imageId);
     }
   }
-  // Load images in reverse order, by adding them at the beginning of the pool.
-  for (const imageToLoad of imageIdsToPrefetch.reverse()) {
-    if (preventCache) {
-      external.cornerstone
-        .loadImage(imageToLoad, { priority: 0, requestType })
-        .then(doneCallback, failCallback);
-    } else {
-      external.cornerstone
-        .loadAndCacheImage(imageToLoad, { priority: 0, requestType })
-        .then(doneCallback, failCallback);
-    }
+
+  let requestFn;
+  const options = {
+    addToBeginning,
+    priority,
+    requestType,
+  };
+
+  if (preventCache) {
+    requestFn = id => external.cornerstone.loadImage(id, options);
+  } else {
+    requestFn = id => external.cornerstone.loadAndCacheImage(id, options);
   }
+
+  imageIdsToPrefetch.forEach(imageId => {
+    external.cornerstone.imageLoadPoolManager.addRequest(
+      requestFn.bind(null, imageId),
+      requestType,
+      // Additional details
+      {
+        imageId,
+      },
+      priority,
+      addToBeginning
+    );
+  });
 }
 
 function getPromiseRemovedHandler(element) {

Original file line number	Diff line number	Diff line change
`@@ -54,7 +54,7 @@`
`54`	`54`	`]`
`55`	`55`	`},`
`56`	`56`	`"peerDependencies": {`
`57`		`- "cornerstone-core": "^2.5.0"`
	`57`	`+ "cornerstone-core": "^2.6.0"`
`58`	`58`	`},`
`59`	`59`	`"devDependencies": {`
`60`	`60`	`"@babel/core": "^7.5.0",`