Fixes for various small issues caught by sentry

edgedb · Jan 16, 2025 · 2c3dd4b · 2c3dd4b
1 parent bed34a7
commit 2c3dd4b
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/shared/studio/tabs/dataview/state/index.ts b/shared/studio/tabs/dataview/state/index.ts
@@ -948,10 +948,13 @@ export class DataInspector extends Model({
       return;
     }
 
-    const dbState = dbCtx.get(this)!;
-
     this.runningDataFetch?.abortController.abort();
 
+    const dbState = dbCtx.get(this);
+    if (!dbState) {
+      return;
+    }
+
     this.runningDataFetch = {abortController: new AbortController(), offset};
     dbState.setLoadingTab(DataView, true);
 

diff --git a/shared/studio/tabs/queryEditor/state/index.ts b/shared/studio/tabs/queryEditor/state/index.ts
@@ -819,7 +819,6 @@ export class QueryEditor extends Model({
       });
       return {success: true, capabilities, status};
     } catch (e: any) {
-      console.error(e);
       this.addHistoryCell({
         queryData,
         timestamp,

diff --git a/shared/studio/tabs/schema/renderers/module.tsx b/shared/studio/tabs/schema/renderers/module.tsx
@@ -26,7 +26,10 @@ export function ModuleRenderer({
     if (isSticky) {
       const observer = new IntersectionObserver(
         ([e]) => {
-          if (e.boundingClientRect.top < e.rootBounds!.height / 2) {
+          if (
+            e.rootBounds &&
+            e.boundingClientRect.top < e.rootBounds.height / 2
+          ) {
             setIsStuck(e.intersectionRatio < 1);
           }
         },

diff --git a/shared/studio/tabs/schema/state/textView.ts b/shared/studio/tabs/schema/state/textView.ts
@@ -144,8 +144,8 @@ export class SchemaTextView extends Model({
   }
 
   getRenderHeight(index: number) {
-    const item = this.renderListItems.itemsList[index].item;
-    return item.schemaType === "Module"
+    const item = this.renderListItems.itemsList[index]?.item;
+    return !item || item.schemaType === "Module"
       ? 42
       : this.renderHeights.get(item.id) ?? 42;
   }