Cache data on Folder Progress Bar Worker

Currently, all information needed for calculating the folder progress
bar is sent to the worker on every click on a new resource. That is slow
and not necessary, as most of the data just changes after saving.
Therefore, in this commit, the progress-bar-worker is modified such that
it caches information and only reloads the cache after saving.

Signed-off-by: Florian Schepers <florianschepers@gmx.de>

Author: FlorianSchepers

src/Frontend/Components/ProgressBar/FolderProgressBar.tsx

@@ -72,18 +72,8 @@ export function FolderProgressBar(props: FolderProgressBarProps): ReactElement {
   const folderProgressBarWorkerArgs = useMemo(
     () => ({
       resourceId,
-      manualAttributions,
-      externalAttributions,
-      resourcesToManualAttributions,
-      resolvedExternalAttributions,
     }),
-    [
-      resourceId,
-      manualAttributions,
-      externalAttributions,
-      resourcesToManualAttributions,
-      resolvedExternalAttributions,
-    ],
+    [resourceId],
   );
 
   const folderProgressBarSyncFallbackArgs = useMemo(

src/Frontend/Components/WorkersContextProvider/WorkersContextProvider.tsx

@@ -7,16 +7,20 @@ import { FC, ReactNode, createContext, useMemo } from 'react';
 import { useAppSelector } from '../../state/hooks';
 import {
   getAttributionBreakpoints,
+  getExternalAttributions,
   getExternalAttributionsToHashes,
   getExternalData,
   getFilesWithChildren,
+  getManualAttributions,
   getManualData,
   getResources,
   getResourcesToExternalAttributions,
+  getResourcesToManualAttributions,
 } from '../../state/selectors/all-views-resource-selectors';
 import { getNewAccordionWorkers } from '../../web-workers/get-new-accordion-workers';
 import { getNewProgressBarWorkers } from '../../web-workers/get-new-progress-bar-workers';
 import { PanelAttributionData } from '../../util/get-contained-packages';
+import { getResolvedExternalAttributions } from '../../state/selectors/audit-view-resource-selectors';
 
 const resourceDetailsTabsWorkers = getNewAccordionWorkers();
 
@@ -93,9 +97,17 @@ export const ProgressBarWorkersContextProvider: FC<{
   children: ReactNode | null;
 }> = ({ children }) => {
   const resources = useAppSelector(getResources);
+  const manualAttributions = useAppSelector(getManualAttributions);
+  const externalAttributions = useAppSelector(getExternalAttributions);
+  const resourcesToManualAttributions = useAppSelector(
+    getResourcesToManualAttributions,
+  );
   const resourcesToExternalAttributions = useAppSelector(
     getResourcesToExternalAttributions,
   );
+  const resolvedExternalAttributions = useAppSelector(
+    getResolvedExternalAttributions,
+  );
   const attributionBreakpoints = useAppSelector(getAttributionBreakpoints);
   const filesWithChildren = useAppSelector(getFilesWithChildren);
 
@@ -104,6 +116,7 @@ export const ProgressBarWorkersContextProvider: FC<{
       progressBarWorkers.TopProgressBarWorker.postMessage({
         isCacheInitializationMessage: true,
         resources: null,
+        externalAttributions: null,
         resourcesToExternalAttributions: null,
         attributionBreakpoints: null,
         filesWithChildren: null,
@@ -112,6 +125,7 @@ export const ProgressBarWorkersContextProvider: FC<{
         worker.postMessage({
           isCacheInitializationMessage: true,
           resources,
+          externalAttributions,
           resourcesToExternalAttributions,
           attributionBreakpoints,
           filesWithChildren,
@@ -122,11 +136,35 @@ export const ProgressBarWorkersContextProvider: FC<{
     }
   }, [
     resources,
+    externalAttributions,
     resourcesToExternalAttributions,
     attributionBreakpoints,
     filesWithChildren,
   ]);
 
+  useMemo(() => {
+    try {
+      progressBarWorkers.TopProgressBarWorker.postMessage({
+        manualAttributions: null,
+        resourcesToManualAttributions: null,
+        resolvedExternalAttributions: null,
+      });
+      Object.values(progressBarWorkers).forEach((worker) => {
+        worker.postMessage({
+          manualAttributions,
+          resourcesToManualAttributions,
+          resolvedExternalAttributions,
+        });
+      });
+    } catch (error) {
+      console.info('Web worker error in workers context provider: ', error);
+    }
+  }, [
+    manualAttributions,
+    resourcesToManualAttributions,
+    resolvedExternalAttributions,
+  ]);
+
   return (
     <ProgressBarWorkersContext.Provider value={progressBarWorkers}>
       {children}

src/Frontend/web-workers/progress-bar-worker.ts

@@ -2,15 +2,23 @@
 //
 // SPDX-License-Identifier: Apache-2.0
 
-import { Resources, ResourcesToAttributions } from '../../shared/shared-types';
+import {
+  Attributions,
+  Resources,
+  ResourcesToAttributions,
+} from '../../shared/shared-types';
 import { getUpdatedProgressBarData } from '../state/helpers/progress-bar-data-helpers';
 import { ProgressBarDataAndResourceId } from '../types/types';
 
 let cachedResources: Resources | null = null;
+let cachedExternalAttributions: Attributions | null = null;
 let cachedResourcesToExternalAttributions: ResourcesToAttributions | null =
   null;
 let cachedAttributionBreakpoints: Set<string> | null = null;
 let cachedFilesWithChildren: Set<string> | null = null;
+let cachedManualAttributions: Attributions | null = null;
+let cachedResourcesToManualAttributions: ResourcesToAttributions | null = null;
+let cachedResolvedExternalAttributions: Set<string> | null = null;
 
 self.onmessage = ({
   data: {
@@ -26,12 +34,29 @@ self.onmessage = ({
     filesWithChildren,
   },
 }): void => {
+  if (
+    manualAttributions !== undefined &&
+    resourcesToManualAttributions !== undefined &&
+    resolvedExternalAttributions !== undefined
+  ) {
+    cachedManualAttributions = manualAttributions;
+    cachedResourcesToManualAttributions = resourcesToManualAttributions;
+    cachedResolvedExternalAttributions = resolvedExternalAttributions;
+  }
+
   if (isCacheInitializationMessage) {
     cachedResources = resources;
+    cachedExternalAttributions = externalAttributions;
     cachedResourcesToExternalAttributions = resourcesToExternalAttributions;
     cachedAttributionBreakpoints = attributionBreakpoints;
     cachedFilesWithChildren = filesWithChildren;
-  } else if (
+  }
+
+  if (
+    cachedManualAttributions &&
+    cachedExternalAttributions &&
+    cachedResourcesToManualAttributions &&
+    cachedResolvedExternalAttributions &&
     cachedResources &&
     cachedResourcesToExternalAttributions &&
     cachedAttributionBreakpoints &&
@@ -40,11 +65,11 @@ self.onmessage = ({
     const progressBarData = getUpdatedProgressBarData({
       resources: cachedResources,
       resourceId,
-      manualAttributions,
-      externalAttributions,
-      resourcesToManualAttributions,
+      manualAttributions: cachedManualAttributions,
+      externalAttributions: cachedExternalAttributions,
+      resourcesToManualAttributions: cachedResourcesToManualAttributions,
       resourcesToExternalAttributions: cachedResourcesToExternalAttributions,
-      resolvedExternalAttributions,
+      resolvedExternalAttributions: cachedResolvedExternalAttributions,
       attributionBreakpoints: cachedAttributionBreakpoints,
       filesWithChildren: cachedFilesWithChildren,
     });