use sessionStorage instead of prefs for shouldDiscardData (#74)

* use sessionStorage for `shouldDiscardData` * rename prefs to session storage keys
loathers · Apr 13, 2024 · b60cd00 · b60cd00
1 parent 505c521
commit b60cd00
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/excavator-projects/projects/designerSweatpants.ts b/packages/excavator-projects/projects/designerSweatpants.ts
@@ -17,7 +17,7 @@ function spadeSweatpants(encounter: string, page: string) {
   if (!sweat) return null;
 
   const location = toNormalisedString(myLocation());
-  if (shouldDiscardData("_excavatorSweatpantsLocations", location)) return null;
+  if (shouldDiscardData("DesignerSweatpants", location)) return null;
 
   return {
     location,

diff --git a/packages/excavator-projects/projects/mummingTrunk.ts b/packages/excavator-projects/projects/mummingTrunk.ts
@@ -135,7 +135,7 @@ function spadeMummingTrunk(encounter: string, page: string) {
       );
 
       if (!match) return null;
-      if (shouldDiscardData("_excavatorMummingTrunk", `${fam.id}:${match[0]}`))
+      if (shouldDiscardData("MummingTrunk", `${fam.id}:${match[0]}`))
         return null;
 
       return {

diff --git a/packages/excavator-projects/utils.ts b/packages/excavator-projects/utils.ts
@@ -16,7 +16,7 @@ import {
   myLocation,
   myPath,
   Path,
-  setProperty,
+  sessionStorage,
 } from "kolmafia";
 
 const ALTERING_EFFECTS = Effect.get([
@@ -90,9 +90,9 @@ export function notNull<T>(value: T | null): value is T {
 }
 
 export function shouldDiscardData(property: string, data: string) {
-  const sentData = getProperty(property).split("|");
+  const sentData = (sessionStorage.getItem(property) ?? "").split("|");
   if (sentData.includes(data)) return true;
 
-  setProperty(property, [...sentData, data].join("|"));
+  sessionStorage.setItem(property, [...sentData, data].join("|"));
   return false;
 }