Merge pull request #4036 from ProjectMirador/manifesto

Don't let manifesto mutate the manifest json.
ProjectMirador · Dec 13, 2024 · d105e6b · d105e6b
2 parents 514d18a + 6591559
commit d105e6b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/state/selectors/manifests.js b/src/state/selectors/manifests.js
@@ -10,7 +10,8 @@ import { getConfig } from './config';
 /** */
 function createManifestoInstance(json, locale) {
   if (!json) return undefined;
-  const manifestoObject = Utils.parseManifest(json, locale ? { locale } : undefined);
+  // Use JSON stringify/parse to create a deep copy and prevent Manifesto from mutating the json
+  const manifestoObject = Utils.parseManifest(JSON.parse(JSON.stringify(json)), locale ? { locale } : undefined);
   // Local patching of Manifesto so that when its a Collection, it behaves similarly
   if (typeof manifestoObject.getSequences != 'function') {
     manifestoObject.getSequences = () => [];