[@mantine/hooks] use-local-storage: Fix error throwing if localStorag…

…e/sessionStorage is blocked by browser (#5091)
mantinedev · Oct 22, 2023 · a8e1615 · a8e1615
1 parent 05e26d7
commit a8e1615
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 10 deletions.
diff --git a/docs/yarn.lock b/docs/yarn.lock
@@ -133,10 +133,10 @@
   version "0.0.0"
   uid ""
 
-"@mantine/[email protected].3":
-  version "7.1.3"
-  resolved "https://registry.yarnpkg.com/@mantine/store/-/store-7.1.3.tgz#43002373de5c0aa0500656e0fc5ea90c3cd8524d"
-  integrity sha512-fjgZyW9TPk/rCewYaMygyUB6Bos0PspgYiSGCx3g81OzzdTjBhtq+xU2Lgb8YPaPhMsTDWZI+TOqceqQg12Cqw==
+"@mantine/[email protected].5":
+  version "7.1.5"
+  resolved "https://registry.yarnpkg.com/@mantine/store/-/store-7.1.5.tgz#f77457e0b2c52e24d28b9ce66ad7591272b7bd93"
+  integrity sha512-iPAt8auWyUs5TyUr31MziCILlLCYCfw6fSqPvLxOwUYpSf9BvtCAoE9JmrRrVi2q5+xO0KPSyK+OHWyBwsAqcQ==
 
 "@mantine/store@link:../src/mantine-store":
   version "0.0.0"

diff --git a/src/mantine-hooks/src/use-local-storage/create-storage.ts b/src/mantine-hooks/src/use-local-storage/create-storage.ts
@@ -81,12 +81,19 @@ export function createStorage<T>(type: StorageType, hookName: string) {
   }: StorageProperties<T>) {
     const readStorageValue = useCallback(
       (skipStorage?: boolean): T => {
-        if (
-          typeof window === 'undefined' ||
-          !(type in window) ||
-          window[type] === null ||
-          skipStorage
-        ) {
+        let storageBlockedOrSkipped;
+
+        try {
+          storageBlockedOrSkipped =
+            typeof window === 'undefined' ||
+            !(type in window) ||
+            window[type] === null ||
+            !!skipStorage;
+        } catch (_e) {
+          storageBlockedOrSkipped = true;
+        }
+
+        if (storageBlockedOrSkipped) {
           return defaultValue as T;
         }